diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-11-26 03:14:05 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-11-26 03:14:05 +0300 |
commit | 118b785094556d1bc1a9ead43b41edfaec5dc8ff (patch) | |
tree | 58788dfb5332c49a0b259fc36c08dc811cf02589 /spec | |
parent | c592490e7fbc4ffe9eab4f797c03a5b7b7db5fb2 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec')
-rw-r--r-- | spec/lib/gitlab/usage_data_spec.rb | 30 | ||||
-rw-r--r-- | spec/migrations/20211116091751_change_namespace_type_default_to_user_spec.rb | 22 | ||||
-rw-r--r-- | spec/models/concerns/case_sensitivity_spec.rb | 1 |
3 files changed, 32 insertions, 21 deletions
diff --git a/spec/lib/gitlab/usage_data_spec.rb b/spec/lib/gitlab/usage_data_spec.rb index cf544c07195..dfa80ac62fc 100644 --- a/spec/lib/gitlab/usage_data_spec.rb +++ b/spec/lib/gitlab/usage_data_spec.rb @@ -1402,33 +1402,21 @@ RSpec.describe Gitlab::UsageData, :aggregate_failures do let(:categories) { ::Gitlab::UsageDataCounters::HLLRedisCounter.categories } - context 'with redis_hll_tracking feature enabled' do - it 'has all known_events' do - stub_feature_flags(redis_hll_tracking: true) + it 'has all known_events' do + expect(subject).to have_key(:redis_hll_counters) - expect(subject).to have_key(:redis_hll_counters) + expect(subject[:redis_hll_counters].keys).to match_array(categories) - expect(subject[:redis_hll_counters].keys).to match_array(categories) + categories.each do |category| + keys = ::Gitlab::UsageDataCounters::HLLRedisCounter.events_for_category(category) - categories.each do |category| - keys = ::Gitlab::UsageDataCounters::HLLRedisCounter.events_for_category(category) + metrics = keys.map { |key| "#{key}_weekly" } + keys.map { |key| "#{key}_monthly" } - metrics = keys.map { |key| "#{key}_weekly" } + keys.map { |key| "#{key}_monthly" } - - if ::Gitlab::UsageDataCounters::HLLRedisCounter::CATEGORIES_FOR_TOTALS.include?(category) - metrics.append("#{category}_total_unique_counts_weekly", "#{category}_total_unique_counts_monthly") - end - - expect(subject[:redis_hll_counters][category].keys).to match_array(metrics) + if ::Gitlab::UsageDataCounters::HLLRedisCounter::CATEGORIES_FOR_TOTALS.include?(category) + metrics.append("#{category}_total_unique_counts_weekly", "#{category}_total_unique_counts_monthly") end - end - end - - context 'with redis_hll_tracking disabled' do - it 'does not have redis_hll_tracking key' do - stub_feature_flags(redis_hll_tracking: false) - expect(subject).not_to have_key(:redis_hll_counters) + expect(subject[:redis_hll_counters][category].keys).to match_array(metrics) end end end diff --git a/spec/migrations/20211116091751_change_namespace_type_default_to_user_spec.rb b/spec/migrations/20211116091751_change_namespace_type_default_to_user_spec.rb new file mode 100644 index 00000000000..c1a9d840317 --- /dev/null +++ b/spec/migrations/20211116091751_change_namespace_type_default_to_user_spec.rb @@ -0,0 +1,22 @@ +# frozen_string_literal: true + +require 'spec_helper' + +require_migration! + +RSpec.describe ChangeNamespaceTypeDefaultToUser do + let(:namespaces) { table(:namespaces) } + + it 'defaults type to User' do + some_namespace1 = namespaces.create!(name: 'magic namespace1', path: 'magicnamespace1', type: nil) + + expect(some_namespace1.reload.type).to be_nil + + migrate! + + some_namespace2 = namespaces.create!(name: 'magic namespace2', path: 'magicnamespace2', type: nil) + + expect(some_namespace1.reload.type).to be_nil + expect(some_namespace2.reload.type).to eq 'User' + end +end diff --git a/spec/models/concerns/case_sensitivity_spec.rb b/spec/models/concerns/case_sensitivity_spec.rb index 269f9577267..124fb939c70 100644 --- a/spec/models/concerns/case_sensitivity_spec.rb +++ b/spec/models/concerns/case_sensitivity_spec.rb @@ -9,6 +9,7 @@ RSpec.describe CaseSensitivity do Class.new(ActiveRecord::Base) do include CaseSensitivity self.table_name = 'namespaces' + self.inheritance_column = :_type_disabled end end |