hammer_cli_foreman_puppet 0.0.6 → 0.1.0

data/test/unit/helpers/fake_searchables.rb

@@ -1,19 +0,0 @@
-
-
-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

@@ -1,24 +0,0 @@
-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

@@ -1,72 +0,0 @@
-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

@@ -1,116 +0,0 @@
-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

@@ -1,114 +0,0 @@
-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

@@ -1,18 +0,0 @@
-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

@@ -1,22 +0,0 @@
-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)