diff options
author | Phil Hughes <me@iamphill.com> | 2016-07-18 15:13:56 +0100 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2016-07-20 10:48:02 +0100 |
commit | 357ad44a744db07b6c849bf262f00100b448382b (patch) | |
tree | e225d18f43a85e9e9413861743956d8dfe19a1e7 /spec/features/merge_requests/form_spec.rb | |
parent | feeb489302dcf82e0e1f7e014c34824d52f8a882 (diff) | |
download | gitlab-ce-357ad44a744db07b6c849bf262f00100b448382b.tar.gz |
Fixed Rubocop errors
Diffstat (limited to 'spec/features/merge_requests/form_spec.rb')
-rw-r--r-- | spec/features/merge_requests/form_spec.rb | 85 |
1 files changed, 55 insertions, 30 deletions
diff --git a/spec/features/merge_requests/form_spec.rb b/spec/features/merge_requests/form_spec.rb index 8da33b0f3c3..9d6706ee9b3 100644 --- a/spec/features/merge_requests/form_spec.rb +++ b/spec/features/merge_requests/form_spec.rb @@ -19,7 +19,7 @@ describe 'New/edit merge request', feature: true, js: true do target_project: project, source_branch: 'fix', target_branch: 'master' - ) + ) login_as(user) @@ -27,7 +27,36 @@ describe 'New/edit merge request', feature: true, js: true do end it 'should update merge request' do - update_merge_request + click_button 'Assignee' + click_link user.name + + page.find '.js-assignee-search' do + expect(page).to have_content user.name + end + + click_button 'Milestone' + click_link milestone.title + + page.find '.js-milestone-select' do + expect(page).to have_content milestone.title + end + + click_button 'Labels' + click_link label.title + click_link label2.title + + page.find '.js-label-select' do + expect(page).to have_content label2.title + end + + click_button 'Save changes' + + page.find '.issuable-sidebar' do + expect(page).to have_content user.name + expect(page).to have_content milestone.title + expect(page).to have_content label.title + expect(page).to have_content label2.title + end end end @@ -40,7 +69,7 @@ describe 'New/edit merge request', feature: true, js: true do target_project: project, source_branch: 'fix', target_branch: 'master' - ) + ) login_as(user) @@ -48,40 +77,36 @@ describe 'New/edit merge request', feature: true, js: true do end it 'should update merge request' do - update_merge_request - end - end - - def update_merge_request - click_button 'Assignee' - click_link user.name + click_button 'Assignee' + click_link user.name - page.find '.js-assignee-search' do - expect(page).to have_content user.name - end + page.find '.js-assignee-search' do + expect(page).to have_content user.name + end - click_button 'Milestone' - click_link milestone.title + click_button 'Milestone' + click_link milestone.title - page.find '.js-milestone-select' do - expect(page).to have_content milestone.title - end + page.find '.js-milestone-select' do + expect(page).to have_content milestone.title + end - click_button 'Labels' - click_link label.title - click_link label2.title + click_button 'Labels' + click_link label.title + click_link label2.title - page.find '.js-label-select' do - expect(page).to have_content label2.title - end + page.find '.js-label-select' do + expect(page).to have_content label2.title + end - click_button 'Save changes' + click_button 'Save changes' - page.find '.issuable-sidebar' do - expect(page).to have_content user.name - expect(page).to have_content milestone.title - expect(page).to have_content label.title - expect(page).to have_content label2.title + page.find '.issuable-sidebar' do + expect(page).to have_content user.name + expect(page).to have_content milestone.title + expect(page).to have_content label.title + expect(page).to have_content label2.title + end end end end |