foreman_puppet 6.0.1 → 6.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9ddb41ece0ec9add7b77e958a35fe39da8ff7588a7cd6bb0900a09bbc5a9853b
-  data.tar.gz: 5054407779deb586f36b4af5ef98a21031511c3f195d8365a2525fe2a5668de8
+  metadata.gz: 68939e16cae1564207cb39add5c7fec7cac4e94c8ebf118dc61621197907a745
+  data.tar.gz: 5058a3df645b3c30aa9d0845b47c7a07286fa1c87aa261b582a6153e1533965b
 SHA512:
-  metadata.gz: c6c095009c3211f066ac1966190edf4598715c636a176ae824eccf3218f85a606c88f89672dd4cb4e6154b9b9a4fe58255e1799f214a5df8848b9e6a6656266d
-  data.tar.gz: a3711124b1bfc9659070261f9a8aaa4ba4a6de3dc2c021967f80d481b29b35a582b12153ad27cc7a7d32c887c3733ef1550edcad43ee37054640b351250ee219
+  metadata.gz: d03b3cf704fb1c44d598f2cdd07f0eb4a42bcbd5f588083b8c47ae8ac91f3c60ed809104321f4942f2adfa4601126883dc973cb9181bcd469136f94c1ccdc748
+  data.tar.gz: 10dceada6b636553d2b29ca4e7e76965e6235ba3227765e0a71ea1144d86d09e332ae2aab22c22d3554f81cef933c50a8d2979d818234ff261588b3cdd8736cd

data/app/models/concerns/foreman_puppet/extensions/host.rb

@@ -13,8 +13,6 @@ module ForemanPuppet
         include_in_clone puppet: %i[config_groups host_config_groups host_classes]
 
         scoped_search relation: :environment, on: :name, complete_value: true, rename: :environment
-        scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :class, only_explicit: true, operators: ['= ', '~ '],
-          ext_method: :search_by_deprecated_class
         scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :puppetclass, only_explicit: true, operators: ['= ', '~ '],
           ext_method: :search_by_puppetclass
         scoped_search relation: :config_groups, on: :name, complete_value: true, rename: :config_group, only_explicit: true, operators: ['= ', '~ '],
@@ -22,11 +20,6 @@ module ForemanPuppet
       end
 
       class_methods do
-        def search_by_deprecated_class(key, operator, value)
-          Foreman::Deprecation.deprecation_warning('3.2', 'search by `class` has been deprecated, please search by `puppetclass` instead')
-          search_by_puppetclass(key, operator, value)
-        end
-
         def search_by_puppetclass(_key, operator, value)
           conditions = sanitize_sql_for_conditions(["puppetclasses.name #{operator} ?", value_to_sql(operator, value)])
           config_group_ids = ForemanPuppet::ConfigGroup.joins(:puppetclasses).where(conditions).pluck(:id)

data/app/models/concerns/foreman_puppet/extensions/hostgroup.rb

@@ -16,7 +16,12 @@ module ForemanPuppet
 
         # will need through relation to work properly
         scoped_search relation: :environment, on: :name, complete_value: true, rename: :environment, only_explicit: true
-        scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :class, only_explicit: true, operators: ['= ', '~ ']
+        scoped_search relation: :puppetclasses, on: :name,
+          complete_value: true,
+          rename: :puppetclass,
+          only_explicit: true,
+          operators: ['= ', '~ '],
+          ext_method: :search_by_puppetclass
         scoped_search relation: :config_groups, on: :name,
           complete_value: true,
           rename: :config_group,
@@ -29,11 +34,22 @@ module ForemanPuppet
       module PatchedClassMethods
         def search_by_config_group(_key, operator, value)
           conditions = sanitize_sql_for_conditions(["config_groups.name #{operator} ?", value_to_sql(operator, value)])
-          hostgroup_ids = ::Hostgroup.unscoped.with_taxonomy_scope.joins(puppet: :config_groups).where(conditions).map(&:subtree_ids).flatten.uniq
+          hostgroup_ids = ::Hostgroup.joins(puppet: :config_groups).where(conditions).map(&:subtree_ids).flatten.uniq
+
+          conds = 'hostgroups.id < 0'
+          conds = "hostgroups.id IN(#{hostgroup_ids.join(',')})" if hostgroup_ids.present?
+
+          { conditions: conds }
+        end
+
+        def search_by_puppetclass(_key, operator, value)
+          conditions = sanitize_sql_for_conditions(["puppetclasses.name #{operator} ?", value_to_sql(operator, value)])
+          hostgroup_ids = ::Hostgroup.joins(puppet: :puppetclasses).where(conditions).map(&:subtree_ids)
+
+          conds = []
+          conds << "hostgroups.id IN (#{hostgroup_ids.join(',')})" if hostgroup_ids.present?
 
-          opts = 'hostgroups.id < 0'
-          opts = "hostgroups.id IN(#{hostgroup_ids.join(',')})" if hostgroup_ids.present?
-          { conditions: opts }
+          { conditions: conds.join(' OR ').presence || 'hostgroups.id < 0' }
         end
       end
     end

data/app/models/foreman_puppet/config_group.rb

@@ -17,7 +17,7 @@ module ForemanPuppet
     validates :name, presence: true, uniqueness: true
 
     scoped_search on: :name, complete_value: true
-    scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :class, only_explicit: true, operators: ['= ', '~ ']
+    scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :puppetclass, only_explicit: true, operators: ['= ', '~ ']
 
     default_scope -> { order('config_groups.name') }
 

data/lib/foreman_puppet/version.rb

@@ -1,3 +1,3 @@
 module ForemanPuppet
-  VERSION = '6.0.1'.freeze
+  VERSION = '6.1.0'.freeze
 end

data/test/models/foreman_puppet/host_test.rb

@@ -48,23 +48,31 @@ module ForemanPuppet
         assert_equal results, results2
       end
 
+      test 'deprecated search by class throws error' do
+        exception = assert_raises(Exception) do
+          host
+          Host.search_for("class = #{host.puppet.puppetclass_names.first}")
+        end
+        assert_equal("Field 'class' not recognized for searching!", exception.message)
+      end
+
       test 'can be found by puppetclass' do
         host
-        result = Host.search_for("class = #{host.puppet.puppetclass_names.first}")
+        result = Host.search_for("puppetclass = #{host.puppet.puppetclass_names.first}")
         assert_includes result, host
       end
 
       test 'search by puppetclass returns only host within that puppetclass' do
         host
         puppetclass = FactoryBot.create(:puppetclass)
-        result = Host.search_for("class = #{puppetclass.name}")
+        result = Host.search_for("puppetclass = #{puppetclass.name}")
         assert_not_includes result, host
       end
 
       test 'search hosts by inherited puppetclass from a hostgroup' do
         hostgroup = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_puppetclass)
         host_with_hg = FactoryBot.create(:host, hostgroup: hostgroup)
-        result = Host.search_for("class = #{hostgroup.puppet.puppetclass_names.first}")
+        result = Host.search_for("puppetclass = #{hostgroup.puppet.puppetclass_names.first}")
         assert_includes result, host_with_hg
       end
 
@@ -72,7 +80,7 @@ module ForemanPuppet
         parent_hg = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_puppetclass)
         hg = FactoryBot.create(:hostgroup, parent: parent_hg)
         host = FactoryBot.create(:host, hostgroup: hg)
-        result = Host.search_for("class = #{parent_hg.puppet.puppetclass_names.first}")
+        result = Host.search_for("puppetclass = #{parent_hg.puppet.puppetclass_names.first}")
         assert_equal 1, result.count
         assert_includes result, host
       end
@@ -82,7 +90,7 @@ module ForemanPuppet
         host = FactoryBot.create(:host, :with_puppet_enc, hostgroup: hostgroup, environment: hostgroup.puppet.environment)
         config_group = FactoryBot.create(:config_group, :with_puppetclass)
         hostgroup.puppet.config_groups << config_group
-        result = Host.search_for("class = #{config_group.puppetclass_names.first}")
+        result = Host.search_for("puppetclass = #{config_group.puppetclass_names.first}")
         assert_equal 1, result.count
         assert_includes result, host
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_puppet
 version: !ruby/object:Gem::Version
-  version: 6.0.1
+  version: 6.1.0
 platform: ruby
 authors:
 - Ondřej Ezr
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-10 00:00:00.000000000 Z
+date: 2023-09-14 00:00:00.000000000 Z
 dependencies: []
 description: Allow assigning Puppet environments and classes to the Foreman Hosts.
 email: