hammer_cli_foreman 3.6.0 → 3.8.0

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/id_resolver_test.rb

@@ -109,7 +109,7 @@ describe HammerCLIForeman::IdResolver do
       let(:resolver_run) { proc { resolver.comment_id({"option_unknown" => "value"}) } }
 
       it "raises exception" do
-        err = _(resolver_run).must_raise HammerCLIForeman::MissingSearchOptions
+        _(resolver_run).must_raise HammerCLIForeman::MissingSearchOptions
       end
 
       it "builds correct error message" do

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"]

data/test/unit/location_test.rb

@@ -8,7 +8,7 @@ describe HammerCLIForeman::Location do
   include CommandTestHelper
   extend ResourceDisabled
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     let(:cmd) { HammerCLIForeman::Location::ListCommand.new("", ctx) }
 
@@ -16,12 +16,12 @@ describe HammerCLIForeman::Location do
       ResourceMocks.locations_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
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
@@ -34,7 +34,7 @@ describe HammerCLIForeman::Location do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Location::InfoCommand.new("", ctx) }
 
@@ -42,13 +42,13 @@ describe HammerCLIForeman::Location do
       ResourceMocks.locations_show
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=arch"]
       # 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_column "Name"
@@ -65,11 +65,11 @@ describe HammerCLIForeman::Location do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Location::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=org"]
       # it_should_fail_with "name missing", []
       # TODO: temporarily disabled, parameters are checked in the api
@@ -81,11 +81,11 @@ describe HammerCLIForeman::Location do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Location::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=org"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -97,11 +97,11 @@ describe HammerCLIForeman::Location do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Location::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=org", "--new-name=org2"]
       it_should_accept "id", ["--id=1", "--new-name=org2"]
       # it_should_fail_with "no params", []

data/test/unit/mail_notification_test.rb

@@ -8,20 +8,20 @@ require 'hammer_cli_foreman/mail_notification'
 describe HammerCLIForeman::MailNotification do
   include CommandTestHelper
 
-  context 'ListCommand' do
+  describe 'ListCommand' do
     before :each do
       ResourceMocks.mail_notifications
     end
 
     let(:cmd) { HammerCLIForeman::MailNotification::ListCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'no arguments'
       it_should_accept 'organization', ['--organization-id=1']
       it_should_accept 'location', ['--location-id=1']
     end
 
-    context 'output' do
+    describe 'output' do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
@@ -30,17 +30,17 @@ describe HammerCLIForeman::MailNotification do
     end
   end
 
-  context 'InfoCommand' do
+  describe 'InfoCommand' do
     let(:cmd) { HammerCLIForeman::MailNotification::InfoCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'id', ['--id=1']
       it_should_accept 'name', ['--name=test']
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
       it_should_accept 'location', %w[--id=1 --location-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'