summaryrefslogtreecommitdiff
path: root/spec/serializers/merge_request_poll_cached_widget_entity_spec.rb
diff options
context:
space:
mode:
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.rb88
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