hammer_cli_foreman 3.7.0 → 3.8.0

data/test/unit/filter_test.rb

@@ -8,7 +8,7 @@ describe HammerCLIForeman::Filter do
   include CommandTestHelper
 
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     let(:cmd) { HammerCLIForeman::Filter::ListCommand.new("", ctx) }
 
@@ -16,12 +16,12 @@ describe HammerCLIForeman::Filter do
       ResourceMocks.mock_action_call(:filters, :index, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
     end
 
-    context "output" do
+    describe "output" do
       it_should_print_column "Id"
       it_should_print_column "Resource type"
       it_should_print_column "Search"
@@ -31,7 +31,7 @@ describe HammerCLIForeman::Filter do
 
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Filter::InfoCommand.new("", ctx) }
 
@@ -39,11 +39,11 @@ describe HammerCLIForeman::Filter do
       ResourceMocks.mock_action_call(:filters, :show, {})
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--id=1"] do
         it_should_print_n_records 1
         it_should_print_column "Id"
@@ -57,7 +57,7 @@ describe HammerCLIForeman::Filter do
     end
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
     let(:cmd) { HammerCLIForeman::Filter::CreateCommand.new("", ctx) }
 
     before do
@@ -65,7 +65,7 @@ describe HammerCLIForeman::Filter do
       cmd.stubs(:option_override).returns(false)
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "role id and permission ids", ["--role-id=1", "--permission-ids=1,2"]
       it_should_accept "role name and permission ids", ["--role=role", "--permission-ids=1,2"]
       it_should_accept "role name, permission ids, search", ["--role=role", "--permission-ids=1,2", "--search='name=*'"]
@@ -73,16 +73,16 @@ describe HammerCLIForeman::Filter do
 
   end
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Filter::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
     end
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
     let(:cmd) { HammerCLIForeman::Filter::UpdateCommand.new("", ctx) }
 
     before do
@@ -90,7 +90,7 @@ describe HammerCLIForeman::Filter do
       cmd.stubs(:option_override).returns(false)
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id, role id and permission ids", ["--id=1", "--role-id=1", "--permission-ids=1,2"]
       it_should_accept "id, role name and permission ids", ["--id=1", "--role=role", "--permission-ids=1,2"]
       it_should_accept "id, role name, permission ids, search", ["--id=1", "--role=role", "--permission-ids=1,2", "--search='name=*'"]
@@ -99,7 +99,7 @@ describe HammerCLIForeman::Filter do
   end
 
 
-  context "AvailablePermissionsCommand" do
+  describe "AvailablePermissionsCommand" do
 
     let(:cmd) { HammerCLIForeman::Filter::AvailablePermissionsCommand.new("", ctx) }
 
@@ -107,12 +107,12 @@ describe HammerCLIForeman::Filter do
       ResourceMocks.mock_action_call(:permissions, :index, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept "pagination options", ["--page=2", "--per-page=10"]
     end
 
-    context "output" do
+    describe "output" do
       it_should_print_column "Id"
       it_should_print_column "Name"
       it_should_print_column "Resource"
@@ -120,7 +120,7 @@ describe HammerCLIForeman::Filter do
 
   end
 
-  context "AvailableResourcesCommand" do
+  describe "AvailableResourcesCommand" do
 
     let(:cmd) { HammerCLIForeman::Filter::AvailableResourcesCommand.new("", ctx) }
 
@@ -128,11 +128,11 @@ describe HammerCLIForeman::Filter do
       ResourceMocks.mock_action_call(:permissions, :resource_types, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
     end
 
-    context "output" do
+    describe "output" do
       it_should_print_column "Name"
     end
 

data/test/unit/helpers/command.rb

@@ -61,7 +61,7 @@ module CommandTestHelper
   module ClassMethods
 
     def with_params(params, &block)
-      context "with params "+params.to_s do
+      describe "with params "+params.to_s do
         let(:with_params) { params }
         self.instance_eval(&block)
       end

data/test/unit/helpers/resource_disabled.rb

@@ -4,7 +4,7 @@ module ResourceDisabled
 
   def it_should_fail_when_disabled
     arguments = @with_params ? @with_params.dup : []
-    context "resource disabled" do
+    describe "resource disabled" do
 
       it "should return error" do
         cmd.class.resource.stubs(:call).raises(RestClient::ResourceNotFound)

data/test/unit/host_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::Host do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:hosts, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Host::ListCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
@@ -28,20 +28,20 @@ describe HammerCLIForeman::Host do
 
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
     before do
       ResourceMocks.hosts_show
     end
 
     let(:cmd) { HammerCLIForeman::Host::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=host"]
       # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--id=1"] do
         it_should_print_n_records 1
         it_should_print_columns ["Id", "Name", "Organization", "Location"]
@@ -53,7 +53,7 @@ describe HammerCLIForeman::Host do
 
   end
 
-  context "StatusCommand" do
+  describe "StatusCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::StatusCommand.new("", ctx) }
 
@@ -62,14 +62,14 @@ describe HammerCLIForeman::Host do
       ResourceMocks.mock_action_call(:hosts, :get_status, { 'status_label' => 'No reports' } )
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       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
+    describe "output" do
       with_params ["--id=1"] do
         it_should_print_columns ["Status", "Power"]
 
@@ -82,7 +82,7 @@ describe HammerCLIForeman::Host do
 
   end
 
-  context "FactsCommand" do
+  describe "FactsCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::FactsCommand.new("", ctx) }
 
@@ -90,14 +90,14 @@ describe HammerCLIForeman::Host do
       ResourceMocks.facts_index
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       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
+    describe "output" do
       with_params ["--name=my5name.mydomain.net"] do
         it_should_print_column "Fact"
         it_should_print_column "Value"
@@ -105,19 +105,19 @@ describe HammerCLIForeman::Host do
     end
   end
 
-  context "ConfigReportsCommand" do
+  describe "ConfigReportsCommand" do
     before do
       ResourceMocks.mock_action_call(:config_reports, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Host::ConfigReportsCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=my.test.host.org"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--id=1"] do
         let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
@@ -136,11 +136,11 @@ describe HammerCLIForeman::Host do
 
   end
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "name or id missing", []
@@ -149,7 +149,7 @@ describe HammerCLIForeman::Host do
 
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::CreateCommand.new("", ctx) }
 
@@ -157,7 +157,7 @@ describe HammerCLIForeman::Host do
       HammerCLIForeman::Hosts::CommonUpdateOptions.stubs(:ask_password).returns("password")
     end
 
-    context "parameters" do
+    describe "parameters" do
       taxonomies = ["--organization-id=1", "--location-id=1"]
       it_should_accept "name, architecture_id, domain_id, operatingsystem_id and more",
           ["--name=host", "--architecture-id=1", "--domain-id=1", "--operatingsystem-id=1",
@@ -199,7 +199,7 @@ describe HammerCLIForeman::Host do
     end
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::UpdateCommand.new("", ctx) }
 
@@ -207,7 +207,7 @@ describe HammerCLIForeman::Host do
       HammerCLIForeman::Hosts::CommonUpdateOptions.stubs(:ask_password).returns("password")
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host", "--new-name=host2"]
       it_should_accept "id and more", ["--id=1", "--new-name=host2", "--architecture-id=1",
             "--domain-id=1", "--operatingsystem-id=1",
@@ -253,7 +253,7 @@ describe HammerCLIForeman::Host do
   end
 
 
-  context "SetParameterCommand" do
+  describe "SetParameterCommand" do
 
     before :each do
       ResourceMocks.parameters_index
@@ -261,7 +261,7 @@ describe HammerCLIForeman::Host do
 
     let(:cmd) { HammerCLIForeman::Host::SetParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "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=1"]
       it_should_accept "name, value, type and host id", ["--name=name", "--parameter-type=integer", "--value=1", "--host-id=1"]
@@ -274,11 +274,11 @@ describe HammerCLIForeman::Host do
   end
 
 
-  context "DeleteParameterCommand" do
+  describe "DeleteParameterCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::DeleteParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name and host name", ["--name=name", "--host=name"]
       it_should_accept "name and host id", ["--name=name", "--host-id=1"]
       # it_should_fail_with "name missing", ["--host=name"]
@@ -288,9 +288,9 @@ describe HammerCLIForeman::Host do
 
   end
 
-  context "StartCommand" do
+  describe "StartCommand" do
     let(:cmd) { HammerCLIForeman::Host::StartCommand.new("", ctx) }
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "empty params", []
@@ -298,9 +298,9 @@ describe HammerCLIForeman::Host do
     end
   end
 
-  context "StopCommand" do
+  describe "StopCommand" do
     let(:cmd) { HammerCLIForeman::Host::StopCommand.new("", ctx) }
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
       it_should_accept "id and force", ["--id=1", "--force"]
@@ -309,9 +309,9 @@ describe HammerCLIForeman::Host do
     end
   end
 
-  context "RebootCommand" do
+  describe "RebootCommand" do
     let(:cmd) { HammerCLIForeman::Host::RebootCommand.new("", ctx) }
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "empty params", []
@@ -319,11 +319,11 @@ describe HammerCLIForeman::Host do
     end
   end
 
-  context "RebuildConfigCommand" do
+  describe "RebuildConfigCommand" do
 
     let(:cmd) { HammerCLIForeman::Host::RebuildConfigCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "no arguments"
@@ -333,9 +333,9 @@ describe HammerCLIForeman::Host do
   end
 
 
-  context "DisassociateCommand" do
+  describe "DisassociateCommand" do
     let(:cmd) { HammerCLIForeman::Host::DisassociateCommand.new("", ctx) }
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
     end

data/test/unit/hostgroup_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::Hostgroup do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:hostgroups, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Hostgroup::ListCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
@@ -29,16 +29,16 @@ describe HammerCLIForeman::Hostgroup do
 
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "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
+    describe "output" do
       with_params ["--id=1"] do
         it_should_print_n_records 1
         it_should_print_columns ["Id", "Name", "Title"]
@@ -49,22 +49,22 @@ describe HammerCLIForeman::Hostgroup do
 
   end
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "no params", [] # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, parent_id, architecture_id, domain_id, operatingsystem_id and more",
           ["--name=hostgroup", "--parent-id=1", "--architecture-id=1", "--domain-id=1",
             "--operatingsystem-id=1", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1", '--root-password=foreman']
@@ -74,11 +74,11 @@ describe HammerCLIForeman::Hostgroup do
     end
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, parent_id, architecture_id, domain_id, operatingsystem_id and more",
           ["--id=1 --name=hostgroup2 --title=default/hostgroup2", "--parent-id=1", "--architecture-id=1", "--domain-id=1",
             "--operatingsystem-id=1", "--medium-id=1", "--partition-table-id=1", "--subnet-id=1", '--root-password=foreman']
@@ -90,7 +90,7 @@ describe HammerCLIForeman::Hostgroup do
   end
 
 
-  context "SetParameterCommand" do
+  describe "SetParameterCommand" do
 
     before :each do
       ResourceMocks.parameters_index
@@ -98,7 +98,7 @@ describe HammerCLIForeman::Hostgroup do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::SetParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, value, parameter-type and hostgroup id", ["--name=name", "--parameter-type=string", "--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"]
@@ -108,11 +108,11 @@ describe HammerCLIForeman::Hostgroup do
   end
 
 
-  context "DeleteParameterCommand" do
+  describe "DeleteParameterCommand" do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::DeleteParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       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"]
@@ -121,11 +121,11 @@ describe HammerCLIForeman::Hostgroup do
 
   end
 
-  context "RebuildConfigCommand" do
+  describe "RebuildConfigCommand" do
 
     let(:cmd) { HammerCLIForeman::Hostgroup::RebuildConfigCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=host"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "no arguments"

data/test/unit/image_test.rb

@@ -9,20 +9,20 @@ describe HammerCLIForeman::Image do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:images, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Image::ListCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "compute resource name", ["--compute-resource=cr"]
       it_should_accept "compute resource id", ["--compute-resource-id=1"]
       #it_should_accept_search_params
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:index, :compute_resource_id=>1)) }
 
       with_params ["--compute-resource-id=1"] do
@@ -37,16 +37,16 @@ describe HammerCLIForeman::Image do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Image::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "compute resource name and image's uuid", ["--compute-resource=cr", "--id=1"]
       it_should_accept "compute resource id and image's uuid", ["--compute-resource-id=1", "--id=1"]
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { cmd.resource.call(:index).length }
 
       with_params ["--compute-resource-id=1", "--id=1"] do
@@ -62,7 +62,7 @@ describe HammerCLIForeman::Image do
     end
   end
 
-  context "AvailableImagesCommand" do
+  describe "AvailableImagesCommand" do
 
     let(:cmd) { HammerCLIForeman::Image::AvailableImagesCommand.new("", ctx) }
 
@@ -71,12 +71,12 @@ describe HammerCLIForeman::Image do
       ResourceMocks.compute_resources_available_images
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "compute resource name", ["--compute-resource=cr"]
       it_should_accept "compute resource id", ["--compute-resource-id=1"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--compute-resource-id=1"] do
         it_should_print_column "Name"
         it_should_print_column "UUID"
@@ -84,11 +84,11 @@ describe HammerCLIForeman::Image do
     end
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Image::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "all required params", ["--name=img", "--operatingsystem-id=1", "--architecture-id=1", "--username=root", "--uuid=aabbcc123", "--compute-resource-id=1"]
       it_should_accept "all required params and resource's name", ["--name=img", "--operatingsystem-id=1", "--architecture-id=1", "--username=root", "--uuid=aabbcc123", "--compute-resource=ec2"]
       # it_should_fail_with "name missing", ["--operatingsystem-id=1", "architecture-id=1", "--username=root", "--uuid=aabbcc123", "--compute-resource-id=1"]
@@ -102,11 +102,11 @@ describe HammerCLIForeman::Image do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Image::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id and resource's id", ["--id=1", "--compute-resource-id=1"]
       it_should_accept "id and resource's name", ["--id=1", "--compute-resource=ec2"]
       # it_should_fail_with "id missing", ["--compute-resource-id=1"]
@@ -117,11 +117,11 @@ describe HammerCLIForeman::Image do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Image::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id and resource's id", ["--id=1", "--compute-resource-id=1"]
       it_should_accept "id and resource's name", ["--id=1", "--compute-resource=ec2"]
       it_should_accept "all available params", ["--id=1", "--name=img", "--operatingsystem-id=1", "--architecture-id=1", "--username=root", "--uuid=aabbcc123", "--compute-resource-id=1"]