Commit 6881d091 authored by Kamil Trzcinski's avatar Kamil Trzcinski

Fix rubocop

parent ee8efb3d
...@@ -33,8 +33,8 @@ describe JobArtifactUploader do ...@@ -33,8 +33,8 @@ describe JobArtifactUploader do
context 'file is stored in valid local_path' do context 'file is stored in valid local_path' do
let(:file) do let(:file) do
fixture_file_upload(Rails.root.join( fixture_file_upload(
'spec/fixtures/ci_build_artifacts.zip'), 'application/zip') Rails.root.join('spec/fixtures/ci_build_artifacts.zip'), 'application/zip')
end end
before do before do
...@@ -45,7 +45,7 @@ describe JobArtifactUploader do ...@@ -45,7 +45,7 @@ describe JobArtifactUploader do
it { is_expected.to start_with(local_path) } it { is_expected.to start_with(local_path) }
it { is_expected.to include("/#{job_artifact.created_at.utc.strftime('%Y_%m_%d')}/") } it { is_expected.to include("/#{job_artifact.created_at.utc.strftime('%Y_%m_%d')}/") }
it { is_expected.to include("/#{job_artifact.project_id.to_s}/") } it { is_expected.to include("/#{job_artifact.project_id}/") }
it { is_expected.to end_with("ci_build_artifacts.zip") } it { is_expected.to end_with("ci_build_artifacts.zip") }
end end
end end
...@@ -59,8 +59,8 @@ describe LegacyArtifactUploader do ...@@ -59,8 +59,8 @@ describe LegacyArtifactUploader do
context 'file is stored in valid path' do context 'file is stored in valid path' do
let(:file) do let(:file) do
fixture_file_upload(Rails.root.join( fixture_file_upload(
'spec/fixtures/ci_build_artifacts.zip'), 'application/zip') Rails.root.join('spec/fixtures/ci_build_artifacts.zip'), 'application/zip')
end end
before do before do
...@@ -71,7 +71,7 @@ describe LegacyArtifactUploader do ...@@ -71,7 +71,7 @@ describe LegacyArtifactUploader do
it { is_expected.to start_with(local_path) } it { is_expected.to start_with(local_path) }
it { is_expected.to include("/#{job.created_at.utc.strftime('%Y_%m')}/") } it { is_expected.to include("/#{job.created_at.utc.strftime('%Y_%m')}/") }
it { is_expected.to include("/#{job.project_id.to_s}/") } it { is_expected.to include("/#{job.project_id}/") }
it { is_expected.to end_with("ci_build_artifacts.zip") } it { is_expected.to end_with("ci_build_artifacts.zip") }
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