hammer_cli_foreman 3.7.0 → 3.9.0

data/test/unit/exception_handler_test.rb

@@ -125,7 +125,7 @@ describe HammerCLIForeman::ExceptionHandler do
     _(err_code).must_equal HammerCLI::EX_SOFTWARE
   end
 
-  context "redirects" do
+  describe "redirects" do
     let(:response) { HammerCLIForeman.foreman_api.api.send(:create_fake_response, 301, '', 'GET', 'http://foreman.example.com/api/architectures') }
     it "should detect http to https redirection error" do
       response.headers[:location] = 'https://foreman.example.com/api/architectures'

data/test/unit/external_usergroup_test.rb

@@ -7,19 +7,19 @@ require 'hammer_cli_foreman/usergroup'
 describe HammerCLIForeman::ExternalUsergroup do
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::ListCommand.new("", ctx) }
 
     before :each do
       ResourceMocks.mock_action_call(:external_usergroups, :index, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "user group name", ["--user-group=cr"]
       it_should_accept "user group id", ["--user-group-id=1"]
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:index, :usergroup_id => 1)) }
 
       with_params ["--user-group-id=1"] do
@@ -31,19 +31,19 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::InfoCommand.new("", ctx) }
 
     before :each do
       ResourceMocks.mock_action_call(:external_usergroups, :show, {})
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "user group name and external usergroup's id", ["--user-group=cr", "--id=1"]
       it_should_accept "user group id and external usergroup's id", ["--user-group-id=1", "--id=1"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--user-group-id=1", "--id=1"] do
         it_should_print_column "Name"
         it_should_print_column "Auth source"
@@ -51,19 +51,19 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
   end
 
-  context "RefreshExternalUsergroupsCommand" do
+  describe "RefreshExternalUsergroupsCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::RefreshExternalUsergroupsCommand.new("", ctx) }
 
     before :each do
       ResourceMocks.mock_action_call(:external_usergroups, :refresh, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "user group name and external usergroup's id", ["--user-group=cr", "--id=1"]
       it_should_accept "user group id and external usergroup's id", ["--user-group-id=1", "--id=1"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--user-group-id=1", "--id=1"] do
         it_should_print_column "Name"
         it_should_print_column "Auth source"
@@ -71,30 +71,30 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       # it_should_fail_with "no name", ["--auth-source-id=1", "--user-group-id=1"]
       it_should_accept "all required params", ["--name=aabbcc123", "--auth-source-id=1", "--user-group-id=1"]
     end
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id and user group's id", ["--id=1", "--user-group-id=1"]
       it_should_accept "id and user group's name", ["--id=1", "--user-group=ec2"]
     end
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id and user group's id", ["--id=1", "--user-group-id=1"]
       it_should_accept "id and user group's name", ["--id=1", "--user-group=ec2"]
       it_should_accept "all available params", ["--id=1", "--name=img", "--auth-source-id=1", "--user-group-id=1"]

data/test/unit/fact_test.rb

@@ -7,7 +7,7 @@ describe HammerCLIForeman::Fact do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     let(:cmd) { HammerCLIForeman::Fact::ListCommand.new("", ctx) }
 
@@ -15,12 +15,12 @@ describe HammerCLIForeman::Fact do
       ResourceMocks.facts_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 "Host"
       it_should_print_column "Fact"
       it_should_print_column "Value"

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