diff options
author | Stan Hu <stanhu@gmail.com> | 2017-08-12 22:48:02 -0700 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2017-08-12 22:48:02 -0700 |
commit | 1eade5854a5cbf5fc9e8a326e976c55006e100de (patch) | |
tree | 3cfa12f5f0905f01d8c84c598a084e59aaa0285d /spec/features/triggers_spec.rb | |
parent | ad73d24235ef22f9012b30be9d9492b11b46ca58 (diff) | |
download | gitlab-ce-1eade5854a5cbf5fc9e8a326e976c55006e100de.tar.gz |
Fix trigger spec by surrounding the accept_confirm in the right place
Diffstat (limited to 'spec/features/triggers_spec.rb')
-rw-r--r-- | spec/features/triggers_spec.rb | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/spec/features/triggers_spec.rb b/spec/features/triggers_spec.rb index 47664de469a..477798e3826 100644 --- a/spec/features/triggers_spec.rb +++ b/spec/features/triggers_spec.rb @@ -94,12 +94,13 @@ feature 'Triggers', js: true do scenario 'take trigger ownership' do # See if "Take ownership" on trigger works post trigger creation - find('a.btn-trigger-take-ownership').click page.accept_confirm do - 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 + find('a.btn-trigger-take-ownership').click 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 end end |