hammer_cli_foreman 0.0.10 → 0.0.11

data/test/unit/exception_handler_test.rb

@@ -0,0 +1,48 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require 'hammer_cli_foreman/exception_handler'
+
+describe HammerCLIForeman::ExceptionHandler do
+
+  let(:output)  { HammerCLI::Output::Output.new }
+  let(:handler) { HammerCLIForeman::ExceptionHandler.new(:output => output) }
+  let(:heading) { "Something went wrong" }
+
+  it "should print resource errors on unprocessable entity exception" do
+   response = <<-RESPONSE
+   {"subnet":{"id":null,"errors":{"network":["can't be blank","is invalid"],"name":["can't be blank"]},"full_messages":["Network address can't be blank","Network address is invalid","Name can't be blank"]}}
+   RESPONSE
+
+    ex = RestClient::UnprocessableEntity.new(response)
+    output.expects(:print_error).with(heading, "Network address can't be blank\nNetwork address is invalid\nName can't be blank")
+    err_code = handler.handle_exception(ex, :heading => heading)
+    err_code.must_equal HammerCLI::EX_DATAERR
+  end
+
+  it "should handle argument error" do
+    ex = ArgumentError.new
+    output.expects(:print_error).with(heading, ex.message)
+    err_code = handler.handle_exception(ex, :heading => heading)
+    err_code.must_equal HammerCLI::EX_USAGE
+  end
+
+  it "should handle forbidden error" do
+    ex = RestClient::Forbidden.new
+    output.expects(:print_error).with('Forbidden - server refused to process the request', nil)
+    err_code = handler.handle_exception(ex)
+    err_code.must_equal HammerCLI::EX_NOPERM
+  end
+
+  it "should handle unknown exception" do
+    output.expects(:print_error).with(heading, "Error: message")
+    MyException = Class.new(Exception)
+    err_code = handler.handle_exception(MyException.new('message'), :heading => heading)
+    err_code.must_equal HammerCLI::EX_SOFTWARE
+  end
+
+  it "should handle unsupported operation error" do
+    output.expects(:print_error).with(heading, "message")
+    err_code = handler.handle_exception(HammerCLIForeman::OperationNotSupportedError.new('message'), :heading => heading)
+    err_code.must_equal HammerCLI::EX_UNAVAILABLE
+  end
+end
+

data/test/unit/fact_test.rb

@@ -0,0 +1,31 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::Fact do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ApipieResourceMock.new(cmd.class.resource.resource_class)
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Fact::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      it_should_print_n_records 2
+      it_should_print_column "Host"
+      it_should_print_column "Fact"
+      it_should_print_column "Value"
+    end
+
+  end
+
+end

data/test/unit/helpers/command.rb

@@ -0,0 +1,69 @@
+require File.join(File.dirname(__FILE__), '../test_output_adapter')
+
+
+module CommandTestHelper
+
+  def with_params(params, &block)
+    context "with params "+params.to_s do
+      let(:with_params) { params }
+      self.instance_eval &block
+    end
+  end
+
+  def it_should_call_action(action, params)
+    it "should call action "+action.to_s do
+      arguments ||= respond_to?(:with_params) ? with_params : []
+      cmd.resource.resource_class.expects_with(action, params)
+      cmd.run(arguments)
+    end
+  end
+
+  def it_should_fail_with(message, arguments=[])
+    it "should fail with " + message.to_s do
+      proc { cmd.run(arguments) }.must_raise Clamp::UsageError
+    end
+  end
+
+  def it_should_accept(message, arguments=[])
+    it "should accept " + message.to_s do
+      cmd.run(arguments).must_equal HammerCLI::EX_OK
+    end
+  end
+
+  def it_should_print_column(column_name, arguments=nil)
+    it "should print column " + column_name do
+      arguments ||= respond_to?(:with_params) ? with_params : []
+
+      cmd.stubs(:context).returns({ :adapter => :test })
+      proc { cmd.run(arguments) }.must_output /.*##{column_name}#.*/
+    end
+  end
+
+  def it_should_print_columns(column_names, arguments=nil)
+    column_names.each do |name|
+      it_should_print_column name, arguments
+    end
+  end
+
+  def it_should_print_n_records(count=nil, arguments=nil)
+    it "should print correct count of records" do
+      arguments ||= respond_to?(:with_params) ? with_params : []
+
+      cmd.stubs(:context).returns({ :adapter => :test })
+      count ||= expected_record_count rescue 0
+      out, err = capture_io do
+        cmd.run(arguments)
+      end
+
+      out.split(/\n/).length.must_equal count+1 # plus 1 for line with column headers
+    end
+  end
+
+  def it_should_accept_search_params
+    it_should_accept "search", ["--search=some_search"]
+    it_should_accept "per page", ["--per-page=1"]
+    it_should_accept "page", ["--page=2"]
+    it_should_accept "order", ["--order=order"]
+  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 ApipieDisabledResourceMock.new(cmd.class.resource.resource_class)
+        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 ApipieDisabledResourceMock.new(cmd.class.resource.resource_class)
+        cmd.stubs(:context).returns({ :adapter => :test })
+
+        arguments = respond_to?(:with_params) ? with_params : []
+        lambda { cmd.run(arguments) }.must_output "", /.*not support.*/
+      end
+    end
+  end
+end

data/test/unit/host_test.rb

@@ -0,0 +1,315 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::Host do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ApipieResourceMock.new(cmd.class.resource.resource_class)
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept_search_params
+    end
+
+    context "output" do
+      let(:expected_record_count) { cmd.resource.call(:index)[0].length }
+
+      it_should_print_n_records
+      it_should_print_columns ["Id", "Name", "Operating System Id", "Host Group Id", "IP", "MAC"]
+    end
+
+  end
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::InfoCommand.new("", ctx) }
+
+    before :each do
+      HammerCLIForeman::Parameter.stubs(:get_parameters).returns([])
+    end
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=host"]
+      it_should_fail_with "no arguments"
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_n_records 1
+        it_should_print_columns ["Id", "Name", "Operating System Id", "Host Group Id", "IP", "MAC"]
+
+        it_should_print_columns ["UUID", "Cert name"]
+        it_should_print_columns ["Environment", "Environment Id"]
+        it_should_print_columns ["Managed", "Enabled", "Build"]
+        it_should_print_columns ["Use image", "Disk", "Image file"]
+        it_should_print_columns ["SP Name", "SP IP", "SP MAC", "SP Subnet", "SP Subnet Id"]
+        it_should_print_columns ["Created at", "Updated at", "Installed at", "Last report"]
+        it_should_print_columns ["Puppet CA Proxy Id", "Medium Id", "Model Id", "Owner Id", "Subnet Id", "Domain Id"]
+        it_should_print_columns ["Puppet Proxy Id", "Owner Type", "Partition Table Id", "Architecture Id", "Image Id", "Compute Resource Id"]
+        it_should_print_columns ["Comment"]
+      end
+    end
+
+  end
+
+  context "StatusCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::StatusCommand.new("", ctx) }
+
+    before :each do
+      resource_mock = ApipieResourceMock.new(cmd.class.resource.resource_class)
+      resource_mock.stub_method(:power, {'power' => 'running'})
+      cmd.class.resource resource_mock
+    end
+
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "no arguments"
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it_should_print_columns ["Status", "Power"]
+
+        it "should output status" do
+          cmd.stubs(:context).returns({ :adapter => :test })
+          proc { cmd.run(with_params) }.must_output "#Status#Power#\n#missing#running#\n"
+        end
+      end
+    end
+
+  end
+
+  context "FactsCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::FactsCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "no arguments"
+    end
+
+    context "output" do
+      with_params ["--name=my5name.mydomain.net"] do
+        it_should_print_n_records 2
+        it_should_print_column "Fact"
+        it_should_print_column "Value"
+      end
+    end
+  end
+
+
+  context "PuppetClassesCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::PuppetClassesCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "name or id missing", []
+    end
+
+    context "output" do
+      with_params ["--name=my5name.mydomain.net"] do
+        it_should_print_column "Id"
+        it_should_print_column "Name"
+      end
+    end
+
+  end
+
+
+  context "PuppetRunCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::PuppetRunCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "no arguments"
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        it "should inform that puppet was triggered" do
+          cmd.stubs(:context).returns({ :adapter => :test })
+          proc { cmd.run(with_params) }.must_output "Puppet run triggered\n"
+        end
+      end
+    end
+  end
+
+
+  context "ReportsCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::ReportsCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=my.test.host.org"]
+    end
+
+    context "output" do
+      with_params ["--id=1"] do
+        let(:expected_record_count) { cmd.resource.call(:index)[0].length }
+
+        it_should_print_n_records
+        it_should_print_column "Id"
+        it_should_print_column "Host"
+        it_should_print_column "Last report"
+        it_should_print_column "Applied"
+        it_should_print_column "Restarted"
+        it_should_print_column "Failed"
+        it_should_print_column "Restart Failures"
+        it_should_print_column "Skipped"
+        it_should_print_column "Pending"
+      end
+    end
+
+  end
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "name or id missing", []
+    end
+
+  end
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name, environment_id, architecture_id, domain_id, puppet_proxy_id, operatingsystem_id and more",
+          ["--name=host", "--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1",
+            "--ip=1.2.3.4", "--mac=11:22:33:44:55:66", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1",
+            "--sp-subnet-id=1", "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1']
+      it_should_fail_with "name or id missing",
+          ["--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1"]
+      it_should_fail_with "environment_id missing",
+          ["--name=host", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1"]
+      it_should_fail_with "architecture_id missing",
+          ["--name=host", "--environment-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1"]
+      it_should_fail_with "domain_id missing",
+          ["--name=host", "--environment-id=1", "--architecture-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1"]
+      it_should_fail_with "puppet_proxy_id missing",
+          ["--name=host", "--environment-id=1", "--architecture-id=1", "--domain-id=1", "--operatingsystem-id=1"]
+      it_should_fail_with "operatingsystem_id missing",
+          ["--name=host", "--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1"]
+    end
+  end
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=host", "--new-name=host2"]
+      it_should_accept "id and more", ["--id=1", "--new-name=host2", "--environment-id=1", "--architecture-id=1",
+            "--domain-id=1", "--puppet-proxy-id=1", "--operatingsystem-id=1",
+            "--ip=1.2.3.4", "--mac=11:22:33:44:55:66", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1",
+            "--sp-subnet-id=1", "--model-id=1", "--hostgroup-id=1", "--owner-id=1", '--puppet-ca-proxy-id=1']
+      it_should_fail_with "no params", []
+      it_should_fail_with "name or id missing", ["--new-name=host2"]
+    end
+
+  end
+
+
+  context "SetParameterCommand" do
+
+    before :each do
+      resource_mock = ApipieResourceMock.new(cmd.class.resource.resource_class)
+      resource_mock.stub_method(:index, [])
+      cmd.class.resource resource_mock
+    end
+
+    let(:cmd) { HammerCLIForeman::Host::SetParameterCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name, value and host name", ["--name=name", "--value=val", "--host-name=name"]
+      it_should_accept "name, value and host id", ["--name=name", "--value=val", "--host-id=id"]
+      it_should_fail_with "name missing", ["--value=val", "--host-name=name"]
+      it_should_fail_with "value missing", ["--name=name", "--host-name=name"]
+      it_should_fail_with "host name or id missing", ["--name=name", "--value=val"]
+    end
+
+  end
+
+
+  context "DeleteParameterCommand" do
+
+    let(:cmd) { HammerCLIForeman::Host::DeleteParameterCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name and host name", ["--name=name", "--host-name=name"]
+      it_should_accept "name and host id", ["--name=name", "--host-id=id"]
+      it_should_fail_with "name missing", ["--host-name=name"]
+      it_should_fail_with "host name or id missing", ["--name=name"]
+    end
+
+  end
+
+  context "StartCommand" do
+    let(:cmd) { HammerCLIForeman::Host::StartCommand.new("", ctx) }
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "empty params", []
+    end
+  end
+
+  context "StopCommand" do
+    let(:cmd) { HammerCLIForeman::Host::StopCommand.new("", ctx) }
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "id and force", ["--id=1", "--force"]
+      it_should_fail_with "empty params", []
+    end
+  end
+
+  context "RebootCommand" do
+    let(:cmd) { HammerCLIForeman::Host::RebootCommand.new("", ctx) }
+    context "parameters" do
+      it_should_accept "name", ["--name=host"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "empty params", []
+    end
+  end
+
+  context "SCParamsCommand" do
+
+    before :each do
+      cmd.class.resource ResourceMocks.smart_class_parameter
+    end
+
+    let(:cmd) { HammerCLIForeman::Host::SCParamsCommand.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", []
+    end
+
+  end
+
+end
+
+