diff options
Diffstat (limited to 'spec/services/ci/build_report_result_service_spec.rb')
-rw-r--r-- | spec/services/ci/build_report_result_service_spec.rb | 32 |
1 files changed, 31 insertions, 1 deletions
diff --git a/spec/services/ci/build_report_result_service_spec.rb b/spec/services/ci/build_report_result_service_spec.rb index 7c2702af086..c5238b7f5e0 100644 --- a/spec/services/ci/build_report_result_service_spec.rb +++ b/spec/services/ci/build_report_result_service_spec.rb @@ -10,13 +10,17 @@ RSpec.describe Ci::BuildReportResultService do let(:build) { create(:ci_build, :success, :test_reports) } it 'creates a build report result entry', :aggregate_failures do + expect { build_report_result }.to change { Ci::BuildReportResult.count }.by(1) expect(build_report_result.tests_name).to eq("test") expect(build_report_result.tests_success).to eq(2) expect(build_report_result.tests_failed).to eq(2) expect(build_report_result.tests_errored).to eq(0) expect(build_report_result.tests_skipped).to eq(0) expect(build_report_result.tests_duration).to eq(0.010284) - expect(Ci::BuildReportResult.count).to eq(1) + end + + it 'tracks unique test cases parsed' do + build_report_result unique_test_cases_parsed = Gitlab::UsageDataCounters::HLLRedisCounter.unique_events( event_names: described_class::EVENT_NAME, @@ -26,6 +30,32 @@ RSpec.describe Ci::BuildReportResultService do expect(unique_test_cases_parsed).to eq(4) end + context 'and build has test report parsing errors' do + let(:build) { create(:ci_build, :success, :broken_test_reports) } + + it 'creates a build report result entry with suite error', :aggregate_failures do + expect { build_report_result }.to change { Ci::BuildReportResult.count }.by(1) + expect(build_report_result.tests_name).to eq("test") + expect(build_report_result.tests_success).to eq(0) + expect(build_report_result.tests_failed).to eq(0) + expect(build_report_result.tests_errored).to eq(0) + expect(build_report_result.tests_skipped).to eq(0) + expect(build_report_result.tests_duration).to eq(0) + expect(build_report_result.suite_error).to be_present + end + + it 'does not track unique test cases parsed' do + build_report_result + + unique_test_cases_parsed = Gitlab::UsageDataCounters::HLLRedisCounter.unique_events( + event_names: described_class::EVENT_NAME, + start_date: 2.weeks.ago, + end_date: 2.weeks.from_now + ) + expect(unique_test_cases_parsed).to eq(0) + end + end + context 'when data has already been persisted' do it 'raises an error and do not persist the same data twice' do expect { 2.times { described_class.new.execute(build) } }.to raise_error(ActiveRecord::RecordNotUnique) |