2020-10-23 11:08:42 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
RSpec.describe BulkImports::Pipeline::Runner do
|
2020-11-30 10:09:21 -05:00
|
|
|
let(:extractor) do
|
|
|
|
Class.new do
|
|
|
|
def initialize(options = {}); end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
def extract(context); end
|
|
|
|
end
|
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
let(:transformer) do
|
|
|
|
Class.new do
|
|
|
|
def initialize(options = {}); end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
def transform(context); end
|
|
|
|
end
|
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
let(:loader) do
|
|
|
|
Class.new do
|
|
|
|
def initialize(options = {}); end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
def load(context); end
|
|
|
|
end
|
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
before do
|
|
|
|
stub_const('BulkImports::Extractor', extractor)
|
|
|
|
stub_const('BulkImports::Transformer', transformer)
|
|
|
|
stub_const('BulkImports::Loader', loader)
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
pipeline = Class.new do
|
|
|
|
include BulkImports::Pipeline
|
2021-02-16 07:09:03 -05:00
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
extractor BulkImports::Extractor
|
|
|
|
transformer BulkImports::Transformer
|
|
|
|
loader BulkImports::Loader
|
2020-10-23 11:08:42 -04:00
|
|
|
end
|
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
stub_const('BulkImports::MyPipeline', pipeline)
|
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2021-03-18 11:09:04 -04:00
|
|
|
let_it_be_with_reload(:entity) { create(:bulk_import_entity) }
|
|
|
|
let_it_be(:tracker) { create(:bulk_import_tracker, entity: entity) }
|
|
|
|
let_it_be(:context) { BulkImports::Pipeline::Context.new(tracker, extra: :data) }
|
2021-03-10 19:09:38 -05:00
|
|
|
|
|
|
|
subject { BulkImports::MyPipeline.new(context) }
|
|
|
|
|
|
|
|
describe 'pipeline runner' do
|
|
|
|
context 'when entity is not marked as failed' do
|
2020-11-30 10:09:21 -05:00
|
|
|
it 'runs pipeline extractor, transformer, loader' do
|
|
|
|
expect_next_instance_of(BulkImports::Extractor) do |extractor|
|
2021-02-01 07:09:03 -05:00
|
|
|
expect(extractor)
|
|
|
|
.to receive(:extract)
|
|
|
|
.with(context)
|
|
|
|
.and_return(extracted_data)
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
expect_next_instance_of(BulkImports::Transformer) do |transformer|
|
2021-02-01 07:09:03 -05:00
|
|
|
expect(transformer)
|
|
|
|
.to receive(:transform)
|
|
|
|
.with(context, extracted_data.data.first)
|
|
|
|
.and_return(extracted_data.data.first)
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
expect_next_instance_of(BulkImports::Loader) do |loader|
|
2021-02-01 07:09:03 -05:00
|
|
|
expect(loader)
|
|
|
|
.to receive(:load)
|
|
|
|
.with(context, extracted_data.data.first)
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
|
|
|
|
expect(logger).to receive(:info)
|
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
message: 'Pipeline started',
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline'
|
|
|
|
)
|
2021-02-04 07:09:25 -05:00
|
|
|
)
|
|
|
|
expect(logger).to receive(:info)
|
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline',
|
|
|
|
pipeline_step: :extractor,
|
|
|
|
step_class: 'BulkImports::Extractor'
|
|
|
|
)
|
2020-11-30 10:09:21 -05:00
|
|
|
)
|
|
|
|
expect(logger).to receive(:info)
|
2021-02-02 13:09:42 -05:00
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline',
|
|
|
|
pipeline_step: :transformer,
|
|
|
|
step_class: 'BulkImports::Transformer'
|
|
|
|
)
|
2021-02-02 13:09:42 -05:00
|
|
|
)
|
2020-11-30 10:09:21 -05:00
|
|
|
expect(logger).to receive(:info)
|
2021-02-02 13:09:42 -05:00
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline',
|
|
|
|
pipeline_step: :loader,
|
|
|
|
step_class: 'BulkImports::Loader'
|
|
|
|
)
|
2021-02-02 13:09:42 -05:00
|
|
|
)
|
2021-02-16 07:09:03 -05:00
|
|
|
expect(logger).to receive(:info)
|
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline',
|
|
|
|
pipeline_step: :after_run
|
|
|
|
)
|
2021-02-16 07:09:03 -05:00
|
|
|
)
|
2020-11-30 10:09:21 -05:00
|
|
|
expect(logger).to receive(:info)
|
2021-02-02 13:09:42 -05:00
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
message: 'Pipeline finished',
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline'
|
|
|
|
)
|
2021-02-02 13:09:42 -05:00
|
|
|
)
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
subject.run
|
2020-10-23 11:08:42 -04:00
|
|
|
end
|
|
|
|
|
2021-03-23 14:09:05 -04:00
|
|
|
context 'when extracted data has multiple pages' do
|
|
|
|
it 'updates tracker information and runs pipeline again' do
|
|
|
|
first_page = extracted_data(has_next_page: true)
|
|
|
|
last_page = extracted_data
|
|
|
|
|
|
|
|
expect_next_instance_of(BulkImports::Extractor) do |extractor|
|
|
|
|
expect(extractor)
|
|
|
|
.to receive(:extract)
|
|
|
|
.with(context)
|
|
|
|
.and_return(first_page, last_page)
|
|
|
|
end
|
|
|
|
|
|
|
|
subject.run
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
context 'when exception is raised' do
|
|
|
|
before do
|
|
|
|
allow_next_instance_of(BulkImports::Extractor) do |extractor|
|
|
|
|
allow(extractor).to receive(:extract).with(context).and_raise(StandardError, 'Error!')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'logs import failure' do
|
2021-03-10 19:09:38 -05:00
|
|
|
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
|
|
|
|
expect(logger).to receive(:error)
|
|
|
|
.with(
|
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
pipeline_step: :extractor,
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline',
|
|
|
|
exception_class: 'StandardError',
|
|
|
|
exception_message: 'Error!'
|
|
|
|
)
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
expect { subject.run }
|
|
|
|
.to change(entity.failures, :count).by(1)
|
2020-11-30 10:09:21 -05:00
|
|
|
|
|
|
|
failure = entity.failures.first
|
|
|
|
|
|
|
|
expect(failure).to be_present
|
|
|
|
expect(failure.pipeline_class).to eq('BulkImports::MyPipeline')
|
2021-01-27 19:09:33 -05:00
|
|
|
expect(failure.pipeline_step).to eq('extractor')
|
2020-11-30 10:09:21 -05:00
|
|
|
expect(failure.exception_class).to eq('StandardError')
|
|
|
|
expect(failure.exception_message).to eq('Error!')
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pipeline is marked to abort on failure' do
|
|
|
|
before do
|
|
|
|
BulkImports::MyPipeline.abort_on_failure!
|
|
|
|
end
|
|
|
|
|
2021-03-18 11:09:04 -04:00
|
|
|
it 'logs a warn message and marks entity as failed' do
|
2020-11-30 10:09:21 -05:00
|
|
|
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
|
|
|
|
expect(logger).to receive(:warn)
|
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
|
|
|
message: 'Pipeline failed',
|
|
|
|
pipeline_class: 'BulkImports::MyPipeline'
|
|
|
|
)
|
2020-11-30 10:09:21 -05:00
|
|
|
)
|
|
|
|
end
|
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
subject.run
|
2021-03-18 11:09:04 -04:00
|
|
|
|
|
|
|
expect(entity.status_name).to eq(:failed)
|
|
|
|
expect(tracker.status_name).to eq(:failed)
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pipeline is not marked to abort on failure' do
|
2021-03-10 19:09:38 -05:00
|
|
|
it 'does not mark entity as failed' do
|
|
|
|
subject.run
|
2020-11-30 10:09:21 -05:00
|
|
|
|
|
|
|
expect(entity.failed?).to eq(false)
|
|
|
|
end
|
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
end
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
context 'when entity is marked as failed' do
|
|
|
|
it 'logs and returns without execution' do
|
2021-03-10 19:09:38 -05:00
|
|
|
entity.fail_op!
|
2021-02-02 13:09:42 -05:00
|
|
|
|
2020-11-30 10:09:21 -05:00
|
|
|
expect_next_instance_of(Gitlab::Import::Logger) do |logger|
|
2021-03-18 11:09:04 -04:00
|
|
|
expect(logger).to receive(:warn)
|
2020-11-30 10:09:21 -05:00
|
|
|
.with(
|
2021-03-10 19:09:38 -05:00
|
|
|
log_params(
|
|
|
|
context,
|
2021-03-18 11:09:04 -04:00
|
|
|
message: 'Skipping pipeline due to failed entity',
|
2021-03-10 19:09:38 -05:00
|
|
|
pipeline_class: 'BulkImports::MyPipeline'
|
|
|
|
)
|
2020-11-30 10:09:21 -05:00
|
|
|
)
|
|
|
|
end
|
|
|
|
|
2021-03-10 19:09:38 -05:00
|
|
|
subject.run
|
2020-11-30 10:09:21 -05:00
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
end
|
2021-03-10 19:09:38 -05:00
|
|
|
|
2021-03-23 14:09:05 -04:00
|
|
|
def log_params(context, extra = {})
|
|
|
|
{
|
|
|
|
bulk_import_id: context.bulk_import.id,
|
|
|
|
bulk_import_entity_id: context.entity.id,
|
|
|
|
bulk_import_entity_type: context.entity.source_type,
|
|
|
|
context_extra: context.extra
|
|
|
|
}.merge(extra)
|
|
|
|
end
|
|
|
|
|
|
|
|
def extracted_data(has_next_page: false)
|
|
|
|
BulkImports::Pipeline::ExtractedData.new(
|
|
|
|
data: { foo: :bar },
|
|
|
|
page_info: {
|
|
|
|
'has_next_page' => has_next_page,
|
|
|
|
'end_cursor' => has_next_page ? 'cursor' : nil
|
|
|
|
}
|
|
|
|
)
|
|
|
|
end
|
2021-03-10 19:09:38 -05:00
|
|
|
end
|
2020-10-23 11:08:42 -04:00
|
|
|
end
|