Expose multi-CV feature downstream

app/models/concerns/setting_branding.rb

@@ -7,7 +7,7 @@ def replace_keywords(keyword)
     end
 
     def [](name)
-      if UpstreamOnlySettings::SETTINGS.include?(name.to_s)
+      if UpstreamOnlySettings.include?(name.to_s)
         Rails.logger.debug "Setting '#{name}' is not available in Satellite; ignoring"
         return nil
       end

app/models/concerns/upstream_only_settings.rb

@@ -1,6 +1,13 @@
 # Settings to hide in downstream (will return nil for all values)
-module UpstreamOnlySettings
+class UpstreamOnlySettings
   SETTINGS = %w[
-    allow_multiple_content_views
   ].freeze
+
+  def self.include?(key)
+    new.include?(key)
+  end
+
+  def include?(key)
+    SETTINGS.include?(key.to_s)
+  end
 end

test/unit/setting_registry_branding_test.rb

@@ -27,14 +27,28 @@ class SettingRegistryBrandingTest < ActiveSupport::TestCase
     end
 
     test 'hides upstream-only settings' do
-      assert_nil Setting['allow_multiple_content_views']
+      Foreman::SettingManager.define('test2') do
+        category(:Content) do
+          setting(
+            :test_setting,
+            description: N_("Upstream-only setting test"),
+            default: 'definitely not falsey',
+            full_name: N_('Upstream-only setting test'),
+            type: :string
+          )
+        end
+      end
+      Foreman.settings.load
+      UpstreamOnlySettings.expects(:include?).with('test_setting').returns(true)
+      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.expects(:include?).with('test_setting').returns(true)
+    Rails.logger.expects(:debug).with('Setting \'test_setting\' is not available in Satellite; ignoring')
+    Setting['test_setting']
   end
 end