diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-06-08 15:08:54 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-06-08 15:08:54 +0300 |
commit | 473b876fe3d7e0b36eb6268cc44a4fe0d94f4422 (patch) | |
tree | 2f6d5f2ecad53015024b6b1509896f5933c5c3e6 /spec/services | |
parent | dca8df0c90d8727d69b3501b15b481546897f3cd (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/merge_requests/mergeability/logger_spec.rb | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/spec/services/merge_requests/mergeability/logger_spec.rb b/spec/services/merge_requests/mergeability/logger_spec.rb index 1f56b6bebdb..7863b69abf6 100644 --- a/spec/services/merge_requests/mergeability/logger_spec.rb +++ b/spec/services/merge_requests/mergeability/logger_spec.rb @@ -40,6 +40,37 @@ RSpec.describe MergeRequests::Mergeability::Logger, :request_store, feature_cate logger.commit end + context 'when block value responds to #success?' do + let(:success?) { true } + let(:check_result) { instance_double(Gitlab::MergeRequests::Mergeability::CheckResult, success?: success?) } + + let(:extra_data) do + { + 'mergeability.expensive_operation.successful.values' => [success?] + } + end + + shared_examples_for 'success state logger' do + it 'records operation success state' do + expect_next_instance_of(Gitlab::AppJsonLogger) do |app_logger| + expect(app_logger).to receive(:info).with(match(a_hash_including(loggable_data(**extra_data)))) + end + + expect(logger.instrument(mergeability_name: :expensive_operation) { check_result }).to eq(check_result) + + logger.commit + end + end + + it_behaves_like 'success state logger' + + context 'when not successful' do + let(:success?) { false } + + it_behaves_like 'success state logger' + end + end + context 'with multiple observations' do let(:operation_count) { 2 } |