foreman_discovery 17.0.1 → 18.0.0

data/test/functional/discovered_hosts_controller_test.rb

@@ -28,10 +28,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
   end
 
   def test_index_with_custom_column
-    FactoryBot.create(:setting,
-                       :name => 'discovery_fact_column',
-                       :value => "bios_vendor",
-                       :category => 'Setting::Discovered')
+    Setting['discovery_fact_column'] = "bios_vendor"
     facts = @facts.merge({"bios_vendor" => "QEMU"})
     discover_host_from_facts(facts)
     get :index, params: {}, session: set_session_user_default_reader
@@ -85,7 +82,11 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
 
   def test_edit_form_quick_submit
     domain = FactoryBot.create(:domain)
-    hostgroup = FactoryBot.create(:hostgroup, :with_subnet, :with_environment, :with_rootpass, :with_os, :domain => domain)
+    if defined? ForemanPuppet
+      hostgroup = FactoryBot.create(:hostgroup, :with_subnet, :with_puppet_enc, :with_rootpass, :with_os, :domain => domain)
+    else
+      hostgroup = FactoryBot.create(:hostgroup, :with_subnet, :with_rootpass, :with_os, :domain => domain)
+    end
     new_ip = hostgroup.subnet.ipaddr.to_s
     host = discover_host_from_facts(@facts.merge({
                                                     'ipaddress' => new_ip,

data/test/integration/discovered_hosts_test.rb

@@ -161,9 +161,11 @@ class DiscoveredHostsTest < IntegrationTestWithJavascript
       let(:hostgroup_environment) { FactoryBot.create(:environment) }
       let(:hostgroup_domain) { FactoryBot.create(:domain) }
       let(:hostgroup) do
-        FactoryBot.create(:hostgroup, :with_os,
-                           environment: hostgroup_environment,
-                           domain: hostgroup_domain)
+        if defined?(ForemanPuppet)
+          FactoryBot.create(:hostgroup, :with_os, environment: hostgroup_environment, domain: hostgroup_domain)
+        else
+          FactoryBot.create(:hostgroup, :with_os, domain: hostgroup_domain)
+        end
       end
 
       setup do
@@ -174,10 +176,14 @@ class DiscoveredHostsTest < IntegrationTestWithJavascript
       end
 
       it 'sets inherited attributes' do
-        %i[environment architecture operatingsystem].each do |attribute|
+        %i[architecture operatingsystem].each do |attribute|
           assert_selected "#host_#{attribute}_id", hostgroup.send(attribute).id
         end
 
+        if defined? ForemanPuppet
+          assert_selected "#host_puppet_attributes_environment_id", hostgroup.send(:environment).id
+        end
+
         page.find('a[href="#network"]').click
         page.find_button('Edit').click
         assert_selected '#host_interfaces_attributes_0_domain_id',

data/test/test_helper_discovery.rb

@@ -52,40 +52,46 @@ def extract_form_errors(response)
 end
 
 def set_default_settings
-  FactoryBot.create(:setting, :name => 'discovery_fact', :value => 'discovery_bootif', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_hostname', :value => 'discovery_bootif', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_auto', :value => true, :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_reboot', :value => true, :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_organization', :value => "Organization 1", :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_location', :value => "Location 1", :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_prefix', :value => 'mac', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_clean_facts', :value => false, :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_lock', :value => false, :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_pxelinux_lock_template', :value => 'pxelinux_discovery', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_pxegrub_lock_template', :value => 'pxegrub_discovery', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_pxegrub2_lock_template', :value => 'pxegrub2_discovery', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_always_rebuild_dns', :value => true, :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_error_on_existing', :value => false, :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_naming', :value => 'Fact', :category => 'Setting::Discovered')
-  FactoryBot.create(:setting, :name => 'discovery_auto_bond', :value => false, :category => 'Setting::Discovered')
+  Setting['discovery_fact'] = 'discovery_bootif'
+  Setting['discovery_hostname'] = 'discovery_bootif'
+  Setting['discovery_auto'] = true
+  Setting['discovery_reboot'] = true
+  Setting['discovery_organization'] = "Organization 1"
+  Setting['discovery_location'] = "Location 1"
+  Setting['discovery_prefix'] = 'mac'
+  Setting['discovery_clean_facts'] = false
+  Setting['discovery_lock'] = false
+  Setting['discovery_pxelinux_lock_template'] = 'pxelinux_discovery'
+  Setting['discovery_pxegrub_lock_template'] = 'pxegrub_discovery'
+  Setting['discovery_pxegrub2_lock_template'] = 'pxegrub2_discovery'
+  Setting['discovery_always_rebuild_dns'] = true
+  Setting['discovery_error_on_existing'] = false
+  Setting['discovery_naming'] = 'Fact'
+  Setting['discovery_auto_bond'] = false
 end
 
 def setup_hostgroup(host)
   domain = FactoryBot.create(:domain)
   subnet = FactoryBot.create(:subnet_ipv4, :network => "192.168.100.0")
-  environment = FactoryBot.create(:environment, :organizations => [host.organization], :locations => [host.location])
   medium = FactoryBot.create(:medium, :organizations => [host.organization], :locations => [host.location])
   os = FactoryBot.create(:operatingsystem, :with_ptables, :with_archs, :media => [medium])
-  hostgroup = FactoryBot.create(
-    :hostgroup, :with_rootpass, :with_puppet_orchestration,
+  args = {
     :operatingsystem => os,
     :architecture => os.architectures.first,
     :ptable => os.ptables.first,
     :medium => os.media.first,
-    :environment => environment,
     :subnet => subnet,
     :domain => domain,
-    :organizations => [host.organization], :locations => [host.location])
+    :organizations => [host.organization],
+    :locations => [host.location]
+  }
+  if defined?(ForemanPuppet)
+    environment = FactoryBot.create(:environment, :organizations => [host.organization], :locations => [host.location])
+    args[:environment] = environment
+    hostgroup = FactoryBot.create(:hostgroup, :with_rootpass, :with_puppet_enc, **args)
+  else
+    hostgroup = FactoryBot.create(:hostgroup, :with_rootpass, **args)
+  end
   domain.subnets << hostgroup.subnet
   hostgroup.medium.organizations |= [host.organization]
   hostgroup.medium.locations |= [host.location]
@@ -95,12 +101,14 @@ def setup_hostgroup(host)
   hostgroup.domain.locations |= [host.location]
   hostgroup.subnet.organizations |= [host.organization]
   hostgroup.subnet.locations |= [host.location]
-  hostgroup.environment.organizations |= [host.organization]
-  hostgroup.environment.locations |= [host.location]
-  hostgroup.puppet_proxy.organizations |= [host.organization]
-  hostgroup.puppet_proxy.locations |= [host.location]
-  hostgroup.puppet_ca_proxy.organizations |= [host.organization]
-  hostgroup.puppet_ca_proxy.locations |= [host.location]
+  if defined?(ForemanPuppet)
+    hostgroup.environment.organizations |= [host.organization]
+    hostgroup.environment.locations |= [host.location]
+    hostgroup.puppet_proxy.organizations |= [host.organization]
+    hostgroup.puppet_proxy.locations |= [host.location]
+    hostgroup.puppet_ca_proxy.organizations |= [host.organization]
+    hostgroup.puppet_ca_proxy.locations |= [host.location]
+  end
   hostgroup
 end
 

data/test/test_plugin_helper.rb

@@ -4,4 +4,11 @@ require_relative './test_helper_discovery'
 
 # Add plugin to FactoryBot's paths
 FactoryBot.definition_file_paths << File.join(File.dirname(__FILE__), 'factories')
+
+# Add dependencies
+if defined? ForemanPuppet
+  FactoryBot.definition_file_paths << File.join("#{ForemanPuppet::Engine.root}/test", 'factories')
+end
+
+# And reload
 FactoryBot.reload

data/test/unit/discovered_extensions_test.rb

@@ -156,7 +156,17 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     subnet = FactoryBot.create(:subnet_ipv4, :tftp, :dhcp, :name => 'subnet_100', :network => '192.168.100.0', :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
     host = discover_host_from_facts(facts)
     assert_equal subnet, host.subnet
-    hostgroup = FactoryBot.create(:hostgroup, :with_environment, :with_rootpass, :with_puppet_orchestration, :with_os, :pxe_loader => "PXELinux BIOS", :subnet => subnet, :domain => domain)
+    if defined? ForemanPuppet
+      hostgroup = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_rootpass, :with_os,
+        pxe_loader: "PXELinux BIOS",
+        subnet: subnet,
+        domain: domain)
+    else
+      hostgroup = FactoryBot.create(:hostgroup, :with_rootpass, :with_os,
+        pxe_loader: "PXELinux BIOS",
+        subnet: subnet,
+        domain: domain)
+    end
     r1 = FactoryBot.create(:discovery_rule, :priority => 1, :search => "facts.somefact = abc", :organizations => [host.organization], :locations => [host.location], :hostgroup => hostgroup)
     host.primary_interface.expects(:queue_tftp).at_least(1)
     host.primary_interface.expects(:queue_dhcp).at_least(1)
@@ -174,12 +184,12 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     assert_equal hostgroup.subnet, managed_host.provision_interface.subnet
     refute_nil hostgroup.domain, managed_host.domain
     assert_equal hostgroup.domain, managed_host.domain
-    refute_nil hostgroup.environment, managed_host.environment
-    assert_equal hostgroup.environment, managed_host.environment
-    refute_nil hostgroup.puppet_proxy, managed_host.puppet_proxy
-    assert_equal hostgroup.puppet_proxy, managed_host.puppet_proxy
-    refute_nil hostgroup.puppet_ca_proxy, managed_host.puppet_ca_proxy
-    assert_equal hostgroup.puppet_ca_proxy, managed_host.puppet_ca_proxy
+    if defined? ForemanPuppet
+      refute_nil hostgroup.environment, managed_host.environment
+      assert_equal hostgroup.environment, managed_host.environment
+      refute_nil hostgroup.puppet_proxy, managed_host.puppet_proxy
+      assert_equal hostgroup.puppet_proxy, managed_host.puppet_proxy
+    end
   end
 
   class StubIPAM
@@ -196,7 +206,7 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     subnet2 = FactoryBot.create(:subnet_ipv4, :tftp, :dhcp, :name => 'subnet_101', :network => '192.168.101.0', :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
     host = discover_host_from_facts(facts)
     assert_equal subnet, host.subnet
-    hostgroup = FactoryBot.create(:hostgroup, :with_environment, :with_rootpass, :with_puppet_orchestration, :with_os, :pxe_loader => "PXELinux BIOS", :subnet => subnet2, :domain => domain)
+    hostgroup = FactoryBot.create(:hostgroup, :with_rootpass, :with_os, :pxe_loader => "PXELinux BIOS", :subnet => subnet2, :domain => domain)
     r1 = FactoryBot.create(:discovery_rule, :priority => 1, :search => "facts.somefact = abc", :organizations => [host.organization], :locations => [host.location], :hostgroup => hostgroup)
     Subnet.any_instance.expects(:unused_ip).with(host.mac).returns(StubIPAM.new)
     host.primary_interface.stubs(:queue_tftp)
@@ -219,7 +229,7 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     subnet2 = FactoryBot.create(:subnet_ipv6, :tftp, :dhcp, :network => "2001:db9::/32", :mask => "ffff:ffff::", :name => "ipv6_provision", :ipam => IPAM::MODES[:eui64], :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
     host = discover_host_from_facts(facts)
     assert_equal subnet, host.subnet6
-    hostgroup = FactoryBot.create(:hostgroup, :with_environment, :with_rootpass, :with_puppet_orchestration, :with_os, :pxe_loader => "PXELinux BIOS", :subnet6 => subnet2, :domain => domain)
+    hostgroup = FactoryBot.create(:hostgroup, :with_rootpass, :with_os, :pxe_loader => "PXELinux BIOS", :subnet6 => subnet2, :domain => domain)
     r1 = FactoryBot.create(:discovery_rule, :priority => 1, :search => "facts.somefact = abc", :organizations => [host.organization], :locations => [host.location], :hostgroup => hostgroup)
     host.primary_interface.stubs(:queue_tftp)
     host.primary_interface.stubs(:queue_dhcp)
@@ -239,7 +249,17 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     refute host.subnet
     domain = FactoryBot.create(:domain)
     subnet = FactoryBot.create(:subnet_ipv4, :tftp, :dhcp, :name => 'subnet_100', :network => '192.168.100.0', :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
-    hostgroup = FactoryBot.create(:hostgroup, :with_environment, :with_rootpass, :with_puppet_orchestration, :with_os, :pxe_loader => "PXELinux BIOS", :subnet => subnet, :domain => domain)
+    if defined? ForemanPuppet
+      hostgroup = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_rootpass, :with_os,
+        pxe_loader: "PXELinux BIOS",
+        subnet: subnet,
+        domain: domain)
+    else
+      hostgroup = FactoryBot.create(:hostgroup, :with_rootpass, :with_os,
+        pxe_loader: "PXELinux BIOS",
+        subnet: subnet,
+        domain: domain)
+    end
     r1 = FactoryBot.create(:discovery_rule, :priority => 1, :search => "facts.somefact = abc", :organizations => [host.organization], :locations => [host.location], :hostgroup => hostgroup)
     host.primary_interface.expects(:queue_tftp).at_least(1)
     host.primary_interface.expects(:queue_dhcp).at_least(1)
@@ -257,12 +277,12 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     assert_equal hostgroup.subnet, managed_host.provision_interface.subnet
     refute_nil hostgroup.domain, managed_host.domain
     assert_equal hostgroup.domain, managed_host.domain
-    refute_nil hostgroup.environment, managed_host.environment
-    assert_equal hostgroup.environment, managed_host.environment
-    refute_nil hostgroup.puppet_proxy, managed_host.puppet_proxy
-    assert_equal hostgroup.puppet_proxy, managed_host.puppet_proxy
-    refute_nil hostgroup.puppet_ca_proxy, managed_host.puppet_ca_proxy
-    assert_equal hostgroup.puppet_ca_proxy, managed_host.puppet_ca_proxy
+    if defined? ForemanPuppet
+      refute_nil hostgroup.environment, managed_host.environment
+      assert_equal hostgroup.environment, managed_host.environment
+      refute_nil hostgroup.puppet_proxy, managed_host.puppet_proxy
+      assert_equal hostgroup.puppet_proxy, managed_host.puppet_proxy
+    end
   end
 
   def setup_normal_renderer

data/test/unit/discovery_attribute_set_test.rb

@@ -6,14 +6,8 @@ class DiscoveryAttributeSetTest < ActiveSupport::TestCase
 
   setup do
     @facts = parse_json_fixture('regular_host', true)
-    FactoryBot.create(:setting,
-                       :name => 'discovery_hostname',
-                       :value => 'discovery_bootif',
-                       :category => 'Setting::Discovered')
-    FactoryBot.create(:setting,
-                       :name => 'discovery_prefix',
-                       :value => 'mac',
-                       :category => 'Setting::Discovered')
+    Setting['discovery_hostname'] = 'discovery_bootif'
+    Setting['discovery_prefix'] = 'mac'
     ::ForemanDiscovery::HostConverter.stubs(:unused_ip_for_host)
   end
 

data/test/unit/fact_parser_test.rb

@@ -55,10 +55,7 @@ class FactParserTest < ActiveSupport::TestCase
   end
 
   test "#suggested_primary_interface detects interface even when 'ignore_puppet_facts_for_provisioning' is set" do
-    FactoryBot.create(:setting,
-                       :name => 'ignore_puppet_facts_for_provisioning',
-                       :value => true,
-                       :category => 'Setting::Provisioning')
+    Setting['ignore_puppet_facts_for_provisioning'] = true
     assert_equal 'eth2', @parser.suggested_primary_interface(@host).try(:first)
   end
 

data/test/unit/managed_extensions_test.rb

@@ -19,6 +19,7 @@ class ManagedExtensionsTest < ActiveSupport::TestCase
       @host = StubHost.new
       @host.type = "Host::Discovered"
       @host.stubs(:ip).returns("192.168.1.1")
+      @host.stubs(:ip6).returns(nil)
       @host.stubs(:old).returns(@host)
       @facts = {}
       @host.stubs(:facts).returns(@facts)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_discovery
 version: !ruby/object:Gem::Version
-  version: 17.0.1
+  version: 18.0.0
 platform: ruby
 authors:
 - Aditi Puntambekar
@@ -41,6 +41,7 @@ authors:
 - Lukáš Zapletal
 - Marek Hulan
 - Marek Hulán
+- MariaAga
 - Martin Bačovský
 - Matt Jarvis
 - Michael Moll
@@ -73,7 +74,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-16 00:00:00.000000000 Z
+date: 2021-09-20 00:00:00.000000000 Z
 dependencies: []
 description: MaaS Discovery Plugin engine for Foreman
 email: gsutclif@redhat.com
@@ -100,12 +101,14 @@ files:
 - app/mailers/discovered_mailer.rb
 - app/models/concerns/discovery_taxonomy_extensions.rb
 - app/models/discovery_attribute_set.rb
+- app/models/discovery_fact_name.rb
 - app/models/discovery_rule.rb
 - app/models/host/discovered.rb
 - app/models/host/managed_extensions.rb
 - app/models/hostgroup_extensions.rb
 - app/models/nic/managed_extensions.rb
 - app/models/setting/discovered.rb
+- app/services/discovery_fact_importer.rb
 - app/services/foreman_discovery/fact_parser.rb
 - app/services/foreman_discovery/fact_to_category_resolver.rb
 - app/services/foreman_discovery/host_converter.rb
@@ -266,12 +269,10 @@ files:
 - test/functional/api/v2/discovered_hosts_controller_test.rb
 - test/functional/api/v2/discovery_rules_controller_test.rb
 - test/functional/api/v2/fact_value_extensions_test.rb
-- test/functional/api/v2/settings_controller_test.rb
 - test/functional/discovered_hosts_controller_test.rb
 - test/functional/discovery_rules_controller_test.rb
 - test/functional/foreman_discovery/concerns/hosts_controller_extensions_test.rb
 - test/integration/discovered_hosts_test.rb
-- test/models/setting_test.rb
 - test/test_helper_discovery.rb
 - test/test_plugin_helper.rb
 - test/unit/discovered_extensions_test.rb
@@ -284,7 +285,6 @@ files:
 - test/unit/host_discovered_test.rb
 - test/unit/lldp_neighbors_test.rb
 - test/unit/managed_extensions_test.rb
-- test/unit/setting_discovered_test.rb
 - test/unit/ui_notifications/destroy_host_test.rb
 - test/unit/ui_notifications/new_host_test.rb
 - webpack/__mocks__/foremanReact/common/I18n.js
@@ -349,26 +349,23 @@ test_files:
 - test/facts/only-ipv6.json
 - test/facts/skylake-ipv6.json
 - test/functional/api/v2/discovery_rules_controller_test.rb
-- test/functional/api/v2/fact_value_extensions_test.rb
-- test/functional/api/v2/settings_controller_test.rb
 - test/functional/api/v2/discovered_hosts_controller_test.rb
+- test/functional/api/v2/fact_value_extensions_test.rb
 - test/functional/foreman_discovery/concerns/hosts_controller_extensions_test.rb
-- test/functional/discovered_hosts_controller_test.rb
 - test/functional/discovery_rules_controller_test.rb
+- test/functional/discovered_hosts_controller_test.rb
 - test/integration/discovered_hosts_test.rb
-- test/models/setting_test.rb
 - test/unit/ui_notifications/destroy_host_test.rb
 - test/unit/ui_notifications/new_host_test.rb
 - test/unit/discovered_mailer_test.rb
 - test/unit/discovery_taxonomy_extensions_test.rb
-- test/unit/fact_parser_test.rb
 - test/unit/lldp_neighbors_test.rb
-- test/unit/setting_discovered_test.rb
-- test/unit/discovered_extensions_test.rb
-- test/unit/discovery_attribute_set_test.rb
 - test/unit/host_discovered_test.rb
-- test/unit/managed_extensions_test.rb
 - test/unit/discovery_rule_test.rb
 - test/unit/fact_to_category_resolver_test.rb
-- test/test_plugin_helper.rb
+- test/unit/managed_extensions_test.rb
+- test/unit/discovery_attribute_set_test.rb
+- test/unit/fact_parser_test.rb
+- test/unit/discovered_extensions_test.rb
 - test/test_helper_discovery.rb
+- test/test_plugin_helper.rb

data/test/functional/api/v2/settings_controller_test.rb

@@ -1,22 +0,0 @@
-require_relative '../../../test_plugin_helper'
-
-class Api::V2::SettingsControllerTest < ActionController::TestCase
-  setup do
-    FactoryBot.create(:setting, :name => 'discovery_prefix', :value => 'mac', :category => 'Setting::Discovered')
-  end
-
-  test_attributes :pid => '2c5ecb7e-87bc-4980-9620-7ae00e3f360e'
-  test "should not update hostname prefix without value" do
-    setting = Setting.find_by_name("discovery_prefix")
-    put :update, params: { :id => setting.id, :setting => { :value => '' } }
-    assert_nil JSON.parse(@response.body)['value']
-  end
-
-  test_attributes :pid => '4969994d-f934-4f0e-9a98-476b87eb0527'
-  test "should update hostname prefix" do
-    value = RFauxFactory.gen_alpha
-    setting = Setting.find_by_name("discovery_prefix")
-    put :update, params: { :id => setting.id, :setting => { :value => value } }
-    assert_equal JSON.parse(@response.body)['value'], value, "Can't update discovery_prefix setting with valid value #{value}"
-  end
-end

data/test/models/setting_test.rb

@@ -1,11 +0,0 @@
-require_relative '../test_plugin_helper'
-
-class SettingTest < ActiveSupport::TestCase
-  test "should update hostname prefix with multiple valid values" do
-    setting = FactoryBot.create(:setting, :name => 'discovery_prefix', :value => 'mac', :category => 'Setting::Discovered')
-    RFauxFactory.gen_strings(exclude: [:numeric, :alphanumeric]).values.each do |value|
-      setting.value = value
-      assert setting.valid?, "Can't update discovery_prefix setting with valid value #{value}"
-    end
-  end
-end

data/test/unit/setting_discovered_test.rb

@@ -1,9 +0,0 @@
-require_relative '../test_plugin_helper'
-
-class SettingDiscoveredTest < ActiveSupport::TestCase
-  test "can create setting of discovered type" do
-    assert Setting.create(:name => "foo", :default => "bar", :description => "test foo", :category => "Setting::Discovered")
-    s = Setting::Discovered.first
-    assert_equal "foo", s.name
-  end
-end