diff options
Diffstat (limited to 'spec/lib/service_ping')
-rw-r--r-- | spec/lib/service_ping/build_payload_spec.rb | 48 | ||||
-rw-r--r-- | spec/lib/service_ping/devops_report_spec.rb | 35 | ||||
-rw-r--r-- | spec/lib/service_ping/permit_data_categories_spec.rb | 45 | ||||
-rw-r--r-- | spec/lib/service_ping/service_ping_settings_spec.rb | 47 |
4 files changed, 175 insertions, 0 deletions
diff --git a/spec/lib/service_ping/build_payload_spec.rb b/spec/lib/service_ping/build_payload_spec.rb new file mode 100644 index 00000000000..6cce07262b2 --- /dev/null +++ b/spec/lib/service_ping/build_payload_spec.rb @@ -0,0 +1,48 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe ServicePing::BuildPayload do + describe '#execute', :without_license do + subject(:service_ping_payload) { described_class.new.execute } + + include_context 'stubbed service ping metrics definitions' do + let(:subscription_metrics) do + [ + metric_attributes('active_user_count', "subscription") + ] + end + end + + context 'when usage_ping_enabled setting is false' do + before do + # Gitlab::CurrentSettings.usage_ping_enabled? == false + stub_config_setting(usage_ping_enabled: false) + end + + it 'returns empty service ping payload' do + expect(service_ping_payload).to eq({}) + end + end + + context 'when usage_ping_enabled setting is true' do + before do + # Gitlab::CurrentSettings.usage_ping_enabled? == true + stub_config_setting(usage_ping_enabled: true) + end + + it_behaves_like 'complete service ping payload' + + context 'with require stats consent enabled' do + before do + allow(User).to receive(:single_user) + .and_return(instance_double(User, :user, requires_usage_stats_consent?: true)) + end + + it 'returns empty service ping payload' do + expect(service_ping_payload).to eq({}) + end + end + end + end +end diff --git a/spec/lib/service_ping/devops_report_spec.rb b/spec/lib/service_ping/devops_report_spec.rb new file mode 100644 index 00000000000..793f3066097 --- /dev/null +++ b/spec/lib/service_ping/devops_report_spec.rb @@ -0,0 +1,35 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe ServicePing::DevopsReport do + let_it_be(:data) { { "conv_index": {} }.to_json } + let_it_be(:subject) { ServicePing::DevopsReport.new(Gitlab::Json.parse(data)) } + let_it_be(:devops_report) { DevOpsReport::Metric.new } + + describe '#execute' do + context 'when metric is persisted' do + before do + allow(DevOpsReport::Metric).to receive(:create).and_return(devops_report) + allow(devops_report).to receive(:persisted?).and_return(true) + end + + it 'does not call `track_and_raise_for_dev_exception`' do + expect(Gitlab::ErrorTracking).not_to receive(:track_and_raise_for_dev_exception) + subject.execute + end + end + + context 'when metric is not persisted' do + before do + allow(DevOpsReport::Metric).to receive(:create).and_return(devops_report) + allow(devops_report).to receive(:persisted?).and_return(false) + end + + it 'calls `track_and_raise_for_dev_exception`' do + expect(Gitlab::ErrorTracking).to receive(:track_and_raise_for_dev_exception) + subject.execute + end + end + end +end diff --git a/spec/lib/service_ping/permit_data_categories_spec.rb b/spec/lib/service_ping/permit_data_categories_spec.rb new file mode 100644 index 00000000000..d1027a6f1ab --- /dev/null +++ b/spec/lib/service_ping/permit_data_categories_spec.rb @@ -0,0 +1,45 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe ServicePing::PermitDataCategories do + describe '#execute', :without_license do + subject(:permitted_categories) { described_class.new.execute } + + context 'when usage ping setting is set to true' do + before do + allow(User).to receive(:single_user) + .and_return(instance_double(User, :user, requires_usage_stats_consent?: false)) + stub_config_setting(usage_ping_enabled: true) + end + + it 'returns all categories' do + expect(permitted_categories).to match_array(%w[standard subscription operational optional]) + end + end + + context 'when usage ping setting is set to false' do + before do + allow(User).to receive(:single_user) + .and_return(instance_double(User, :user, requires_usage_stats_consent?: false)) + stub_config_setting(usage_ping_enabled: false) + end + + it 'returns no categories' do + expect(permitted_categories).to match_array([]) + end + end + + context 'when User.single_user&.requires_usage_stats_consent? is required' do + before do + allow(User).to receive(:single_user) + .and_return(instance_double(User, :user, requires_usage_stats_consent?: true)) + stub_config_setting(usage_ping_enabled: true) + end + + it 'returns no categories' do + expect(permitted_categories).to match_array([]) + end + end + end +end diff --git a/spec/lib/service_ping/service_ping_settings_spec.rb b/spec/lib/service_ping/service_ping_settings_spec.rb new file mode 100644 index 00000000000..040a5027274 --- /dev/null +++ b/spec/lib/service_ping/service_ping_settings_spec.rb @@ -0,0 +1,47 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe ServicePing::ServicePingSettings do + using RSpec::Parameterized::TableSyntax + + describe '#product_intelligence_enabled?' do + where(:usage_ping_enabled, :requires_usage_stats_consent, :expected_product_intelligence_enabled) do + # Usage ping enabled + true | false | true + true | true | false + + # Usage ping disabled + false | false | false + false | true | false + end + + with_them do + before do + allow(User).to receive(:single_user) + .and_return(instance_double(User, :user, requires_usage_stats_consent?: requires_usage_stats_consent)) + stub_config_setting(usage_ping_enabled: usage_ping_enabled) + end + + it 'has the correct product_intelligence_enabled?' do + expect(described_class.product_intelligence_enabled?).to eq(expected_product_intelligence_enabled) + end + end + end + + describe '#enabled?' do + describe 'has the correct enabled' do + it 'when false' do + stub_config_setting(usage_ping_enabled: false) + + expect(described_class.enabled?).to eq(false) + end + + it 'when true' do + stub_config_setting(usage_ping_enabled: true) + + expect(described_class.enabled?).to eq(true) + end + end + end +end |