Commit 472cb9ee authored by Kamil Trzcinski's avatar Kamil Trzcinski

Add missing :archive to :ci_job_artifact

parent e21c0dee
...@@ -324,7 +324,7 @@ describe API::Jobs do ...@@ -324,7 +324,7 @@ describe API::Jobs do
context 'when artifacts are stored remotely' do context 'when artifacts are stored remotely' do
let(:job) { create(:ci_build, pipeline: pipeline) } let(:job) { create(:ci_build, pipeline: pipeline) }
let!(:artifact) { create(:ci_job_artifact, :remote_store, job: job) } let!(:artifact) { create(:ci_job_artifact, :archive, :remote_store, job: job) }
before do before do
job.reload job.reload
...@@ -453,7 +453,7 @@ describe API::Jobs do ...@@ -453,7 +453,7 @@ describe API::Jobs do
context 'when artifacts are stored remotely' do context 'when artifacts are stored remotely' do
let(:job) { create(:ci_build, pipeline: pipeline, user: api_user) } let(:job) { create(:ci_build, pipeline: pipeline, user: api_user) }
let!(:artifact) { create(:ci_job_artifact, :remote_store, job: job) } let!(:artifact) { create(:ci_job_artifact, :archive, :remote_store, job: job) }
before do before do
job.reload job.reload
......
...@@ -1166,7 +1166,7 @@ describe API::Runner do ...@@ -1166,7 +1166,7 @@ describe API::Runner do
let(:store) { JobArtifactUploader::LOCAL_STORE } let(:store) { JobArtifactUploader::LOCAL_STORE }
before do before do
create(:ci_job_artifact, file_store: store, job: job) create(:ci_job_artifact, :archive, file_store: store, job: job)
download_artifact download_artifact
end end
......
...@@ -216,7 +216,7 @@ describe API::V3::Builds do ...@@ -216,7 +216,7 @@ describe API::V3::Builds do
context 'when artifacts are stored remotely' do context 'when artifacts are stored remotely' do
let(:build) { create(:ci_build, pipeline: pipeline) } let(:build) { create(:ci_build, pipeline: pipeline) }
let!(:artifact) { create(:ci_job_artifact, :remote_store, job: build) } let!(:artifact) { create(:ci_job_artifact, :archive, :remote_store, job: build) }
it 'returns location redirect' do it 'returns location redirect' do
get v3_api("/projects/#{project.id}/builds/#{build.id}/artifacts", api_user) get v3_api("/projects/#{project.id}/builds/#{build.id}/artifacts", api_user)
...@@ -313,7 +313,7 @@ describe API::V3::Builds do ...@@ -313,7 +313,7 @@ describe API::V3::Builds do
context 'when artifacts are stored remotely' do context 'when artifacts are stored remotely' do
let(:build) { create(:ci_build, pipeline: pipeline) } let(:build) { create(:ci_build, pipeline: pipeline) }
let!(:artifact) { create(:ci_job_artifact, :remote_store, job: build) } let!(:artifact) { create(:ci_job_artifact, :archive, :remote_store, job: build) }
before do before do
build.reload build.reload
......
...@@ -69,7 +69,7 @@ describe 'gitlab:artifacts namespace rake task' do ...@@ -69,7 +69,7 @@ describe 'gitlab:artifacts namespace rake task' do
end end
context 'job artifacts' do context 'job artifacts' do
let!(:artifact) { create(:ci_job_artifact, file_store: store) } let!(:artifact) { create(:ci_job_artifact, :archive, file_store: store) }
context 'when local storage is used' do context 'when local storage is used' do
let(:store) { ObjectStoreUploader::LOCAL_STORE } let(:store) { ObjectStoreUploader::LOCAL_STORE }
......
...@@ -83,7 +83,7 @@ describe ObjectStorageUploadWorker do ...@@ -83,7 +83,7 @@ describe ObjectStorageUploadWorker do
end end
context 'for job artifacts' do context 'for job artifacts' do
let(:artifact) { create(:ci_job_artifact) } let(:artifact) { create(:ci_job_artifact, :archive) }
let(:uploader_class) { JobArtifactUploader } let(:uploader_class) { JobArtifactUploader }
let(:subject_class) { Ci::JobArtifact } let(:subject_class) { Ci::JobArtifact }
let(:file_field) { :file } let(:file_field) { :file }
......
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