hammer_cli_foreman 0.2.0 → 0.3.0

data/locale/zh_TW/hammer-cli-foreman.po

@@ -5,7 +5,7 @@
 # Translators:
 msgid ""
 msgstr ""
-"Project-Id-Version: hammer-cli-foreman 0.2.0\n"
+"Project-Id-Version: hammer-cli-foreman 0.3.0\n"
 "Report-Msgid-Bugs-To: \n"
 "POT-Creation-Date: 2014-12-11 13:21+0100\n"
 "PO-Revision-Date: 2015-03-22 03:37+0000\n"

data/test/unit/domain_test.rb

@@ -98,7 +98,7 @@ describe HammerCLIForeman::Domain do
 
     context "parameters" do
       it_should_accept "name, value and domain name", ["--name=name", "--value=val", "--domain=name"]
-      it_should_accept "name, value and domain id", ["--name=name", "--value=val", "--domain-id=id"]
+      it_should_accept "name, value and domain id", ["--name=name", "--value=val", "--domain-id=1"]
       # it_should_fail_with "name missing", ["--value=val", "--domain=name"]
       # it_should_fail_with "value missing", ["--name=name", "--domain=name"]
       # it_should_fail_with "domain name or id missing", ["--name=name", "--value=val"]
@@ -114,7 +114,7 @@ describe HammerCLIForeman::Domain do
 
     context "parameters" do
       it_should_accept "name and domain name", ["--name=param", "--domain=name"]
-      it_should_accept "name and domain id", ["--name=param", "--domain-id=id"]
+      it_should_accept "name and domain id", ["--name=param", "--domain-id=1"]
 
       # it_should_fail_with "name missing", ["--domain=name"]
       # it_should_fail_with "domain name or id missing", ["--name=param"]

data/test/unit/exception_handler_test.rb

@@ -98,7 +98,19 @@ describe HammerCLIForeman::ExceptionHandler do
     ex = RestClient::InternalServerError.new(response)
     ex.stubs(:message).returns(response)
 
-    output.expects(:print_error).with("Something went wrong", "Unformatted\nlines\n")
+    output.expects(:print_error).with(heading, "Unformatted\nlines\n")
+    err_code = handler.handle_exception(ex, :heading => heading)
+    err_code.must_equal HammerCLI::EX_SOFTWARE
+  end
+
+  it "should print exception message on internal error exception with message that is not nested" do
+    response = <<-RESPONSE
+    {"message":"Some internal exception"}
+    RESPONSE
+    ex = RestClient::InternalServerError.new(response)
+    ex.stubs(:message).returns(response)
+
+    output.expects(:print_error).with(heading, "Some internal exception")
     err_code = handler.handle_exception(ex, :heading => heading)
     err_code.must_equal HammerCLI::EX_SOFTWARE
   end

data/test/unit/host_test.rb

@@ -217,7 +217,7 @@ describe HammerCLIForeman::Host do
       it_should_fail_with "operatingsystem_id missing",
           ["--name=host", "--environment-id=1", "--architecture-id=1", "--domain-id=1", "--puppet-proxy-id=1", "--interface=primary=true,provision=true"]
       it_should_accept "only hostgroup name", ["--hostgroup=example", "--name=host", "--interface=primary=true,provision=true"]
-      it_should_accept "only hostgroup ID", ["--hostgroup-id=example", "--name=host", "--interface=primary=true,provision=true"]
+      it_should_accept "only hostgroup ID", ["--hostgroup-id=1", "--name=host", "--interface=primary=true,provision=true"]
 
       with_params ["--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",
@@ -231,6 +231,10 @@ describe HammerCLIForeman::Host do
         it_should_call_action_and_test_params(:create) { |par| par["host"]["interfaces_attributes"]["0"]["provision"] == "true" }
       end
 
+      with_params ["--name=host", "--hostgroup-id=1", "--interface=primary=true,provision=true", "--parameters=servers=[pool.ntp.org,ntp.time.org]"] do
+        it_should_call_action_and_test_params(:create) { |par| par["host"]["host_parameters_attributes"][0]["value"] == "[\"pool.ntp.org\", \"ntp.time.org\"]" }
+      end
+
       it_should_fail_with "primary interface missing", ["--hostgroup-id=example", "--interface=primary=true"]
       it_should_fail_with "provision interface missing", ["--hostgroup-id=example", "--interface=provision=true"]
     end
@@ -295,7 +299,7 @@ describe HammerCLIForeman::Host do
 
     context "parameters" do
       it_should_accept "name, value and host name", ["--name=name", "--value=val", "--host=name"]
-      it_should_accept "name, value and host id", ["--name=name", "--value=val", "--host-id=id"]
+      it_should_accept "name, value and host id", ["--name=name", "--value=val", "--host-id=1"]
       it_should_fail_with "name missing", ["--value=val", "--host=name"]
       it_should_fail_with "value missing", ["--name=name", "--host=name"]
       # it_should_fail_with "host name or id missing", ["--name=name", "--value=val"]
@@ -311,7 +315,7 @@ describe HammerCLIForeman::Host do
 
     context "parameters" do
       it_should_accept "name and host name", ["--name=name", "--host=name"]
-      it_should_accept "name and host id", ["--name=name", "--host-id=id"]
+      it_should_accept "name and host id", ["--name=name", "--host-id=1"]
       # it_should_fail_with "name missing", ["--host=name"]
       # it_should_fail_with "host name or id missing", ["--name=name"]
       # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -383,5 +387,3 @@ describe HammerCLIForeman::Host do
   end
 
 end
-
-

data/test/unit/hostgroup_test.rb

@@ -96,7 +96,7 @@ describe HammerCLIForeman::Hostgroup do
     let(:cmd) { HammerCLIForeman::Hostgroup::SetParameterCommand.new("", ctx) }
 
     context "parameters" do
-      it_should_accept "name, value and hostgroup id", ["--name=name", "--value=val", "--hostgroup-id=id"]
+      it_should_accept "name, value and hostgroup id", ["--name=name", "--value=val", "--hostgroup-id=1"]
       it_should_fail_with "name missing", ["--value=val", "--hostgroup-id=1"]
       it_should_fail_with "value missing", ["--name=name", "--hostgroup-id=1"]
       # it_should_fail_with "hostgroup id missing", ["--name=name", "--value=val"] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -110,7 +110,7 @@ describe HammerCLIForeman::Hostgroup do
     let(:cmd) { HammerCLIForeman::Hostgroup::DeleteParameterCommand.new("", ctx) }
 
     context "parameters" do
-      it_should_accept "name and hostgroup id", ["--name=param", "--hostgroup-id=id"]
+      it_should_accept "name and hostgroup id", ["--name=param", "--hostgroup-id=1"]
       # it_should_fail_with "name missing", ["--hostgroup-id=id"]
       # it_should_fail_with "hostgroup id missing", ["--name=param"]
       # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -152,4 +152,3 @@ describe HammerCLIForeman::Hostgroup do
   end
 
 end
-

data/test/unit/operating_system_test.rb

@@ -124,7 +124,7 @@ describe HammerCLIForeman::OperatingSystem do
     let(:cmd) { HammerCLIForeman::OperatingSystem::SetParameterCommand.new("", ctx) }
 
     context "parameters" do
-      it_should_accept "name, value and os id", ["--name=domain", "--value=val", "--operatingsystem-id=id"]
+      it_should_accept "name, value and os id", ["--name=domain", "--value=val", "--operatingsystem-id=1"]
       it_should_accept "name, value and os title", ["--name=domain", "--value=val", "--operatingsystem=Rhel 6.5"]
       # it_should_fail_with "name missing", ["--value=val", "--operatingsystem-id=id"]
       # it_should_fail_with "value missing", ["--name=name", "--operatingsystem-id=id"]
@@ -140,7 +140,7 @@ describe HammerCLIForeman::OperatingSystem do
     let(:cmd) { HammerCLIForeman::OperatingSystem::DeleteParameterCommand.new("", ctx) }
 
     context "parameters" do
-      it_should_accept "name and os id", ["--name=domain", "--operatingsystem-id=id"]
+      it_should_accept "name and os id", ["--name=domain", "--operatingsystem-id=1"]
       it_should_accept "name and os title", ["--name=domain", "--operatingsystem=Rhel 6.5"]
       # it_should_fail_with "name missing", ["--operatingsystem-id=id"]
       # it_should_fail_with "os id missing", ["--name=name"]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Tomáš Strachota
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-23 00:00:00.000000000 Z
+date: 2015-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
   name: apipie-bindings
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +66,7 @@ executables: []
 extensions: []
 extra_rdoc_files:
 - doc/configuration.md
+- doc/using_hammer_cli_foreman_command.md
 - doc/developer_docs.md
 - doc/option_builder.md
 - doc/host_create.md
@@ -81,6 +82,7 @@ files:
 - doc/name_id_resolution.md
 - doc/option_builder.md
 - doc/release_notes.md
+- doc/using_hammer_cli_foreman_command.md
 - lib/hammer_cli_foreman.rb
 - lib/hammer_cli_foreman/architecture.rb
 - lib/hammer_cli_foreman/associating_commands.rb