Skip to content

Commit

Permalink
Expose multi-CV feature downstream
Browse files Browse the repository at this point in the history
  • Loading branch information
jeremylenz committed Sep 27, 2024
1 parent 6d995cc commit 0ed46ea
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
1 change: 0 additions & 1 deletion app/models/concerns/upstream_only_settings.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
# Settings to hide in downstream (will return nil for all values)
module UpstreamOnlySettings
SETTINGS = %w[
allow_multiple_content_views
].freeze
end
8 changes: 5 additions & 3 deletions test/unit/setting_registry_branding_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,16 @@ class SettingRegistryBrandingTest < ActiveSupport::TestCase
end

test 'hides upstream-only settings' do
assert_nil Setting['allow_multiple_content_views']
UpstreamOnlySettings.stubs(:SETTINGS).returns(%w[test_setting])
assert_nil Setting['test_setting']
end
end
end

class SettingBrandingTest < ActiveSupport::TestCase
test 'replaces warning for upstream-only settings' do
Rails.logger.expects(:debug).with('Setting \'allow_multiple_content_views\' is not available in Satellite; ignoring')
Setting['allow_multiple_content_views']
UpstreamOnlySettings.stubs(:SETTINGS).returns(%w[test_setting])
Rails.logger.expects(:debug).with('Setting \'test_setting\' is not available in Satellite; ignoring')
Setting['test_setting']
end
end

0 comments on commit 0ed46ea

Please sign in to comment.