hammer_cli_foreman_puppet 0.0.1

data/test/unit/helpers/fake_searchables.rb

@@ -0,0 +1,19 @@
+
+
+class FakeSearchables
+
+  def initialize(searchable_fields, editable_fields = [])
+    @searchables = []
+    @searchables += searchable_fields.collect do |name|
+      HammerCLIForeman::Searchable.new(name, "Search by #{name}", :editable => false)
+    end
+    @searchables += editable_fields.collect do |name|
+      HammerCLIForeman::Searchable.new(name, "Search by #{name}", :editable => true)
+    end
+  end
+
+  def for(resource)
+    @searchables
+  end
+
+end

data/test/unit/helpers/resource_disabled.rb

@@ -0,0 +1,24 @@
+require File.join(File.dirname(__FILE__), '../apipie_resource_mock')
+
+module ResourceDisabled
+
+  def it_should_fail_when_disabled
+    arguments = @with_params ? @with_params.dup : []
+    context "resource disabled" do
+
+      it "should return error" do
+        cmd.class.resource.stubs(:call).raises(RestClient::ResourceNotFound)
+        arguments = respond_to?(:with_params) ? with_params : []
+        _(cmd.run(arguments)).must_equal HammerCLI::EX_UNAVAILABLE
+      end
+
+      it "should print error message" do
+        cmd.class.resource.stubs(:call).raises(RestClient::ResourceNotFound)
+        cmd.stubs(:context).returns(ctx.update(:adapter => :test))
+
+        arguments = respond_to?(:with_params) ? with_params : []
+        _(lambda { cmd.run(arguments) }).must_output "", /.*not support.*/
+      end
+    end
+  end
+end

data/test/unit/puppet_class_test.rb

@@ -0,0 +1,72 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman_puppet/class'
+
+describe HammerCLIForemanPuppet::PuppetClass do
+
+  include CommandTestHelper
+
+  context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:puppetclasses, :index, {})
+    end
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetClass::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      # FIXME: the command should accept search parameters in future
+      # it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) do
+        # data are retuned in specific format
+        HammerCLIForemanPuppet.collection_to_common_format(cmd.resource.call(:index)).first.keys.count
+      end
+
+      it_should_print_n_records
+      it_should_print_column "Id"
+      it_should_print_column "Name"
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetClass::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_column "Id"
+        it_should_print_column "Name"
+      end
+    end
+
+  end
+
+  context "SCParamsCommand" do
+
+    before :each do
+      ResourceMocks.smart_class_parameters_index
+    end
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetClass::SCParamsCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "puppet-class", ["--puppet-class=cls"]
+      it_should_accept "puppet-class-id", ["--puppet-class-id=1"]
+      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+end

data/test/unit/puppet_environment_test.rb

@@ -0,0 +1,116 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman_puppet/environment'
+
+describe HammerCLIForemanPuppet::PuppetEnvironment do
+
+  include CommandTestHelper
+
+  context "ListCommand" do
+    before do
+      ResourceMocks.mock_action_call(:environments, :index, [])
+    end
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetEnvironment::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_column "Name"
+      it_should_print_column "Id"
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetEnvironment::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=env"]
+      # it_should_fail_with "no arguments"
+      # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_column "Name"
+        it_should_print_column "Id"
+        it_should_print_column "Created at"
+        it_should_print_column "Updated at"
+      end
+    end
+
+  end
+
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetEnvironment::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=env"]
+      # it_should_fail_with "name missing", []
+      # TODO: temporarily disabled, parameters are checked by the api
+    end
+
+  end
+
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetEnvironment::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=env"]
+      it_should_accept "id", ["--id=1"]
+      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetEnvironment::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=env", "--new-name=env2"]
+      it_should_accept "id", ["--id=1", "--new-name=env2"]
+      # it_should_fail_with "no params", [] # TODO: temporarily disabled, parameters are checked in the id resolver
+      # it_should_fail_with "name or id missing", ["--new-name=env2"] # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+  context "SCParamsCommand" do
+
+    before :each do
+      ResourceMocks.smart_class_parameters_index
+    end
+
+    let(:cmd) { HammerCLIForemanPuppet::PuppetEnvironment::SCParamsCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_fail_with "environment", ["--environment=env"]
+      it_should_accept "puppet environment", ["--puppet-environment=env"]
+      it_should_fail_with "environment-id", ["--environment-id=1"]
+      it_should_accept "puppet-environment-id", ["--puppet-environment-id=1"]
+      it_should_accept "puppet-environment", ["--puppet-environment=env"]
+      it_should_accept "puppet-environment-id", ["--puppet-environment-id=1"]
+      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+end

data/test/unit/smart_class_parameter_test.rb

@@ -0,0 +1,114 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+require 'hammer_cli_foreman_puppet/smart_class_parameter'
+
+describe HammerCLIForemanPuppet::SmartClassParameter do
+
+  include CommandTestHelper
+
+  context "ListCommand" do
+
+    before :each do
+      ResourceMocks.smart_class_parameters_index
+    end
+
+    let(:cmd) { HammerCLIForemanPuppet::SmartClassParameter::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept "hostgroup id", ["--hostgroup-id=1"]
+      it_should_accept "host id", ["--host-id=1"]
+      it_should_fail_with "environment id", ["--environment-id=1"]
+      it_should_accept "puppet environment id", ["--puppet-environment-id=1"]
+      it_should_accept "puppet-class-id", ["--puppet-class-id=1"]
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
+
+      it_should_print_n_records
+      it_should_print_column "Id"
+      it_should_print_column "Class Id"
+      it_should_print_column "Puppet class"
+      it_should_print_column "Parameter"
+      it_should_print_column "Default Value"
+      it_should_print_column "Override"
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    before :each do
+      ResourceMocks.smart_class_parameters_show
+    end
+
+    let(:cmd) { HammerCLIForemanPuppet::SmartClassParameter::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name, puppet-class", ["--name=par", "--puppet-class=ntp"]
+      it_should_fail_with "name", ["--name=par"]
+      # it_should_fail_with "no arguments"
+      # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_columns ["Description", "Type", "Omit", "Required"]
+      end
+    end
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::SmartClassParameter::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name, puppet-class", ["--name=par", "--puppet-class=ntp"]
+      it_should_fail_with "name", ["--name=par"]
+      it_should_accept "override", ["--id=1","--override=true"]
+      it_should_accept "description", ["--id=1","--description=descr"]
+      it_should_accept "default-value", ["--id=1","--default-value=1"]
+      it_should_accept "path  ", ["--id=1","--path=path"]
+      it_should_accept "validator-type", ["--id=1","--validator-type=list"]
+      it_should_accept "validator-rule ", ["--id=1","--validator-rule=''"]
+      it_should_accept "override-value-order", ["--id=1","--override-value-order=fqdn"]
+      it_should_accept "parameter-type ", ["--id=1","--parameter-type=string"]
+      it_should_accept "required", ["--id=1","--required=true"]
+
+      # it_should_fail_with "no params", []
+      # TODO: temporarily disabled, parameters are checked in the id resolver
+    end
+
+  end
+
+  context "AddMatcherCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::SmartClassParameter::AddMatcherCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "smart-class-parametr-id, match, value", ["--smart-class-parameter-id=1", "--match='domain=my.lan'", "--value=1"]
+      it_should_accept "smart-class-parameter, puppet-class, match, value", ["--smart-class-parameter=par", "--puppet-class=ntp", "--match='domain=my.lan'", "--value=1"]
+      it_should_fail_with "smart-class-parameter, match, value", ["--smart-class-parameter=par", "--match='domain=my.lan'", "--value=1"]
+    end
+  end
+
+  context "RemoveMatcherCommand" do
+
+    let(:cmd) { HammerCLIForemanPuppet::SmartClassParameter::RemoveMatcherCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "smart-class-parametr-id, id", ["--smart-class-parameter-id=1", "--id=1"]
+      it_should_accept "smart-class-parameter, puppet-class, id", ["--smart-class-parameter=par", "--puppet-class=ntp", "--id=1"]
+      it_should_fail_with "smart-class-parameter, id", ["--smart-class-parameter=par", "--id=1"]
+    end
+  end
+
+end

data/test/unit/test_helper.rb

@@ -0,0 +1,18 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+def ctx
+  {
+    :adapter => :silent,
+    :username => 'admin',
+    :password => 'admin',
+    :interactive => false
+  }
+end
+
+
+require File.join(File.dirname(__FILE__), 'test_output_adapter')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+require File.join(File.dirname(__FILE__), 'helpers/command')
+require File.join(File.dirname(__FILE__), 'helpers/resource_disabled')
+
+HammerCLI::Settings.load({:_params => {:interactive => false}})

data/test/unit/test_output_adapter.rb

@@ -0,0 +1,22 @@
+require 'hammer_cli/output/adapter/abstract'
+
+
+class TestAdapter < HammerCLI::Output::Adapter::Abstract
+
+  def print_record(fields, record)
+    print_collection(fields, [record].flatten(1))
+  end
+
+
+  def print_collection(fields, data, _options = {})
+    @separator = '#'
+    puts @separator+fields.collect{|f| f.label.to_s}.join(@separator)+@separator
+
+    data.collect do |d|
+      puts @separator+fields.collect{ |f| data_for_field(f, d).to_s }.join(@separator)+@separator
+    end
+  end
+
+end
+
+HammerCLI::Output::Output.register_adapter(:test, TestAdapter)

metadata

@@ -0,0 +1,143 @@
+--- !ruby/object:Gem::Specification
+name: hammer_cli_foreman_puppet
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+platform: ruby
+authors:
+- Amir Fefer
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2021-08-02 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: hammer_cli_foreman
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 4.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 4.0.0
+description: 
+email:
+- amirfefer@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- LICENSE
+- README.md
+- config/foreman_puppet.yml
+- lib/hammer_cli_foreman_puppet.rb
+- lib/hammer_cli_foreman_puppet/associating_commands.rb
+- lib/hammer_cli_foreman_puppet/associating_commands/associating_commands.rb
+- lib/hammer_cli_foreman_puppet/class.rb
+- lib/hammer_cli_foreman_puppet/combination.rb
+- lib/hammer_cli_foreman_puppet/command_extensions.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/discovery.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/environment.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/environments.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/host.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/hostgroup.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/location.rb
+- lib/hammer_cli_foreman_puppet/command_extensions/organization.rb
+- lib/hammer_cli_foreman_puppet/commands.rb
+- lib/hammer_cli_foreman_puppet/config_group.rb
+- lib/hammer_cli_foreman_puppet/discovery.rb
+- lib/hammer_cli_foreman_puppet/environment.rb
+- lib/hammer_cli_foreman_puppet/environment_name_mapping.rb
+- lib/hammer_cli_foreman_puppet/host.rb
+- lib/hammer_cli_foreman_puppet/hostgroup.rb
+- lib/hammer_cli_foreman_puppet/id_resolver.rb
+- lib/hammer_cli_foreman_puppet/location.rb
+- lib/hammer_cli_foreman_puppet/option_sources.rb
+- lib/hammer_cli_foreman_puppet/option_sources/puppet_environment_params.rb
+- lib/hammer_cli_foreman_puppet/organization.rb
+- lib/hammer_cli_foreman_puppet/puppet_references.rb
+- lib/hammer_cli_foreman_puppet/references.rb
+- lib/hammer_cli_foreman_puppet/smart_class_parameter.rb
+- lib/hammer_cli_foreman_puppet/smart_proxy.rb
+- lib/hammer_cli_foreman_puppet/version.rb
+- test/data/2.1/foreman_api.json
+- test/data/3.0/foreman_api.json
+- test/data/README.md
+- test/functional/config_group_test.rb
+- test/functional/host/create_test.rb
+- test/functional/host/update_test.rb
+- test/functional/hostgroup/create_test.rb
+- test/functional/hostgroup/update_test.rb
+- test/functional/proxy_test.rb
+- test/functional/smart_class_parameter_test.rb
+- test/functional/template_test.rb
+- test/functional/test_helper.rb
+- test/test_helper.rb
+- test/unit/apipie_resource_mock.rb
+- test/unit/config_group_test.rb
+- test/unit/helpers/command.rb
+- test/unit/helpers/fake_searchables.rb
+- test/unit/helpers/resource_disabled.rb
+- test/unit/puppet_class_test.rb
+- test/unit/puppet_environment_test.rb
+- test/unit/smart_class_parameter_test.rb
+- test/unit/test_helper.rb
+- test/unit/test_output_adapter.rb
+homepage: https://github.com/theforeman/hammer-cli-foreman-puppet
+licenses:
+- GPL-3.0
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.7.6.2
+signing_key: 
+specification_version: 4
+summary: Foreman Puppet plugin for Hammer CLI
+test_files:
+- test/data/2.1/foreman_api.json
+- test/data/3.0/foreman_api.json
+- test/data/README.md
+- test/test_helper.rb
+- test/unit/apipie_resource_mock.rb
+- test/unit/helpers/command.rb
+- test/unit/helpers/fake_searchables.rb
+- test/unit/helpers/resource_disabled.rb
+- test/unit/puppet_class_test.rb
+- test/unit/puppet_environment_test.rb
+- test/unit/smart_class_parameter_test.rb
+- test/unit/test_helper.rb
+- test/unit/test_output_adapter.rb
+- test/unit/config_group_test.rb
+- test/functional/config_group_test.rb
+- test/functional/host/create_test.rb
+- test/functional/host/update_test.rb
+- test/functional/hostgroup/create_test.rb
+- test/functional/hostgroup/update_test.rb
+- test/functional/proxy_test.rb
+- test/functional/smart_class_parameter_test.rb
+- test/functional/template_test.rb
+- test/functional/test_helper.rb