hammer_cli_foreman 0.0.10 → 0.0.11

data/test/unit/puppet_class_test.rb

@@ -0,0 +1,71 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::PuppetClass do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ApipieResourceMock.new(cmd.class.resource.resource_class)
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::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) { cmd.resource.call(:index)[0].length }
+
+      it_should_print_n_records
+      it_should_print_column "Id"
+      it_should_print_column "Name"
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::PuppetClass::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "no arguments"
+    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"
+        it_should_print_column "Smart variables"
+      end
+    end
+
+  end
+
+  context "SCParamsCommand" do
+
+    before :each do
+      cmd.class.resource ResourceMocks.smart_class_parameter
+    end
+
+    let(:cmd) { HammerCLIForeman::PuppetClass::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

data/test/unit/report_test.rb

@@ -0,0 +1,64 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::Report do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ApipieResourceMock.new(cmd.class.resource.resource_class)
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Report::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_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
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::Report::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "no arguments"
+    end
+
+  end
+
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Report::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "id missing", []
+    end
+
+  end
+
+
+end

data/test/unit/smart_class_parameter_test.rb

@@ -0,0 +1,77 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::SmartClassParameter do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ResourceMocks.smart_class_parameter
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::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_accept "environment id", ["--environment-id=1"]
+      it_should_accept "puppet class id", ["--puppetclass-id=1"]
+      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_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
+
+    let(:cmd) { HammerCLIForeman::SmartClassParameter::InfoCommand.new("", ctx) }
+    
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=param"]
+      it_should_fail_with "no arguments"
+    end
+
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartClassParameter::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      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 "id missing", []
+    end
+
+  end
+
+
+end

data/test/unit/smart_proxy_test.rb

@@ -0,0 +1,102 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::SmartProxy do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ApipieResourceMock.new(cmd.class.resource.resource_class)
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::ListCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+      it_should_accept "type, page, per_page", ["--type=tftp", "--page=1", "--per-page=2"]
+    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", "URL"]
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=proxy"]
+      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", "URL", "Features", "Created at", "Updated at"]
+      end
+    end
+
+  end
+
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name and url", ["--name=proxy", "--url=localhost"]
+      it_should_fail_with "name missing",     ["--url=loaclhost"]
+      it_should_fail_with "url missing",  ["--name=proxy"]
+    end
+
+  end
+
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=proxy"]
+      it_should_fail_with "name or id missing", []
+    end
+
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1", "--new-name=proxy2", "--url=localhost"]
+      it_should_accept "name", ["--name=proxy", "--new-name=proxy2", "--url=localhost"]
+      it_should_fail_with "no params", []
+      it_should_fail_with "name or id missing", ["--new-name=proxy2"]
+    end
+
+  end
+
+  context "ImportPuppetClassesCommand" do
+
+    let(:cmd) { HammerCLIForeman::SmartProxy::ImportPuppetClassesCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id, environment-id and dryrun", ["--id=1", "--environment-id=1", "--dryrun"]
+      it_should_fail_with "id missing", [""]
+    end
+
+  end
+
+end

data/test/unit/subnet_test.rb

@@ -0,0 +1,95 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::Subnet do
+
+  extend CommandTestHelper
+
+  before :each do
+    cmd.class.resource ApipieResourceMock.new(cmd.class.resource.resource_class)
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Subnet::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", "Network", "Mask"]
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::Subnet::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=arch"]
+      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", "Network", "Mask"]
+        it_should_print_columns ["Priority"]
+        it_should_print_columns ["DNS", "Primary DNS", "Secondary DNS"]
+        it_should_print_columns ["Domain ids", "TFTP", "TFTP id", "DHCP", "DHCP id"]
+        it_should_print_columns ["vlan id", "Gateway", "From", "To"]
+      end
+    end
+
+  end
+
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Subnet::CreateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=arch", "--network=192.168.83.0", "--mask=255.255.255.0"]
+      it_should_fail_with "name missing",     ["--network=192.168.83.0", "--mask=255.255.255.0"]
+      it_should_fail_with "network missing",  ["--name=arch", "--mask=255.255.255.0"]
+      it_should_fail_with "mask missing",     ["--name=arch", "--network=192.168.83.0"]
+    end
+
+  end
+
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Subnet::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=arch"]
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "name or id missing", []
+    end
+
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Subnet::UpdateCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "name", ["--name=arch", "--new-name=arch2"]
+      it_should_accept "id", ["--id=1", "--new-name=arch2"]
+      it_should_fail_with "no params", []
+      it_should_fail_with "name or id missing", ["--new-name=arch2"]
+    end
+
+  end
+end

data/test/unit/template_test.rb

@@ -0,0 +1,141 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
+
+
+describe HammerCLIForeman::Template do
+
+  extend CommandTestHelper
+
+  let(:template_hash) {
+    {
+      "config_template" => {
+        "template_kind" => {}
+      }
+    }
+  }
+
+  before :each do
+    resource_mock = ApipieResourceMock.new(cmd.class.resource.resource_class)
+    resource_mock.stub_method(:index, [])
+    resource_mock.stub_method(:show, template_hash)
+    cmd.class.resource resource_mock
+    File.stubs(:read).returns("")
+  end
+
+  context "ListCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::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", "Type"]
+
+      it "should print template without kind set" do
+        template_wo_kind = {
+          "config_template" => {
+            :id => 1, :name => "PXE"
+          }
+        }
+        mock_resource_method(:index, [[template_wo_kind], nil])
+        cmd.run([]).must_equal 0
+      end
+    end
+
+  end
+
+
+  context "InfoCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::InfoCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      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", "Type", "OS ids"]
+      end
+    end
+
+  end
+
+
+  context "ListKindsCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::ListKindsCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "no arguments"
+    end
+
+  end
+
+
+  context "DumpCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::DumpCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "id missing", []
+    end
+
+  end
+
+
+  context "CreateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::CreateCommand.new("", ctx) }
+
+    before :each do
+      cmd.stubs(:template_kind_id).returns(1)
+    end
+
+    context "parameters" do
+      it_should_accept "name, file, type, audit comment, os ids", ["--name=tpl", "--file=~/tpl.sh", "--type=snippet", "--audit-comment=fix", "--operatingsystem-ids=1,2,3"]
+      it_should_fail_with "name missing", ["--file=~/tpl.sh", "--type=snippet", "--audit-comment=fix", "--operatingsystem-ids=1,2,3"]
+      it_should_fail_with "type missing", ["--name=tpl", "--file=~/tpl.sh", "--audit-comment=fix", "--operatingsystem-ids=1,2,3"]
+      it_should_fail_with "file missing", ["--name=tpl", "--type=snippet", "--audit-comment=fix", "--operatingsystem-ids=1,2,3"]
+    end
+
+  end
+
+
+  context "UpdateCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::UpdateCommand.new("", ctx) }
+
+    before :each do
+      cmd.stubs(:template_kind_id).returns(1)
+    end
+
+    context "parameters" do
+      it_should_accept "id, name, file, type, audit comment, os ids", ["--id=83", "--name=tpl", "--file=~/tpl.sh", "--type=snippet", "--audit-comment=fix", "--operatingsystem-ids=1,2,3"]
+      it_should_fail_with "id missing", ["--name=tpl", "--file=~/tpl.sh", "--type=snippet", "--audit-comment=fix", "--operatingsystem-ids=1,2,3"]
+    end
+
+  end
+
+
+  context "DeleteCommand" do
+
+    let(:cmd) { HammerCLIForeman::Template::DeleteCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_fail_with "id missing", []
+    end
+
+  end
+
+end