diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-04-28 11:32:31 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-04-28 11:32:31 +0300 |
commit | 0f9e3a905651933ea4ca04712ae77ec47d8562e5 (patch) | |
tree | 46c0f3e5d36131856aaccabbe10ee8cfec8dd4ef /spec | |
parent | 744353cf92f3554af6e71e2da32bec1f2936e47f (diff) |
Add latest changes from gitlab-org/gitlab@15-11-stable-ee
Diffstat (limited to 'spec')
-rw-r--r-- | spec/finders/merge_requests_finder_spec.rb | 36 |
1 files changed, 30 insertions, 6 deletions
diff --git a/spec/finders/merge_requests_finder_spec.rb b/spec/finders/merge_requests_finder_spec.rb index aa167fe7aba..6d576bc8e38 100644 --- a/spec/finders/merge_requests_finder_spec.rb +++ b/spec/finders/merge_requests_finder_spec.rb @@ -498,16 +498,40 @@ RSpec.describe MergeRequestsFinder, feature_category: :code_review_workflow do create(:approval, merge_request: merge_request3, user: user2) end - it 'for approved' do - merge_requests = described_class.new(user, { approved: true }).execute + context 'when flag `mr_approved_filter` is disabled' do + before do + stub_feature_flags(mr_approved_filter: false) + end - expect(merge_requests).to contain_exactly(merge_request3) + it 'for approved' do + merge_requests = described_class.new(user, { approved: true }).execute + + expect(merge_requests).to contain_exactly(merge_request1, merge_request2, merge_request3, merge_request4, merge_request5) + end + + it 'for not approved' do + merge_requests = described_class.new(user, { approved: false }).execute + + expect(merge_requests).to contain_exactly(merge_request1, merge_request2, merge_request3, merge_request4, merge_request5) + end end - it 'for not approved' do - merge_requests = described_class.new(user, { approved: false }).execute + context 'when flag `mr_approved_filter` is enabled' do + before do + stub_feature_flags(mr_approved_filter: true) + end + + it 'for approved' do + merge_requests = described_class.new(user, { approved: true }).execute + + expect(merge_requests).to contain_exactly(merge_request3) + end + + it 'for not approved' do + merge_requests = described_class.new(user, { approved: false }).execute - expect(merge_requests).to contain_exactly(merge_request1, merge_request2, merge_request4, merge_request5) + expect(merge_requests).to contain_exactly(merge_request1, merge_request2, merge_request4, merge_request5) + end end end |