foreman_puppet 1.0.5 → 2.0.0.alpha.2

data/test/helpers/foreman_puppet/hosts_and_hostgroups_helper_test.rb

@@ -9,88 +9,49 @@ module ForemanPuppet
     include HostsAndHostgroupsHelper
 
     describe 'puppet environment field' do
-      let(:fields) { mock('fields_for') }
-      let(:f) { mock('form_for') }
-      let(:object) { mock('object') }
-      let(:puppet_facet) { mock('puppet') }
-
       setup do
-        f.stubs(:object).returns(object)
-        f.stubs(:fields_for).yields(fields)
-        fields.stubs(:object).returns(puppet_facet)
+        @host = mock('host')
+        puppet = mock('puppet')
+        @host.stubs(:hostgroup)
+        @host.stubs(:puppet).returns(puppet)
+        @host.stubs(:id).returns(999)
+        @f = mock('f')
+        @f.stubs(:object).returns(@host)
+        @fields = mock('fields')
+        @fields.stubs(:object).returns(puppet)
+        @f.stubs(:fields_for).yields(@fields)
       end
 
-      describe '#host_puppet_environment_field' do
-        let(:object) do
-          host = mock('host')
-          host.stubs(:hostgroup)
-          host.stubs(:puppet).returns(puppet_facet)
-          host.stubs(:id).returns(999)
-          host
-        end
-
-        test 'it adds new first level attributes' do
-          fields.expects(:collection_select).with do |*attrs|
-            select_options, html_options = extract_collection_options(attrs)
-            select_options[:test_select_option] == 'test_value1' &&
-              html_options[:test_html_option] == 'test_value2'
-          end
-
-          host_puppet_environment_field(f, { test_select_option: 'test_value1' }, { test_html_option: 'test_value2' })
+      test 'it adds new first level attributes' do
+        @fields.expects(:collection_select).with do |*attrs|
+          select_options, html_options = extract_collection_options(attrs)
+          select_options[:test_select_option] == 'test_value1' &&
+            html_options[:test_html_option] == 'test_value2'
         end
 
-        test 'it adds new data attributes' do
-          fields.expects(:collection_select).with do |*attrs|
-            select_options, html_options = extract_collection_options(attrs)
-            select_options[:test_select_option] == 'test_value1' &&
-              html_options[:data][:test] == 'test_value2'
-          end
+        host_puppet_environment_field(@f, { test_select_option: 'test_value1' }, { test_html_option: 'test_value2' })
+      end
 
-          host_puppet_environment_field(f, { test_select_option: 'test_value1' }, { data: { test: 'test_value2' } })
+      test 'it adds new data attributes' do
+        @fields.expects(:collection_select).with do |*attrs|
+          select_options, html_options = extract_collection_options(attrs)
+          select_options[:test_select_option] == 'test_value1' &&
+            html_options[:data][:test] == 'test_value2'
         end
 
-        test 'it overrides existing attributes' do
-          fields.expects(:collection_select).with do |*attrs|
-            html_options = attrs.pop
-            html_options[:data][:test] == 'some_test_value' &&
-              html_options[:data][:url] == '/test/url'
-          end.returns('')
-
-          html = host_puppet_environment_field(f, { disable_button: false }, { data: { url: '/test/url', test: 'some_test_value' } })
-
-          assert_no_match(/btn/, html)
-        end
+        host_puppet_environment_field(@f, { test_select_option: 'test_value1' }, { data: { test: 'test_value2' } })
       end
 
-      describe '#hostgroup_puppet_environment_field' do
-        let(:object) { FactoryBot.build_stubbed(:hostgroup, parent: parent_hg) }
-        let(:puppet_facet) { object.puppet || object.build_puppet }
-
-        context 'parent without puppet' do
-          let(:parent_hg) { FactoryBot.create(:hostgroup) }
+      test 'it overrides existing attributes' do
+        @fields.expects(:collection_select).with do |*attrs|
+          html_options = attrs.pop
+          html_options[:data][:test] == 'some_test_value' &&
+            html_options[:data][:url] == '/test/url'
+        end.returns('')
 
-          it 'shows Inherit option with no value' do
-            fields.expects(:collection_select).with do |*attrs|
-              options = attrs.second
-              options.first.to_label == 'Inherit parent (no value)'
-            end.returns('')
-
-            hostgroup_puppet_environment_field(f)
-          end
-        end
+        html = host_puppet_environment_field(@f, { disable_button: false }, { data: { url: '/test/url', test: 'some_test_value' } })
 
-        context 'parent with puppet' do
-          let(:parent_hg) { FactoryBot.create(:hostgroup, :with_puppet_enc) }
-
-          it 'shows Inherit option with no value' do
-            fields.expects(:collection_select).with do |*attrs|
-              options = attrs.second
-              options.first.to_label == "Inherit parent (#{parent_hg.puppet.environment.name})"
-            end.returns('')
-
-            hostgroup_puppet_environment_field(f)
-          end
-        end
+        assert_no_match(/btn/, html)
       end
     end
 

data/test/integration/foreman_puppet/host_js_test.rb

@@ -362,7 +362,6 @@ module ForemanPuppet
       page.find('#puppet_klasses_parameters_table')
     end
 
-    # TODO: unless ForemanPuppet.extracted_from_core?
     def switch_form_tab_to_interfaces
       switch_form_tab('Interfaces')
       disable_interface_modal_animation

data/test/integration/foreman_puppet/puppetclass_js_test.rb

@@ -11,7 +11,7 @@ module ForemanPuppet
     let(:puppetclass) { FactoryBot.create(:puppetclass, environments: [environment]) }
 
     test 'edit page' do
-      FactoryBot.create(:puppetclass, name: 'vim', environments: [environment]) if ForemanPuppet.extracted_from_core?
+      FactoryBot.create(:puppetclass, name: 'vim', environments: [environment])
       visit puppetclasses_path
       click_link 'vim'
       assert page.has_no_link? 'Common'

data/test/models/foreman_puppet/host_puppet_facet_test.rb

@@ -107,13 +107,7 @@ module ForemanPuppet
       test 'should return empty array if host has no hostgroup' do
         host = FactoryBot.create(:host, :with_puppet_enc)
         assert_not host.hostgroup
-        assert_equal [], host.puppet.parent_config_groups
-      end
-
-      test 'should return empty array if hostgroup do not have puppet data' do
-        hostgroup = FactoryBot.create(:hostgroup)
-        host = FactoryBot.create(:host, :with_puppet_enc, hostgroup: hostgroup)
-        assert_equal [], host.puppet.parent_config_groups
+        assert_empty host.puppet.parent_config_groups
       end
     end
 

data/test/models/foreman_puppet/host_test.rb

@@ -150,6 +150,32 @@ module ForemanPuppet
       end
     end
 
+    describe '#available_template_kinds' do
+      let(:template_kinds) { [stub(name: 'iPXE'), stub(name: 'finish')] }
+
+      test 'calls find_template with Puppet environment' do
+        host = FactoryBot.create(:host, :with_hostgroup, :with_puppet_enc)
+        host.expects(:template_kinds).returns(template_kinds)
+        template_kinds.each do |kind|
+          ::ProvisioningTemplate.expects(:find_template)
+                                .with(kind: kind.name, operatingsystem_id: host.operatingsystem.id, hostgroup_id: host.hostgroup.id, environment_id: host.puppet.environment.id)
+                                .returns(stub(name: "default #{kind.name}"))
+        end
+        assert_equal template_kinds.map { |k| "default #{k.name}" }, host.available_template_kinds.map(&:name)
+      end
+
+      test 'calls find_template without Puppet environment' do
+        host = FactoryBot.create(:host, :with_hostgroup)
+        host.expects(:template_kinds).returns(template_kinds)
+        template_kinds.each do |kind|
+          ::ProvisioningTemplate.expects(:find_template)
+                                .with(kind: kind.name, operatingsystem_id: host.operatingsystem.id, hostgroup_id: host.hostgroup.id, environment_id: nil)
+                                .returns(stub(name: "default #{kind.name}"))
+        end
+        assert_equal template_kinds.map { |k| "default #{k.name}" }, host.available_template_kinds.map(&:name)
+      end
+    end
+
     test 'should import from external nodes output' do
       # create a dummy node
       Parameter.destroy_all

data/test/models/foreman_puppet/user_test.rb

@@ -5,11 +5,7 @@ module ForemanPuppet
     describe '#visible_environments' do
       let(:environment) { FactoryBot.create(:environment) }
       let(:untaxed_env) { FactoryBot.create(:environment, organizations: [], locations: []) }
-      let(:env_names) do
-        env_names = [environment.name, untaxed_env.name]
-        env_names += %w[production global_puppetmaster testing] unless ForemanPuppet.extracted_from_core?
-        env_names
-      end
+      let(:env_names) { [environment.name, untaxed_env.name] }
 
       setup do
         environment

data/test/unit/foreman_puppet/puppet_class_importer_test.rb

@@ -3,7 +3,7 @@ require 'test_puppet_helper'
 module ForemanPuppet
   class PuppetClassImporterTest < ActiveSupport::TestCase
     def setup
-      FactoryBot.create(:environment, name: 'production') if ForemanPuppet.extracted_from_core?
+      FactoryBot.create(:environment, name: 'production')
       ProxyAPI::Puppet.any_instance.stubs(:environments).returns(%w[foreman-testing foreman-testing-1])
       ProxyAPI::Puppet.any_instance.stubs(:classes).returns(mocked_classes)
       User.current = users(:admin)

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: foreman_puppet
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 2.0.0.alpha.2
 platform: ruby
 authors:
 - Ondřej Ezr
 - Shira Maximov
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-11-15 00:00:00.000000000 Z
+date: 2021-10-14 00:00:00.000000000 Z
 dependencies: []
-description: Allow assigning Puppet environmets and classes to the Foreman Hosts.
+description: Allow assigning Puppet environments and classes to the Foreman Hosts.
 email:
 - foreman-dev@googlegroups.com
 executables: []
@@ -22,10 +22,10 @@ files:
 - README.md
 - Rakefile
 - app/assets/stylesheets/foreman_puppet.scss
-- app/controllers/concerns/foreman_puppet/api/import_puppetclasses_common_controller.rb
 - app/controllers/concerns/foreman_puppet/environments_import.rb
 - app/controllers/concerns/foreman_puppet/extensions/api_base_controller.rb
 - app/controllers/concerns/foreman_puppet/extensions/api_hostgroups_controller.rb
+- app/controllers/concerns/foreman_puppet/extensions/api_smart_proxies_controller.rb
 - app/controllers/concerns/foreman_puppet/extensions/api_template_combinations_controller.rb
 - app/controllers/concerns/foreman_puppet/extensions/api_v2_hosts_controller.rb
 - app/controllers/concerns/foreman_puppet/extensions/hostgroups_controller_extensions.rb
@@ -113,9 +113,11 @@ files:
 - app/views/foreman_puppet/api/v2/environments/show.json.rabl
 - app/views/foreman_puppet/api/v2/environments/update.json.rabl
 - app/views/foreman_puppet/api/v2/host_puppet_facets/base.json.rabl
+- app/views/foreman_puppet/api/v2/host_puppet_facets/host_list.json.rabl
 - app/views/foreman_puppet/api/v2/host_puppet_facets/host_single.json.rabl
 - app/views/foreman_puppet/api/v2/host_puppet_facets/show.json.rabl
 - app/views/foreman_puppet/api/v2/hostgroup_puppet_facets/base.json.rabl
+- app/views/foreman_puppet/api/v2/hostgroup_puppet_facets/hostgroup_list.json.rabl
 - app/views/foreman_puppet/api/v2/hostgroup_puppet_facets/hostgroup_single.json.rabl
 - app/views/foreman_puppet/api/v2/hostgroup_puppet_facets/show.json.rabl
 - app/views/foreman_puppet/api/v2/import_puppetclasses/index.json.rabl
@@ -183,15 +185,22 @@ files:
 - db/migrate/20200803113803_migrate_environment_to_puppet_facet.foreman_puppet.rb
 - db/migrate/20200803113903_migrate_host_type_in_host_config_groups.foreman_puppet.rb
 - db/migrate/20201125113903_migrate_puppetclasses_to_facets.foreman_puppet.rb
-- db/migrate/20211112130803_cleanup_environment_from_core_tables.foreman_puppet.rb
+- db/migrate/20210924103241_remove_katello_id_from_environments.foreman_puppet.rb
 - db/migrate_foreman/20090722141107_create_environments.rb
+- db/migrate_foreman/20090802062223_create_puppetclasses.rb
+- db/migrate_foreman/20090905150132_create_hostgroups_puppetclasses.rb
+- db/migrate_foreman/20110412103238_remove_unused_fields_from_puppet_classes.rb
+- db/migrate_foreman/20110712070522_create_host_class.rb
+- db/migrate_foreman/20120824142048_add_some_indexes.rb
 - db/migrate_foreman/20120905095532_create_environment_classes.rb
+- db/migrate_foreman/20121018152459_create_hostgroup_classes.rb
 - db/migrate_foreman/20140407161817_create_config_groups.rb
 - db/migrate_foreman/20140407162007_create_config_group_classes.rb
 - db/migrate_foreman/20140407162059_create_host_config_groups.rb
 - db/migrate_foreman/20161205142618_delete_orphaned_smart_class_parameters.rb
 - db/migrate_foreman/20170109115157_fix_lookup_key_auditable_type.rb
 - db/migrate_foreman/20180816134832_cast_lookup_key_values.rb
+- db/migrate_foreman/20180831115634_add_uniqueness_to_puppetclass_name.rb
 - db/migrate_foreman/20181023112532_add_environment_puppetclass_id.rb
 - db/migrate_foreman/20181224174419_add_index_to_environment_class_by_lookup_key_and_puppetclass.rb
 - db/seeds.d/111_puppet_proxy_feature.rb
@@ -202,8 +211,6 @@ files:
 - lib/tasks/foreman_puppet_tasks.rake
 - locale/Makefile
 - locale/action_names.rb
-- locale/cs_CZ/foreman_puppet.edit.po
-- locale/cs_CZ/foreman_puppet.po.time_stamp
 - locale/en/LC_MESSAGES/foreman_puppet.mo
 - locale/en/foreman_puppet.edit.po
 - locale/en/foreman_puppet.po
@@ -224,6 +231,7 @@ files:
 - test/controllers/foreman_puppet/api/v2/provisioning_templates_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/puppetclasses_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/smart_class_parameters_controller_test.rb
+- test/controllers/foreman_puppet/api/v2/smart_proxies_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/template_combinations_controller_test.rb
 - test/controllers/foreman_puppet/config_groups_controller_test.rb
 - test/controllers/foreman_puppet/environments_controller_test.rb
@@ -301,7 +309,7 @@ homepage: https://github.com/theforeman/foreman_puppet
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -312,14 +320,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.1.6
-signing_key:
+signing_key: 
 specification_version: 4
-summary: Adds puppet ENC features
+summary: Add Puppet features to Foreman
 test_files:
 - test/services/foreman_puppet/host_info_providers/config_groups_info_test.rb
 - test/services/foreman_puppet/host_info_providers/puppet_info_test.rb
@@ -379,6 +387,7 @@ test_files:
 - test/controllers/foreman_puppet/api/v2/lookups_common_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/host_classes_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/provisioning_templates_controller_test.rb
+- test/controllers/foreman_puppet/api/v2/smart_proxies_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/puppetclasses_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/hosts_controller_test.rb
 - test/controllers/foreman_puppet/puppetclasses_controller_test.rb

data/db/migrate/20211112130803_cleanup_environment_from_core_tables.foreman_puppet.rb

@@ -1,10 +0,0 @@
-class CleanupEnvironmentFromCoreTables < ActiveRecord::Migration[6.0]
-  def up
-    ::Hostgroup.update_all(environment_id: nil) if column_exists?(:hostgroups, :environment_id)
-    Host::Managed.update_all(environment_id: nil) if column_exists?(:hosts, :environment_id)
-  end
-
-  def down
-    # nothing to do
-  end
-end