Commit 639d0fae authored by Bob Van Landuyt's avatar Bob Van Landuyt

Merge branch 'eb-fix-internal-report-artifact-download' into 'master'

Pass file type to report download endpoint

See merge request gitlab-org/gitlab!19247
parents 99069a4c 0963498d
...@@ -8,6 +8,6 @@ class JobArtifactReportEntity < Grape::Entity ...@@ -8,6 +8,6 @@ class JobArtifactReportEntity < Grape::Entity
expose :size expose :size
expose :download_path do |artifact| expose :download_path do |artifact|
download_project_job_artifacts_path(artifact.job.project, artifact.job, file_type: artifact.file_format) download_project_job_artifacts_path(artifact.job.project, artifact.job, file_type: artifact.file_type)
end end
end end
...@@ -22,7 +22,7 @@ describe JobArtifactReportEntity do ...@@ -22,7 +22,7 @@ describe JobArtifactReportEntity do
end end
it 'exposes download path' do it 'exposes download path' do
expect(subject[:download_path]).to include("jobs/#{report.job.id}/artifacts/download") expect(subject[:download_path]).to include("jobs/#{report.job.id}/artifacts/download?file_type=#{report.file_type}")
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment