diff options
author | Natalia Tepluhina <ntepluhina@gitlab.com> | 2019-02-07 15:14:56 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2019-02-07 15:14:56 +0000 |
commit | 407fd2edc791c5d159eedad3dfe5521543b50d91 (patch) | |
tree | 75c526d7e55051720507d955a09a6f1d81a0ccbc /spec/features | |
parent | 232ae06a829ba1a7e795564f711e730bd839be41 (diff) | |
download | gitlab-ce-407fd2edc791c5d159eedad3dfe5521543b50d91.tar.gz |
Resolve "Override squash commit message"
Diffstat (limited to 'spec/features')
-rw-r--r-- | spec/features/merge_request/user_accepts_merge_request_spec.rb | 4 | ||||
-rw-r--r-- | spec/features/merge_request/user_customizes_merge_commit_message_spec.rb | 10 |
2 files changed, 7 insertions, 7 deletions
diff --git a/spec/features/merge_request/user_accepts_merge_request_spec.rb b/spec/features/merge_request/user_accepts_merge_request_spec.rb index 00ac7c72a11..5fa23dbb998 100644 --- a/spec/features/merge_request/user_accepts_merge_request_spec.rb +++ b/spec/features/merge_request/user_accepts_merge_request_spec.rb @@ -80,8 +80,8 @@ describe 'User accepts a merge request', :js do end it 'accepts a merge request' do - click_button('Modify commit message') - fill_in('Commit message', with: 'wow such merge') + find('.js-mr-widget-commits-count').click + fill_in('merge-message-edit', with: 'wow such merge') click_button('Merge') diff --git a/spec/features/merge_request/user_customizes_merge_commit_message_spec.rb b/spec/features/merge_request/user_customizes_merge_commit_message_spec.rb index 8d2d4279d3c..c6b11fce388 100644 --- a/spec/features/merge_request/user_customizes_merge_commit_message_spec.rb +++ b/spec/features/merge_request/user_customizes_merge_commit_message_spec.rb @@ -13,7 +13,7 @@ describe 'Merge request < User customizes merge commit message', :js do description: "Description\n\nclosing #{issue_1.to_reference}, #{issue_2.to_reference}" ) end - let(:textbox) { page.find(:css, '.js-commit-message', visible: false) } + let(:textbox) { page.find(:css, '#merge-message-edit', visible: false) } let(:default_message) do [ "Merge branch 'feature' into 'master'", @@ -38,16 +38,16 @@ describe 'Merge request < User customizes merge commit message', :js do end it 'toggles commit message between message with description and without description' do - expect(page).not_to have_selector('.js-commit-message') - click_button "Modify commit message" + expect(page).not_to have_selector('#merge-message-edit') + first('.js-mr-widget-commits-count').click expect(textbox).to be_visible expect(textbox.value).to eq(default_message) - click_link "Include description in commit message" + check('Include merge request description') expect(textbox.value).to eq(message_with_description) - click_link "Don't include description in commit message" + uncheck('Include merge request description') expect(textbox.value).to eq(default_message) end |