Commit 0963498d authored by Erick Bajao's avatar Erick Bajao

Pass file type to report download endpoint

parent 2f1f43ae
......@@ -8,6 +8,6 @@ class JobArtifactReportEntity < Grape::Entity
expose :size
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
......@@ -22,7 +22,7 @@ describe JobArtifactReportEntity do
end
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
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