diff options
Diffstat (limited to 'spec/services/projects/after_import_service_spec.rb')
-rw-r--r-- | spec/services/projects/after_import_service_spec.rb | 131 |
1 files changed, 0 insertions, 131 deletions
diff --git a/spec/services/projects/after_import_service_spec.rb b/spec/services/projects/after_import_service_spec.rb deleted file mode 100644 index a16aec891a9..00000000000 --- a/spec/services/projects/after_import_service_spec.rb +++ /dev/null @@ -1,131 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe Projects::AfterImportService do - include GitHelpers - - subject { described_class.new(project) } - - let(:project) { create(:project, :repository) } - let(:repository) { project.repository } - let(:sha) { project.commit.sha } - let(:housekeeping_service) { double(:housekeeping_service) } - - describe '#execute' do - before do - allow(Repositories::HousekeepingService) - .to receive(:new).with(project).and_return(housekeeping_service) - - allow(housekeeping_service) - .to receive(:execute).and_yield - - allow(housekeeping_service).to receive(:increment!) - end - - it 'performs housekeeping' do - subject.execute - - expect(housekeeping_service).to have_received(:execute) - end - - context 'with some refs in refs/pull/**/*' do - before do - repository.write_ref('refs/pull/1/head', sha) - repository.write_ref('refs/pull/1/merge', sha) - - subject.execute - end - - it 'removes refs/pull/**/*' do - expect(rugged.references.map(&:name)) - .not_to include(%r{\Arefs/pull/}) - end - end - - Repository::RESERVED_REFS_NAMES.each do |name| - context "with a ref in refs/#{name}/tmp" do - before do - repository.write_ref("refs/#{name}/tmp", sha) - - subject.execute - end - - it "does not remove refs/#{name}/tmp" do - expect(rugged.references.map(&:name)) - .to include("refs/#{name}/tmp") - end - end - end - - context 'when after import action throw non-retriable exception' do - let(:exception) { StandardError.new('after import error') } - - before do - allow(repository) - .to receive(:delete_all_refs_except) - .and_raise(exception) - end - - it 'throws after import error' do - expect { subject.execute }.to raise_exception('after import error') - end - end - - context 'when housekeeping service lease is taken' do - let(:exception) { Repositories::HousekeepingService::LeaseTaken.new } - - it 'logs the error message' do - allow_next_instance_of(Repositories::HousekeepingService) do |instance| - expect(instance).to receive(:execute).and_raise(exception) - end - - expect(Gitlab::Import::Logger).to receive(:info).with( - { - message: 'Project housekeeping failed', - project_full_path: project.full_path, - project_id: project.id, - 'error.message' => exception.to_s - }).and_call_original - - subject.execute - end - end - - context 'when after import action throw retriable exception one time' do - let(:exception) { GRPC::DeadlineExceeded.new } - - before do - expect(repository) - .to receive(:delete_all_refs_except) - .and_raise(exception) - expect(repository) - .to receive(:delete_all_refs_except) - .and_call_original - - subject.execute - end - - it 'removes refs/pull/**/*' do - expect(rugged.references.map(&:name)) - .not_to include(%r{\Arefs/pull/}) - end - - it 'records the failures in the database', :aggregate_failures do - import_failure = ImportFailure.last - - expect(import_failure.source).to eq('delete_all_refs') - expect(import_failure.project_id).to eq(project.id) - expect(import_failure.relation_key).to be_nil - expect(import_failure.relation_index).to be_nil - expect(import_failure.exception_class).to eq('GRPC::DeadlineExceeded') - expect(import_failure.exception_message).to be_present - expect(import_failure.correlation_id_value).not_to be_empty - end - end - - def rugged - rugged_repo(repository) - end - end -end |