diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2022-06-20 14:10:13 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2022-06-20 14:10:13 +0300 |
commit | 0ea3fcec397b69815975647f5e2aa5fe944a8486 (patch) | |
tree | 7979381b89d26011bcf9bdc989a40fcc2f1ed4ff /spec/controllers/projects/merge_requests/drafts_controller_spec.rb | |
parent | 72123183a20411a36d607d70b12d57c484394c8e (diff) |
Add latest changes from gitlab-org/gitlab@15-1-stable-eev15.1.0-rc42
Diffstat (limited to 'spec/controllers/projects/merge_requests/drafts_controller_spec.rb')
-rw-r--r-- | spec/controllers/projects/merge_requests/drafts_controller_spec.rb | 32 |
1 files changed, 32 insertions, 0 deletions
diff --git a/spec/controllers/projects/merge_requests/drafts_controller_spec.rb b/spec/controllers/projects/merge_requests/drafts_controller_spec.rb index 222bb977beb..b9ede84157d 100644 --- a/spec/controllers/projects/merge_requests/drafts_controller_spec.rb +++ b/spec/controllers/projects/merge_requests/drafts_controller_spec.rb @@ -385,6 +385,38 @@ RSpec.describe Projects::MergeRequests::DraftsController do expect(discussion.resolved?).to eq(false) end end + + context 'publish with note' do + before do + create(:draft_note, merge_request: merge_request, author: user) + end + + context 'when feature flag is disabled' do + before do + stub_feature_flags(mr_review_submit_comment: false) + end + + it 'does not create note' do + post :publish, params: params.merge!(note: 'Hello world') + + expect(merge_request.notes.reload.size).to be(1) + end + end + + context 'when feature flag is enabled' do + it 'creates note' do + post :publish, params: params.merge!(note: 'Hello world') + + expect(merge_request.notes.reload.size).to be(2) + end + + it 'does not create note when note param is empty' do + post :publish, params: params.merge!(note: '') + + expect(merge_request.notes.reload.size).to be(1) + end + end + end end describe 'DELETE #destroy' do |