diff options
Diffstat (limited to 'spec/models/bulk_imports')
-rw-r--r-- | spec/models/bulk_imports/entity_spec.rb | 26 | ||||
-rw-r--r-- | spec/models/bulk_imports/export_status_spec.rb | 99 | ||||
-rw-r--r-- | spec/models/bulk_imports/file_transfer/project_config_spec.rb | 2 | ||||
-rw-r--r-- | spec/models/bulk_imports/tracker_spec.rb | 2 |
4 files changed, 86 insertions, 43 deletions
diff --git a/spec/models/bulk_imports/entity_spec.rb b/spec/models/bulk_imports/entity_spec.rb index 6f6a7c9bcd8..874009d552a 100644 --- a/spec/models/bulk_imports/entity_spec.rb +++ b/spec/models/bulk_imports/entity_spec.rb @@ -160,7 +160,7 @@ RSpec.describe BulkImports::Entity, type: :model do it 'returns group pipelines' do entity = build(:bulk_import_entity, :group_entity) - expect(entity.pipelines.flatten).to include(BulkImports::Groups::Pipelines::GroupPipeline) + expect(entity.pipelines.collect { _1[:pipeline] }).to include(BulkImports::Groups::Pipelines::GroupPipeline) end end @@ -168,29 +168,7 @@ RSpec.describe BulkImports::Entity, type: :model do it 'returns project pipelines' do entity = build(:bulk_import_entity, :project_entity) - expect(entity.pipelines.flatten).to include(BulkImports::Projects::Pipelines::ProjectPipeline) - end - end - end - - describe '#create_pipeline_trackers!' do - context 'when entity is group' do - it 'creates trackers for group entity' do - entity = create(:bulk_import_entity, :group_entity) - entity.create_pipeline_trackers! - - expect(entity.trackers.count).to eq(BulkImports::Groups::Stage.new(entity).pipelines.count) - expect(entity.trackers.map(&:pipeline_name)).to include(BulkImports::Groups::Pipelines::GroupPipeline.to_s) - end - end - - context 'when entity is project' do - it 'creates trackers for project entity' do - entity = create(:bulk_import_entity, :project_entity) - entity.create_pipeline_trackers! - - expect(entity.trackers.count).to eq(BulkImports::Projects::Stage.new(entity).pipelines.count) - expect(entity.trackers.map(&:pipeline_name)).to include(BulkImports::Projects::Pipelines::ProjectPipeline.to_s) + expect(entity.pipelines.collect { _1[:pipeline] }).to include(BulkImports::Projects::Pipelines::ProjectPipeline) end end end diff --git a/spec/models/bulk_imports/export_status_spec.rb b/spec/models/bulk_imports/export_status_spec.rb index 79ed6b39358..6ade82409dc 100644 --- a/spec/models/bulk_imports/export_status_spec.rb +++ b/spec/models/bulk_imports/export_status_spec.rb @@ -10,11 +10,9 @@ RSpec.describe BulkImports::ExportStatus do let_it_be(:tracker) { create(:bulk_import_tracker, entity: entity) } let(:response_double) do - double(parsed_response: [{ 'relation' => 'labels', 'status' => status, 'error' => 'error!' }]) - end - - let(:invalid_response_double) do - double(parsed_response: [{ 'relation' => 'not_a_real_relation', 'status' => status, 'error' => 'error!' }]) + instance_double(HTTParty::Response, + parsed_response: [{ 'relation' => 'labels', 'status' => status, 'error' => 'error!' }] + ) end subject { described_class.new(tracker, relation) } @@ -40,22 +38,34 @@ RSpec.describe BulkImports::ExportStatus do it 'returns false' do expect(subject.started?).to eq(false) end + end - context 'when returned relation is invalid' do - before do - allow_next_instance_of(BulkImports::Clients::HTTP) do |client| - allow(client).to receive(:get).and_return(invalid_response_double) - end - end + context 'when export status is not present' do + let(:response_double) do + instance_double(HTTParty::Response, parsed_response: []) + end - it 'returns false' do - expect(subject.started?).to eq(false) + it 'returns false' do + expect(subject.started?).to eq(false) + end + end + + context 'when something goes wrong during export status fetch' do + before do + allow_next_instance_of(BulkImports::Clients::HTTP) do |client| + allow(client).to receive(:get).and_raise( + BulkImports::NetworkError.new("Unsuccessful response", response: nil) + ) end end + + it 'returns false' do + expect(subject.started?).to eq(false) + end end end - describe '#failed' do + describe '#failed?' do context 'when export status is failed' do let(:status) { BulkImports::Export::FAILED } @@ -74,12 +84,67 @@ RSpec.describe BulkImports::ExportStatus do context 'when export status is not present' do let(:response_double) do - double(parsed_response: []) + instance_double(HTTParty::Response, parsed_response: []) + end + + it 'returns false' do + expect(subject.started?).to eq(false) + end + end + + context 'when something goes wrong during export status fetch' do + before do + allow_next_instance_of(BulkImports::Clients::HTTP) do |client| + allow(client).to receive(:get).and_raise( + BulkImports::NetworkError.new("Unsuccessful response", response: nil) + ) + end + end + + it 'returns false' do + expect(subject.started?).to eq(false) + end + end + end + + describe '#empty?' do + context 'when export status is present' do + let(:status) { 'any status' } + + it { expect(subject.empty?).to eq(false) } + end + + context 'when export status is not present' do + let(:response_double) do + instance_double(HTTParty::Response, parsed_response: []) end it 'returns true' do - expect(subject.failed?).to eq(true) - expect(subject.error).to eq('Empty relation export status') + expect(subject.empty?).to eq(true) + end + end + + context 'when export status is empty' do + let(:response_double) do + instance_double(HTTParty::Response, parsed_response: nil) + end + + it 'returns true' do + expect(subject.empty?).to eq(true) + end + end + + context 'when something goes wrong during export status fetch' do + before do + allow_next_instance_of(BulkImports::Clients::HTTP) do |client| + allow(client).to receive(:get).and_raise( + BulkImports::NetworkError.new("Unsuccessful response", response: nil) + ) + end + end + + it 'returns false' do + expect(subject.started?).to eq(false) end end end diff --git a/spec/models/bulk_imports/file_transfer/project_config_spec.rb b/spec/models/bulk_imports/file_transfer/project_config_spec.rb index 61caff647d6..0f02c5c546f 100644 --- a/spec/models/bulk_imports/file_transfer/project_config_spec.rb +++ b/spec/models/bulk_imports/file_transfer/project_config_spec.rb @@ -94,7 +94,7 @@ RSpec.describe BulkImports::FileTransfer::ProjectConfig do describe '#file_relations' do it 'returns project file relations' do - expect(subject.file_relations).to contain_exactly('uploads', 'lfs_objects') + expect(subject.file_relations).to contain_exactly('uploads', 'lfs_objects', 'repository', 'design') end end end diff --git a/spec/models/bulk_imports/tracker_spec.rb b/spec/models/bulk_imports/tracker_spec.rb index 0b6f692a477..1aa76d4dadd 100644 --- a/spec/models/bulk_imports/tracker_spec.rb +++ b/spec/models/bulk_imports/tracker_spec.rb @@ -67,7 +67,7 @@ RSpec.describe BulkImports::Tracker, type: :model do describe '#pipeline_class' do it 'returns the pipeline class' do entity = create(:bulk_import_entity) - pipeline_class = BulkImports::Groups::Stage.new(entity).pipelines.first[1] + pipeline_class = BulkImports::Groups::Stage.new(entity).pipelines.first[:pipeline] tracker = create(:bulk_import_tracker, pipeline_name: pipeline_class) expect(tracker.pipeline_class).to eq(pipeline_class) |