diff options
Diffstat (limited to 'spec/serializers/merge_request_poll_cached_widget_entity_spec.rb')
-rw-r--r-- | spec/serializers/merge_request_poll_cached_widget_entity_spec.rb | 88 |
1 files changed, 80 insertions, 8 deletions
diff --git a/spec/serializers/merge_request_poll_cached_widget_entity_spec.rb b/spec/serializers/merge_request_poll_cached_widget_entity_spec.rb index 8c72430ff5c..5845a868e53 100644 --- a/spec/serializers/merge_request_poll_cached_widget_entity_spec.rb +++ b/spec/serializers/merge_request_poll_cached_widget_entity_spec.rb @@ -8,6 +8,7 @@ RSpec.describe MergeRequestPollCachedWidgetEntity do let_it_be(:project, refind: true) { create :project, :repository } let_it_be(:resource, refind: true) { create(:merge_request, source_project: project, target_project: project) } let_it_be(:user) { create(:user) } + let(:pipeline) { create(:ci_empty_pipeline, project: project) } let(:request) { double('request', current_user: user, project: project) } @@ -248,14 +249,6 @@ RSpec.describe MergeRequestPollCachedWidgetEntity do expect(subject[:pipeline]).to eq(pipeline_payload) end - - context 'when merge_request_cached_pipeline_serializer is disabled' do - it 'does not return pipeline' do - stub_feature_flags(merge_request_cached_pipeline_serializer: false) - - expect(subject[:pipeline]).to be_nil - end - end end context 'when user does not have access to pipelines' do @@ -276,4 +269,83 @@ RSpec.describe MergeRequestPollCachedWidgetEntity do end end end + + describe 'merge_pipeline' do + it 'returns nil' do + expect(subject[:merge_pipeline]).to be_nil + end + + context 'when is merged' do + let(:resource) { create(:merged_merge_request, source_project: project, merge_commit_sha: project.commit.id) } + let(:pipeline) { create(:ci_empty_pipeline, project: project, ref: resource.target_branch, sha: resource.merge_commit_sha) } + + before do + project.add_maintainer(user) + end + + it 'returns merge_pipeline' do + pipeline.reload + pipeline_payload = + MergeRequests::PipelineEntity + .represent(pipeline, request: request) + .as_json + + expect(subject[:merge_pipeline]).to eq(pipeline_payload) + end + + context 'when user cannot read pipelines on target project' do + before do + project.add_guest(user) + end + + it 'returns nil' do + expect(subject[:merge_pipeline]).to be_nil + end + end + + context 'when merge_request_cached_merge_pipeline_serializer is disabled' do + before do + stub_feature_flags(merge_request_cached_merge_pipeline_serializer: false) + end + + it 'returns nil' do + expect(subject[:merge_pipeline]).to be_nil + end + end + end + end + + describe 'ci related paths' do + using RSpec::Parameterized::TableSyntax + + where(:path_field, :method_for_existence_check) do + :terraform_reports_path | :has_terraform_reports? + :accessibility_report_path | :has_accessibility_reports? + :exposed_artifacts_path | :has_exposed_artifacts? + :test_reports_path | :has_test_reports? + :codequality_reports_path | :has_codequality_reports? + end + + with_them do + context 'when merge request has reports' do + before do + allow(resource).to receive(method_for_existence_check).and_return(true) + end + + it 'set the path to poll data' do + expect(subject[path_field]).to be_present + end + end + + context 'when merge request has no reports' do + before do + allow(resource).to receive(method_for_existence_check).and_return(false) + end + + it 'does not set reports path' do + expect(subject[path_field]).to be_nil + end + end + end + end end |