Skip to content
Snippets Groups Projects
Commit f5d3b921 authored by Zeger-Jan van de Weg's avatar Zeger-Jan van de Weg Committed by Kamil Trzcinski
Browse files

Remove Ci::Build#artifacts_file?

parent bf6126b1
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -6,10 +6,6 @@ module ArtifactMigratable
job_archive&.file || legacy_artifacts_file
end
 
def artifacts_file?
job_archive&.file? || legacy_artifacts_file?
end
def artifacts_metadata
job_metadata&.file || legacy_artifacts_metadata
end
Loading
Loading
Loading
Loading
@@ -58,7 +58,7 @@ module Gitlab
end
 
def artifact_upload_ok
{ TempPath: LegacyArtifactUploader.artifacts_upload_path }
{ TempPath: JobArtifactUploader.artifacts_upload_path }
end
 
def send_git_blob(repository, blob)
Loading
Loading
Loading
Loading
@@ -39,7 +39,7 @@ describe Projects::UpdatePagesService do
it "doesn't delete artifacts" do
expect(execute).to eq(:success)
 
expect(build.reload.artifacts_file?).to eq(true)
expect(build.reload.artifacts?).to eq(true)
end
end
 
Loading
Loading
@@ -47,7 +47,7 @@ describe Projects::UpdatePagesService do
it "does delete artifacts" do
expect(execute).to eq(:success)
 
expect(build.reload.artifacts_file?).to eq(false)
expect(build.reload.artifacts?).to eq(false)
end
end
end
Loading
Loading
@@ -110,7 +110,7 @@ describe Projects::UpdatePagesService do
it "doesn't delete artifacts" do
expect(execute).to eq(:success)
 
expect(build.artifacts_file?).to eq(true)
expect(build.artifacts?).to eq(true)
end
end
 
Loading
Loading
@@ -118,7 +118,7 @@ describe Projects::UpdatePagesService do
it "does delete artifacts" do
expect(execute).to eq(:success)
 
expect(build.reload.artifacts_file?).to eq(false)
expect(build.reload.artifacts?).to eq(false)
end
end
end
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment