foreman_discovery 22.0.4 → 23.0.0

data/locale/zh_TW/foreman_discovery.po

@@ -5,7 +5,7 @@
 # Translators:
 msgid ""
 msgstr ""
-"Project-Id-Version: foreman_discovery 22.0.2\n"
+"Project-Id-Version: foreman_discovery v22.0.4\n"
 "Report-Msgid-Bugs-To: \n"
 "PO-Revision-Date: 2013-11-15 17:25+0000\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
@@ -256,7 +256,7 @@ msgstr ""
 msgid "Expected discovery_fact '%s' is missing, unable to detect primary interface and set hostname"
 msgstr ""
 
-msgid "Extra facter columns to show in host lists (separate by comma)"
+msgid "Extra facter columns to show in host lists"
 msgstr ""
 
 msgid "Fact + prefix"
@@ -446,7 +446,7 @@ msgstr ""
 msgid "List all fact values of a given discovered host"
 msgstr ""
 
-msgid "List of facts to use for the hostname (separated by comma, first wins)"
+msgid "List of facts to use for the hostname (first wins)"
 msgstr ""
 
 msgid "Location"
@@ -548,6 +548,18 @@ msgstr ""
 msgid "One or more hosts have been discovered"
 msgstr ""
 
+msgid "One or more hosts with failed discovery due to error: %s"
+msgstr ""
+
+msgid "Option 'url' must be provided"
+msgstr ""
+
+msgid "Option 'url' must be valid URI: %s"
+msgstr ""
+
+msgid "Options must be hash"
+msgstr ""
+
 msgid "Organization"
 msgstr "組織"
 

data/package.json

@@ -27,7 +27,6 @@
     "@babel/core": "^7.7.0",
     "@theforeman/builder": ">= 0",
     "@theforeman/eslint-plugin-foreman": ">= 0",
-    "@theforeman/stories": ">= 0",
     "@theforeman/test": ">= 0",
     "eslint": "^6.7.2",
     "prettier": "^1.19.1",

data/test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb

@@ -25,65 +25,51 @@ class MigrateDiscoveryHostnameAndFactColumnToArrayTest < ActiveSupport::TestCase
     end
   end
 
-  def test_discovery_hostname_string
-    Setting['discovery_hostname'] = 'discovery_bootif'
+  def setup_setting(name, value)
+    setting = Setting.find_or_create_by(name: name)
+    setting.value = value
+    setting.save(validate: false)
+  end
 
+  def test_discovery_hostname_string
+    setup_setting('discovery_hostname', 'discovery_bootif')
     migrate_up
-
     assert_equal ['discovery_bootif'], Setting['discovery_hostname']
   end
 
   def test_discovery_hostname_multistring
-    setting = Setting.find_or_create_by(name: 'discovery_hostname')
-    setting.value = 'discovery_bootif, fqdn'
-    setting.save(validate: false)
-
+    setup_setting('discovery_hostname', 'discovery_bootif, fqdn')
     migrate_up
-
     assert_equal ['discovery_bootif', 'fqdn'], Setting['discovery_hostname']
   end
 
   def test_discovery_hostname_array
     Setting['discovery_hostname'] = ['discovery_bootif']
-
     migrate_up
-
     assert_equal ['discovery_bootif'], Setting['discovery_hostname']
   end
 
   def test_discovery_fact_column_empty
-    setting = Setting.find_or_create_by(name: 'discovery_fact_column')
-    setting.value = ''
-    setting.save(validate: false)
-
+    setup_setting('discovery_fact_column', '')
     migrate_up
-
     assert_equal [], Setting['discovery_fact_column']
   end
 
   def test_discovery_fact_column_string
-    Setting['discovery_fact_column'] = 'bios_vendor'
-
+    setup_setting('discovery_fact_column', 'bios_vendor')
     migrate_up
-
     assert_equal ['bios_vendor'], Setting['discovery_fact_column']
   end
 
   def test_discovery_fact_column_multistring
-    setting = Setting.find_or_create_by(name: 'discovery_fact_column')
-    setting.value = 'bios_vendor, fqdn'
-    setting.save(validate: false)
-
+    setup_setting('discovery_fact_column', 'bios_vendor, fqdn')
     migrate_up
-
     assert_equal ['bios_vendor', 'fqdn'], Setting['discovery_fact_column']
   end
 
   def test_discovery_fact_column_array
     Setting['discovery_fact_column'] = ['bios_vendor']
-
     migrate_up
-
     assert_equal ['bios_vendor'], Setting['discovery_fact_column']
   end
 end

data/test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb.orig

@@ -0,0 +1,81 @@
+require_relative '../test_plugin_helper'
+require ForemanDiscovery::Engine.root.join('db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array')
+
+class MigrateDiscoveryHostnameAndFactColumnToArrayTest < ActiveSupport::TestCase
+  let(:migrations_paths) { ActiveRecord::Migrator.migrations_paths + [ForemanDiscovery::Engine.root.join('db/migrate/').to_s] }
+
+  let(:previous_version) { '20221102065954'.to_i }
+  let(:current_version) { '20221102075151'.to_i }
+
+  #only load the two migrations we care about (previous one and current one)
+  let(:migrations) do
+    [
+      ActiveRecord::MigrationProxy.new("FixDiscoverySettingsCategoryToDsl", previous_version, "#{ForemanDiscovery::Engine.root}/db/migrate/20221102065954_fix_discovery_settings_category_to_dsl.rb", ""),
+      ActiveRecord::MigrationProxy.new("MigrateDiscoveryHostnameAndFactColumnToArray", current_version, "#{ForemanDiscovery::Engine.root}/db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array.rb", "")
+    ]
+  end
+
+  def migrate_up
+    ActiveRecord::Migrator.new(:up, migrations, ActiveRecord::SchemaMigration, current_version).migrate
+  end
+
+  def setup
+    ActiveRecord::Migration.suppress_messages do
+      ActiveRecord::Migrator.new(:down, migrations, ActiveRecord::SchemaMigration, previous_version).migrate
+    end
+  end
+
+  def setup_setting(name, value)
+    setting = Setting.find_or_create_by(name: name)
+    setting.value = value
+    setting.save(validate: false)
+  end
+
+  def test_discovery_hostname_string
+    setup_setting('discovery_hostname', 'discovery_bootif')
+    migrate_up
+    assert_equal ['discovery_bootif'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_hostname_multistring
+    setup_setting('discovery_hostname', 'discovery_bootif, fqdn')
+    migrate_up
+    assert_equal ['discovery_bootif', 'fqdn'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_hostname_array
+    Setting['discovery_hostname'] = ['discovery_bootif']
+    migrate_up
+    assert_equal ['discovery_bootif'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_fact_column_empty
+    setup_setting('discovery_fact_column', '')
+    migrate_up
+    assert_equal [], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_string
+    setup_setting('discovery_fact_column', 'bios_vendor')
+    migrate_up
+    assert_equal ['bios_vendor'], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_multistring
+    setup_setting('discovery_fact_column', 'bios_vendor, fqdn')
+    migrate_up
+    assert_equal ['bios_vendor', 'fqdn'], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_array
+    Setting['discovery_fact_column'] = ['bios_vendor']
+<<<<<<< HEAD
+    migrate_up
+=======
+    
+    migrate_up
+    
+>>>>>>> fd3524f (Fix tests for array settings migrations)
+    assert_equal ['bios_vendor'], Setting['discovery_fact_column']
+  end
+end

data/webpack/src/ForemanDiscovery/DiscoveryRules/Components/EmptyState/EmptyState.js

@@ -17,7 +17,7 @@ const EmptyState = props => {
   };
   return (
     <ForemanEmptyState
-      header="Discovery Rules"
+      header={__('Discovery Rules')}
       description={description}
       icon="gavel"
       iconType="fa"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_discovery
 version: !ruby/object:Gem::Version
-  version: 22.0.4
+  version: 23.0.0
 platform: ruby
 authors:
 - Aditi Puntambekar
@@ -76,7 +76,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-09 00:00:00.000000000 Z
+date: 2023-11-22 00:00:00.000000000 Z
 dependencies: []
 description: MaaS Discovery Plugin engine for Foreman
 email: gsutclif@redhat.com
@@ -88,6 +88,23 @@ extra_rdoc_files:
 files:
 - LICENSE
 - README.md
+- app/assets/javascripts/foreman_discovery/locale/ca/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/cs_CZ/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/de/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/en/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/en_GB/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/es/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/fr/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/gl/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/it/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ja/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ka/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ko/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/pt_BR/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ru/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/sv_SE/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/zh_CN/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/zh_TW/foreman_discovery.js
 - app/controllers/api/v2/discovered_hosts_controller.rb
 - app/controllers/api/v2/discovery_rules_controller.rb
 - app/controllers/api/v2/fact_values_controller_extensions.rb
@@ -201,37 +218,56 @@ files:
 - lib/foreman_discovery/engine.rb
 - lib/foreman_discovery/version.rb
 - locale/ca/LC_MESSAGES/foreman_discovery.mo
+- locale/ca/foreman_discovery.edit.po
 - locale/ca/foreman_discovery.po
+- locale/cs_CZ/LC_MESSAGES/foreman_discovery.mo
+- locale/cs_CZ/foreman_discovery.edit.po
+- locale/cs_CZ/foreman_discovery.po
 - locale/de/LC_MESSAGES/foreman_discovery.mo
+- locale/de/foreman_discovery.edit.po
 - locale/de/foreman_discovery.po
 - locale/en/LC_MESSAGES/foreman_discovery.mo
+- locale/en/foreman_discovery.edit.po
 - locale/en/foreman_discovery.po
 - locale/en_GB/LC_MESSAGES/foreman_discovery.mo
+- locale/en_GB/foreman_discovery.edit.po
 - locale/en_GB/foreman_discovery.po
 - locale/es/LC_MESSAGES/foreman_discovery.mo
+- locale/es/foreman_discovery.edit.po
 - locale/es/foreman_discovery.po
 - locale/foreman_discovery.pot
 - locale/fr/LC_MESSAGES/foreman_discovery.mo
+- locale/fr/foreman_discovery.edit.po
 - locale/fr/foreman_discovery.po
 - locale/gl/LC_MESSAGES/foreman_discovery.mo
+- locale/gl/foreman_discovery.edit.po
 - locale/gl/foreman_discovery.po
 - locale/it/LC_MESSAGES/foreman_discovery.mo
+- locale/it/foreman_discovery.edit.po
 - locale/it/foreman_discovery.po
 - locale/ja/LC_MESSAGES/foreman_discovery.mo
+- locale/ja/foreman_discovery.edit.po
 - locale/ja/foreman_discovery.po
 - locale/ka/LC_MESSAGES/foreman_discovery.mo
+- locale/ka/foreman_discovery.edit.po
 - locale/ka/foreman_discovery.po
 - locale/ko/LC_MESSAGES/foreman_discovery.mo
+- locale/ko/foreman_discovery.edit.po
 - locale/ko/foreman_discovery.po
 - locale/pt_BR/LC_MESSAGES/foreman_discovery.mo
+- locale/pt_BR/foreman_discovery.edit.po
 - locale/pt_BR/foreman_discovery.po
 - locale/ru/LC_MESSAGES/foreman_discovery.mo
+- locale/ru/foreman_discovery.edit.po
 - locale/ru/foreman_discovery.po
 - locale/sv_SE/LC_MESSAGES/foreman_discovery.mo
+- locale/sv_SE/foreman_discovery.edit.po
 - locale/sv_SE/foreman_discovery.po
 - locale/zh_CN/LC_MESSAGES/foreman_discovery.mo
+- locale/zh_CN/foreman_discovery.edit.po
 - locale/zh_CN/foreman_discovery.po
 - locale/zh_TW/LC_MESSAGES/foreman_discovery.mo
+- locale/zh_TW/foreman_discovery.edit.po
 - locale/zh_TW/foreman_discovery.po
 - package.json
 - test/factories/discovery_host_related.rb
@@ -260,6 +296,7 @@ files:
 - test/functional/foreman_discovery/concerns/hosts_controller_extensions_test.rb
 - test/integration/discovered_hosts_test.rb
 - test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb
+- test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb.orig
 - test/static_fixtures/redhat_kexec.erb
 - test/test_helper_discovery.rb
 - test/test_plugin_helper.rb
@@ -318,20 +355,8 @@ signing_key:
 specification_version: 4
 summary: MaaS Discovery Plugin for Foreman
 test_files:
-- test/unit/discovered_extensions_test.rb
-- test/unit/discovered_mailer_test.rb
-- test/unit/discovery_rule_test.rb
-- test/unit/discovery_taxonomy_extensions_test.rb
-- test/unit/fact_parser_test.rb
-- test/unit/fact_to_category_resolver_test.rb
-- test/unit/lldp_neighbors_test.rb
-- test/unit/ui_notifications/destroy_host_test.rb
-- test/unit/ui_notifications/new_host_test.rb
-- test/unit/discovery_attribute_set_test.rb
-- test/unit/host_discovered_test.rb
-- test/unit/managed_extensions_test.rb
-- test/factories/discovery_host_related.rb
 - test/factories/discovery_rule_related.rb
+- test/factories/discovery_host_related.rb
 - test/facts/bond0-eth0-eth1-active-passive.json
 - test/facts/default.json
 - test/facts/dell_npars.json
@@ -351,11 +376,24 @@ test_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/discovery_rules_controller_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/integration/discovered_hosts_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/discovered_extensions_test.rb
+- test/unit/discovery_attribute_set_test.rb
+- test/unit/discovery_rule_test.rb
+- test/unit/discovery_taxonomy_extensions_test.rb
+- test/unit/fact_parser_test.rb
+- test/unit/fact_to_category_resolver_test.rb
+- test/unit/lldp_neighbors_test.rb
+- test/unit/managed_extensions_test.rb
+- test/unit/host_discovered_test.rb
 - test/test_plugin_helper.rb
+- test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb.orig
 - test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb
 - test/static_fixtures/redhat_kexec.erb
 - test/test_helper_discovery.rb