diff options
Diffstat (limited to 'spec/finders/ci')
-rw-r--r-- | spec/finders/ci/jobs_finder_spec.rb | 33 | ||||
-rw-r--r-- | spec/finders/ci/testing/daily_build_group_report_results_finder_spec.rb | 99 |
2 files changed, 131 insertions, 1 deletions
diff --git a/spec/finders/ci/jobs_finder_spec.rb b/spec/finders/ci/jobs_finder_spec.rb index 4a6585e3f2b..ab056dd26e8 100644 --- a/spec/finders/ci/jobs_finder_spec.rb +++ b/spec/finders/ci/jobs_finder_spec.rb @@ -9,7 +9,7 @@ RSpec.describe Ci::JobsFinder, '#execute' do let_it_be(:pipeline) { create(:ci_pipeline, project: project) } let_it_be(:job_1) { create(:ci_build) } let_it_be(:job_2) { create(:ci_build, :running) } - let_it_be(:job_3) { create(:ci_build, :success, pipeline: pipeline) } + let_it_be(:job_3) { create(:ci_build, :success, pipeline: pipeline, name: 'build') } let(:params) { {} } @@ -95,4 +95,35 @@ RSpec.describe Ci::JobsFinder, '#execute' do end end end + + context 'when pipeline is present' do + before_all do + project.add_maintainer(user) + job_3.update!(retried: true) + end + + let_it_be(:job_4) { create(:ci_build, :success, pipeline: pipeline, name: 'build') } + + subject { described_class.new(current_user: user, pipeline: pipeline, params: params).execute } + + it 'does not return retried jobs by default' do + expect(subject).to match_array([job_4]) + end + + context 'when include_retried is false' do + let(:params) { { include_retried: false } } + + it 'does not return retried jobs' do + expect(subject).to match_array([job_4]) + end + end + + context 'when include_retried is true' do + let(:params) { { include_retried: true } } + + it 'returns retried jobs' do + expect(subject).to match_array([job_3, job_4]) + end + end + end end diff --git a/spec/finders/ci/testing/daily_build_group_report_results_finder_spec.rb b/spec/finders/ci/testing/daily_build_group_report_results_finder_spec.rb new file mode 100644 index 00000000000..a703f3b800c --- /dev/null +++ b/spec/finders/ci/testing/daily_build_group_report_results_finder_spec.rb @@ -0,0 +1,99 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Ci::Testing::DailyBuildGroupReportResultsFinder do + describe '#execute' do + let_it_be(:project) { create(:project, :private) } + let(:user_without_permission) { create(:user) } + let_it_be(:user_with_permission) { project.owner } + let_it_be(:ref_path) { 'refs/heads/master' } + let(:limit) { nil } + let_it_be(:default_branch) { false } + let(:start_date) { '2020-03-09' } + let(:end_date) { '2020-03-10' } + let(:sort) { true } + + let_it_be(:rspec_coverage_1) { create_daily_coverage('rspec', 79.0, '2020-03-09') } + let_it_be(:karma_coverage_1) { create_daily_coverage('karma', 89.0, '2020-03-09') } + let_it_be(:rspec_coverage_2) { create_daily_coverage('rspec', 95.0, '2020-03-10') } + let_it_be(:karma_coverage_2) { create_daily_coverage('karma', 92.0, '2020-03-10') } + let_it_be(:rspec_coverage_3) { create_daily_coverage('rspec', 97.0, '2020-03-11') } + let_it_be(:karma_coverage_3) { create_daily_coverage('karma', 99.0, '2020-03-11') } + + let(:finder) { described_class.new(params: params, current_user: current_user) } + + let(:params) do + { + project: project, + coverage: true, + ref_path: ref_path, + start_date: start_date, + end_date: end_date, + limit: limit, + sort: sort + } + end + + subject(:coverages) { finder.execute } + + context 'when params are provided' do + context 'when current user is not allowed to read data' do + let(:current_user) { user_without_permission } + + it 'returns an empty collection' do + expect(coverages).to be_empty + end + end + + context 'when current user is allowed to read data' do + let(:current_user) { user_with_permission } + + it 'returns matching coverages within the given date range' do + expect(coverages).to match_array([ + karma_coverage_2, + rspec_coverage_2, + karma_coverage_1, + rspec_coverage_1 + ]) + end + + context 'when ref_path is nil' do + let(:default_branch) { true } + let(:ref_path) { nil } + + it 'returns coverages for the default branch' do + rspec_coverage_4 = create_daily_coverage('rspec', 66.0, '2020-03-10') + + expect(coverages).to contain_exactly(rspec_coverage_4) + end + end + + context 'when limit is specified' do + let(:limit) { 2 } + + it 'returns limited number of matching coverages within the given date range' do + expect(coverages).to match_array([ + karma_coverage_2, + rspec_coverage_2 + ]) + end + end + end + end + end + + private + + def create_daily_coverage(group_name, coverage, date) + create( + :ci_daily_build_group_report_result, + project: project, + ref_path: ref_path || 'feature-branch', + group_name: group_name, + data: { 'coverage' => coverage }, + date: date, + default_branch: default_branch + ) + end +end |