diff options
Diffstat (limited to 'spec/features/triggers_spec.rb')
-rw-r--r-- | spec/features/triggers_spec.rb | 112 |
1 files changed, 56 insertions, 56 deletions
diff --git a/spec/features/triggers_spec.rb b/spec/features/triggers_spec.rb index 919859c145a..dbe9b1bda97 100644 --- a/spec/features/triggers_spec.rb +++ b/spec/features/triggers_spec.rb @@ -1,7 +1,7 @@ -require 'spec_helper' +require "spec_helper" -describe 'Triggers', :js do - let(:trigger_title) { 'trigger desc' } +describe "Triggers", :js do + let(:trigger_title) { "trigger desc" } let(:user) { create(:user) } let(:user2) { create(:user) } let(:guest_user) { create(:user) } @@ -17,51 +17,51 @@ describe 'Triggers', :js do visit project_settings_ci_cd_path(@project) end - describe 'create trigger workflow' do - it 'prevents adding new trigger with no description' do - fill_in 'trigger_description', with: '' - click_button 'Add trigger' + describe "create trigger workflow" do + it "prevents adding new trigger with no description" do + fill_in "trigger_description", with: "" + click_button "Add trigger" # See if input has error due to empty value - expect(page.find('form.gl-show-field-errors .gl-field-error')).to be_visible + expect(page.find("form.gl-show-field-errors .gl-field-error")).to be_visible end - it 'adds new trigger with description' do - fill_in 'trigger_description', with: 'trigger desc' - click_button 'Add trigger' + it "adds new trigger with description" do + fill_in "trigger_description", with: "trigger desc" + click_button "Add trigger" # See if "trigger creation successful" message displayed and description and owner are correct - expect(page.find('.flash-notice')).to have_content 'Trigger was created successfully.' - expect(page.find('.triggers-list')).to have_content 'trigger desc' - expect(page.find('.triggers-list .trigger-owner')).to have_content user.name + expect(page.find(".flash-notice")).to have_content "Trigger was created successfully." + expect(page.find(".triggers-list")).to have_content "trigger desc" + expect(page.find(".triggers-list .trigger-owner")).to have_content user.name end end - describe 'edit trigger workflow' do - let(:new_trigger_title) { 'new trigger' } + describe "edit trigger workflow" do + let(:new_trigger_title) { "new trigger" } - it 'click on edit trigger opens edit trigger page' do + it "click on edit trigger opens edit trigger page" do create(:ci_trigger, owner: user, project: @project, description: trigger_title) visit project_settings_ci_cd_path(@project) # See if edit page has correct descrption find('a[title="Edit"]').send_keys(:return) - expect(page.find('#trigger_description').value).to have_content 'trigger desc' + expect(page.find("#trigger_description").value).to have_content "trigger desc" end - it 'edit trigger and save' do + it "edit trigger and save" do create(:ci_trigger, owner: user, project: @project, description: trigger_title) visit project_settings_ci_cd_path(@project) # See if edit page opens, then fill in new description and save find('a[title="Edit"]').send_keys(:return) - fill_in 'trigger_description', with: new_trigger_title - click_button 'Save trigger' + fill_in "trigger_description", with: new_trigger_title + click_button "Save trigger" # See if "trigger updated successfully" message displayed and description and owner are correct - expect(page.find('.flash-notice')).to have_content 'Trigger was successfully updated.' - expect(page.find('.triggers-list')).to have_content new_trigger_title - expect(page.find('.triggers-list .trigger-owner')).to have_content user.name + expect(page.find(".flash-notice")).to have_content "Trigger was successfully updated." + expect(page.find(".triggers-list")).to have_content new_trigger_title + expect(page.find(".triggers-list .trigger-owner")).to have_content user.name end it 'edit "legacy" trigger and save' do @@ -71,13 +71,13 @@ describe 'Triggers', :js do # See if the trigger can be edited and description is blank find('a[title="Edit"]').send_keys(:return) - expect(page.find('#trigger_description').value).to have_content '' + expect(page.find("#trigger_description").value).to have_content "" # See if trigger can be updated with description and saved successfully - fill_in 'trigger_description', with: new_trigger_title - click_button 'Save trigger' - expect(page.find('.flash-notice')).to have_content 'Trigger was successfully updated.' - expect(page.find('.triggers-list')).to have_content new_trigger_title + fill_in "trigger_description", with: new_trigger_title + click_button "Save trigger" + expect(page.find(".flash-notice")).to have_content "Trigger was successfully updated." + expect(page.find(".triggers-list")).to have_content new_trigger_title end end @@ -88,19 +88,19 @@ describe 'Triggers', :js do end it 'button "Take ownership" has correct alert' do - expected_alert = 'By taking ownership you will bind this trigger to your user account. With this the trigger will have access to all your projects as if it was you. Are you sure?' - expect(page.find('a.btn-trigger-take-ownership')['data-confirm']).to eq expected_alert + expected_alert = "By taking ownership you will bind this trigger to your user account. With this the trigger will have access to all your projects as if it was you. Are you sure?" + expect(page.find("a.btn-trigger-take-ownership")["data-confirm"]).to eq expected_alert end - it 'take trigger ownership' do + it "take trigger ownership" do # See if "Take ownership" on trigger works post trigger creation page.accept_confirm do first(:link, "Take ownership").send_keys(:return) end - expect(page.find('.flash-notice')).to have_content 'Trigger was re-assigned.' - expect(page.find('.triggers-list')).to have_content trigger_title - expect(page.find('.triggers-list .trigger-owner')).to have_content user.name + expect(page.find(".flash-notice")).to have_content "Trigger was re-assigned." + expect(page.find(".triggers-list")).to have_content trigger_title + expect(page.find(".triggers-list .trigger-owner")).to have_content user.name end end @@ -111,24 +111,24 @@ describe 'Triggers', :js do end it 'button "Revoke" has correct alert' do - expected_alert = 'By revoking a trigger you will break any processes making use of it. Are you sure?' - expect(page.find('a.btn-trigger-revoke')['data-confirm']).to eq expected_alert + expected_alert = "By revoking a trigger you will break any processes making use of it. Are you sure?" + expect(page.find("a.btn-trigger-revoke")["data-confirm"]).to eq expected_alert end - it 'revoke trigger' do + it "revoke trigger" do # See if "Revoke" on trigger works post trigger creation page.accept_confirm do - find('a.btn-trigger-revoke').send_keys(:return) + find("a.btn-trigger-revoke").send_keys(:return) end - expect(page.find('.flash-notice')).to have_content 'Trigger removed' - expect(page).to have_selector('p.settings-message.text-center.append-bottom-default') + expect(page.find(".flash-notice")).to have_content "Trigger removed" + expect(page).to have_selector("p.settings-message.text-center.append-bottom-default") end end - describe 'show triggers workflow' do - it 'contains trigger description placeholder' do - expect(page.find('#trigger_description')['placeholder']).to eq 'Trigger description' + describe "show triggers workflow" do + it "contains trigger description placeholder" do + expect(page.find("#trigger_description")["placeholder"]).to eq "Trigger description" end it 'show "legacy" badge for legacy trigger' do @@ -136,8 +136,8 @@ describe 'Triggers', :js do visit project_settings_ci_cd_path(@project) # See if trigger without owner (i.e. legacy) shows "legacy" badge and is editable - expect(page.find('.triggers-list')).to have_content 'legacy' - expect(page.find('.triggers-list')).to have_selector('a[title="Edit"]') + expect(page.find(".triggers-list")).to have_content "legacy" + expect(page.find(".triggers-list")).to have_selector('a[title="Edit"]') end it 'show "invalid" badge for trigger with owner having insufficient permissions' do @@ -145,8 +145,8 @@ describe 'Triggers', :js do visit project_settings_ci_cd_path(@project) # See if trigger without owner (i.e. legacy) shows "legacy" badge and is non-editable - expect(page.find('.triggers-list')).to have_content 'invalid' - expect(page.find('.triggers-list')).not_to have_selector('a[title="Edit"]') + expect(page.find(".triggers-list")).to have_content "invalid" + expect(page.find(".triggers-list")).not_to have_selector('a[title="Edit"]') end it 'do not show "Edit" or full token for not owned trigger' do @@ -155,12 +155,12 @@ describe 'Triggers', :js do visit project_settings_ci_cd_path(@project) # See if trigger not owned by current_user shows only first few token chars and doesn't have copy-to-clipboard button - expect(page.find('.triggers-list')).to have_content(@project.triggers.first.token[0..3]) - expect(page.find('.triggers-list')).not_to have_selector('button.btn-clipboard') + expect(page.find(".triggers-list")).to have_content(@project.triggers.first.token[0..3]) + expect(page.find(".triggers-list")).not_to have_selector("button.btn-clipboard") # See if trigger owner name doesn't match with current_user and trigger is non-editable - expect(page.find('.triggers-list .trigger-owner')).not_to have_content user.name - expect(page.find('.triggers-list')).not_to have_selector('a[title="Edit"]') + expect(page.find(".triggers-list .trigger-owner")).not_to have_content user.name + expect(page.find(".triggers-list")).not_to have_selector('a[title="Edit"]') end it 'show "Edit" and full token for owned trigger' do @@ -168,12 +168,12 @@ describe 'Triggers', :js do visit project_settings_ci_cd_path(@project) # See if trigger shows full token and has copy-to-clipboard button - expect(page.find('.triggers-list')).to have_content @project.triggers.first.token - expect(page.find('.triggers-list')).to have_selector('button.btn-clipboard') + expect(page.find(".triggers-list")).to have_content @project.triggers.first.token + expect(page.find(".triggers-list")).to have_selector("button.btn-clipboard") # See if trigger owner name matches with current_user and is editable - expect(page.find('.triggers-list .trigger-owner')).to have_content user.name - expect(page.find('.triggers-list')).to have_selector('a[title="Edit"]') + expect(page.find(".triggers-list .trigger-owner")).to have_content user.name + expect(page.find(".triggers-list")).to have_selector('a[title="Edit"]') end end end |