2017-09-21 08:34:12 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe Ci::JobArtifact do
|
2018-03-20 23:03:50 +00:00
|
|
|
let(:artifact) { create(:ci_job_artifact, :archive) }
|
2017-09-21 08:34:12 +00:00
|
|
|
|
|
|
|
describe "Associations" do
|
|
|
|
it { is_expected.to belong_to(:project) }
|
|
|
|
it { is_expected.to belong_to(:job) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to respond_to(:file) }
|
|
|
|
it { is_expected.to respond_to(:created_at) }
|
|
|
|
it { is_expected.to respond_to(:updated_at) }
|
|
|
|
|
2018-02-06 14:18:32 +00:00
|
|
|
it { is_expected.to delegate_method(:open).to(:file) }
|
|
|
|
it { is_expected.to delegate_method(:exists?).to(:file) }
|
|
|
|
|
2017-12-08 09:09:06 +00:00
|
|
|
describe 'callbacks' do
|
|
|
|
subject { create(:ci_job_artifact, :archive) }
|
|
|
|
|
2018-03-01 21:30:31 +00:00
|
|
|
describe '#schedule_background_upload' do
|
2017-12-08 09:09:06 +00:00
|
|
|
context 'when object storage is disabled' do
|
|
|
|
before do
|
|
|
|
stub_artifacts_object_storage(enabled: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not schedule the migration' do
|
2018-07-06 15:08:27 +00:00
|
|
|
expect(ObjectStorage::BackgroundMoveWorker).not_to receive(:perform_async)
|
2017-12-08 09:09:06 +00:00
|
|
|
|
|
|
|
subject
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when object storage is enabled' do
|
|
|
|
context 'when background upload is enabled' do
|
2018-03-06 21:19:24 +00:00
|
|
|
before do
|
|
|
|
stub_artifacts_object_storage(background_upload: true)
|
2017-12-08 09:09:06 +00:00
|
|
|
end
|
|
|
|
|
2018-03-06 21:19:24 +00:00
|
|
|
it 'schedules the model for migration' do
|
2018-03-07 18:27:49 +00:00
|
|
|
expect(ObjectStorage::BackgroundMoveWorker).to receive(:perform_async).with('JobArtifactUploader', described_class.name, :file, kind_of(Numeric))
|
2017-12-08 09:09:06 +00:00
|
|
|
|
2018-03-06 21:19:24 +00:00
|
|
|
subject
|
2017-12-08 09:09:06 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when background upload is disabled' do
|
|
|
|
before do
|
|
|
|
stub_artifacts_object_storage(background_upload: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'schedules the model for migration' do
|
2018-03-07 18:27:49 +00:00
|
|
|
expect(ObjectStorage::BackgroundMoveWorker).not_to receive(:perform_async)
|
2017-12-08 09:09:06 +00:00
|
|
|
|
|
|
|
subject
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-03-20 23:03:50 +00:00
|
|
|
context 'creating the artifact' do
|
|
|
|
let(:project) { create(:project) }
|
|
|
|
let(:artifact) { create(:ci_job_artifact, :archive, project: project) }
|
|
|
|
|
|
|
|
it 'sets the size from the file size' do
|
2017-09-21 08:34:12 +00:00
|
|
|
expect(artifact.size).to eq(106365)
|
|
|
|
end
|
2018-03-20 23:03:50 +00:00
|
|
|
|
|
|
|
it 'updates the project statistics' do
|
|
|
|
expect { artifact }
|
|
|
|
.to change { project.statistics.reload.build_artifacts_size }
|
|
|
|
.by(106365)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'updating the artifact file' do
|
|
|
|
it 'updates the artifact size' do
|
2018-06-05 21:18:06 +00:00
|
|
|
artifact.update!(file: fixture_file_upload('spec/fixtures/dk.png'))
|
2018-03-20 23:03:50 +00:00
|
|
|
expect(artifact.size).to eq(1062)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'updates the project statistics' do
|
2018-06-05 21:18:06 +00:00
|
|
|
expect { artifact.update!(file: fixture_file_upload('spec/fixtures/dk.png')) }
|
2018-03-20 23:03:50 +00:00
|
|
|
.to change { artifact.project.statistics.reload.build_artifacts_size }
|
|
|
|
.by(1062 - 106365)
|
|
|
|
end
|
2017-09-21 08:34:12 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
describe '#file' do
|
|
|
|
subject { artifact.file }
|
|
|
|
|
|
|
|
context 'the uploader api' do
|
|
|
|
it { is_expected.to respond_to(:store_dir) }
|
|
|
|
it { is_expected.to respond_to(:cache_dir) }
|
|
|
|
it { is_expected.to respond_to(:work_dir) }
|
|
|
|
end
|
|
|
|
end
|
2017-12-03 11:02:11 +00:00
|
|
|
|
|
|
|
describe '#expire_in' do
|
|
|
|
subject { artifact.expire_in }
|
|
|
|
|
|
|
|
it { is_expected.to be_nil }
|
|
|
|
|
|
|
|
context 'when expire_at is specified' do
|
|
|
|
let(:expire_at) { Time.now + 7.days }
|
|
|
|
|
|
|
|
before do
|
|
|
|
artifact.expire_at = expire_at
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to be_within(5).of(expire_at - Time.now) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#expire_in=' do
|
|
|
|
subject { artifact.expire_in }
|
|
|
|
|
|
|
|
it 'when assigning valid duration' do
|
|
|
|
artifact.expire_in = '7 days'
|
|
|
|
|
|
|
|
is_expected.to be_within(10).of(7.days.to_i)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'when assigning invalid duration' do
|
|
|
|
expect { artifact.expire_in = '7 elephants' }.to raise_error(ChronicDuration::DurationParseError)
|
2017-12-03 15:21:59 +00:00
|
|
|
|
2017-12-03 11:02:11 +00:00
|
|
|
is_expected.to be_nil
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'when resetting value' do
|
|
|
|
artifact.expire_in = nil
|
|
|
|
|
|
|
|
is_expected.to be_nil
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'when setting to 0' do
|
|
|
|
artifact.expire_in = '0'
|
|
|
|
|
|
|
|
is_expected.to be_nil
|
|
|
|
end
|
|
|
|
end
|
2018-03-20 23:03:50 +00:00
|
|
|
|
|
|
|
context 'when destroying the artifact' do
|
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:pipeline) { create(:ci_pipeline, project: project) }
|
|
|
|
let!(:build) { create(:ci_build, :artifacts, pipeline: pipeline) }
|
|
|
|
|
|
|
|
it 'updates the project statistics' do
|
|
|
|
artifact = build.job_artifacts.first
|
|
|
|
|
|
|
|
expect(ProjectStatistics)
|
|
|
|
.to receive(:increment_statistic)
|
|
|
|
.and_call_original
|
|
|
|
|
|
|
|
expect { artifact.destroy }
|
|
|
|
.to change { project.statistics.reload.build_artifacts_size }
|
|
|
|
.by(-106365)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when it is destroyed from the project level' do
|
|
|
|
it 'does not update the project statistics' do
|
|
|
|
expect(ProjectStatistics)
|
|
|
|
.not_to receive(:increment_statistic)
|
|
|
|
|
|
|
|
project.update_attributes(pending_delete: true)
|
|
|
|
project.destroy!
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2018-04-13 08:20:07 +00:00
|
|
|
|
|
|
|
describe 'file is being stored' do
|
|
|
|
subject { create(:ci_job_artifact, :archive) }
|
|
|
|
|
|
|
|
context 'when object has nil store' do
|
|
|
|
before do
|
|
|
|
subject.update_column(:file_store, nil)
|
|
|
|
subject.reload
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'is stored locally' do
|
|
|
|
expect(subject.file_store).to be(nil)
|
|
|
|
expect(subject.file).to be_file_storage
|
|
|
|
expect(subject.file.object_store).to eq(ObjectStorage::Store::LOCAL)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when existing object has local store' do
|
|
|
|
it 'is stored locally' do
|
|
|
|
expect(subject.file_store).to be(ObjectStorage::Store::LOCAL)
|
|
|
|
expect(subject.file).to be_file_storage
|
|
|
|
expect(subject.file.object_store).to eq(ObjectStorage::Store::LOCAL)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when direct upload is enabled' do
|
|
|
|
before do
|
|
|
|
stub_artifacts_object_storage(direct_upload: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when file is stored' do
|
|
|
|
it 'is stored remotely' do
|
|
|
|
expect(subject.file_store).to eq(ObjectStorage::Store::REMOTE)
|
|
|
|
expect(subject.file).not_to be_file_storage
|
|
|
|
expect(subject.file.object_store).to eq(ObjectStorage::Store::REMOTE)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-09-21 08:34:12 +00:00
|
|
|
end
|