diff options
author | Jose Ivan Vargas <jvargas@gitlab.com> | 2017-09-29 13:07:04 -0500 |
---|---|---|
committer | Jose Ivan Vargas <jvargas@gitlab.com> | 2017-09-29 13:07:27 -0500 |
commit | b452b0c7056e481778bfc5838d47dca813bf12ed (patch) | |
tree | 88449011bc2589de03315283e75278e40f51436f /spec/features/triggers_spec.rb | |
parent | 06d41721e6d229a87459ad9578534b4a5caefeda (diff) | |
download | gitlab-ce-b452b0c7056e481778bfc5838d47dca813bf12ed.tar.gz |
Used send_keys(:return) for elements that are not clickable at one point
Diffstat (limited to 'spec/features/triggers_spec.rb')
-rw-r--r-- | spec/features/triggers_spec.rb | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/spec/features/triggers_spec.rb b/spec/features/triggers_spec.rb index 477798e3826..a6ec1a1a96a 100644 --- a/spec/features/triggers_spec.rb +++ b/spec/features/triggers_spec.rb @@ -45,7 +45,7 @@ feature 'Triggers', js: true do visit project_settings_ci_cd_path(@project) # See if edit page has correct descrption - find('a[title="Edit"]').click + find('a[title="Edit"]').send_keys(:return) expect(page.find('#trigger_description').value).to have_content 'trigger desc' end @@ -54,7 +54,7 @@ feature 'Triggers', js: true do visit project_settings_ci_cd_path(@project) # See if edit page opens, then fill in new description and save - find('a[title="Edit"]').click + find('a[title="Edit"]').send_keys(:return) fill_in 'trigger_description', with: new_trigger_title click_button 'Save trigger' @@ -70,7 +70,7 @@ feature 'Triggers', js: true do visit project_settings_ci_cd_path(@project) # See if the trigger can be edited and description is blank - find('a[title="Edit"]').click + find('a[title="Edit"]').send_keys(:return) expect(page.find('#trigger_description').value).to have_content '' # See if trigger can be updated with description and saved successfully @@ -95,7 +95,7 @@ feature 'Triggers', js: true do scenario 'take trigger ownership' do # See if "Take ownership" on trigger works post trigger creation page.accept_confirm do - find('a.btn-trigger-take-ownership').click + first(:link, "Take ownership").send_keys(:return) end expect(page.find('.flash-notice')).to have_content 'Trigger was re-assigned.' @@ -117,11 +117,12 @@ feature 'Triggers', js: true do scenario 'revoke trigger' do # See if "Revoke" on trigger works post trigger creation - find('a.btn-trigger-revoke').click page.accept_confirm do - expect(page.find('.flash-notice')).to have_content 'Trigger removed' - expect(page).to have_selector('p.settings-message.text-center.append-bottom-default') + 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') end end |