hammer_cli_foreman 3.6.0 → 3.8.0

data/test/unit/role_test.rb

@@ -8,7 +8,7 @@ describe HammerCLIForeman::Role do
   include CommandTestHelper
 
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     let(:cmd) { HammerCLIForeman::Role::ListCommand.new("", ctx) }
 
@@ -16,12 +16,12 @@ describe HammerCLIForeman::Role do
       ResourceMocks.mock_action_call(:roles, :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 "Name"
     end
@@ -29,11 +29,11 @@ describe HammerCLIForeman::Role do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Role::InfoCommand.new("", ctx) }
 
-    context "output" do
+    describe "output" do
 
       with_params ["--name=role"] do
         it_should_print_n_records 1
@@ -44,11 +44,11 @@ describe HammerCLIForeman::Role do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Role::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=role"]
     end
 
@@ -57,21 +57,21 @@ describe HammerCLIForeman::Role do
     end
   end
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Role::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=role"]
       it_should_accept "id", ["--id=1"]
     end
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Role::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=role"]
       it_should_accept "id", ["--id=1"]
       it_should_accept "name and new name", ["--name=role", "--new-name=role2"]
@@ -83,7 +83,7 @@ describe HammerCLIForeman::Role do
 
   end
 
-  context "FiltersCommand" do
+  describe "FiltersCommand" do
 
     let(:cmd) { HammerCLIForeman::Role::FiltersCommand.new("", ctx) }
 
@@ -91,13 +91,13 @@ describe HammerCLIForeman::Role do
       ResourceMocks.mock_action_call(:filters, :index, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=role"]
       it_should_accept "id", ["--id=1"]
       it_should_fail_with "no arguments"
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--id=1"] do
         it_should_print_column "Id"
         it_should_print_column "Resource type"

data/test/unit/settings_test.rb

@@ -7,7 +7,7 @@ describe HammerCLIForeman::Settings do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     before do
       ResourceMocks.mock_action_call(:settings, :index, [])
@@ -15,14 +15,14 @@ describe HammerCLIForeman::Settings do
 
     let(:cmd) { HammerCLIForeman::Settings::ListCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
       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
@@ -34,10 +34,10 @@ describe HammerCLIForeman::Settings do
 
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
     let(:cmd) { HammerCLIForeman::Settings::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=setting1", "--value=setting2"]
       it_should_accept "id", ["--id=1", "--value=setting2"]
       it_should_accept 'organization', %w[--id=1 --organization-id=1]

data/test/unit/smart_proxy_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::SmartProxy do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:smart_proxies, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::SmartProxy::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,17 +29,17 @@ describe HammerCLIForeman::SmartProxy do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::SmartProxy::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=proxy"]
       # 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", "URL", "Features", "Created at", "Updated at"]
@@ -49,11 +49,11 @@ describe HammerCLIForeman::SmartProxy do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::SmartProxy::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name and url", ["--name=proxy", "--url=localhost"]
       # it_should_fail_with "name missing",     ["--url=localhost"]
       # it_should_fail_with "url missing",  ["--name=proxy"]
@@ -63,11 +63,11 @@ describe HammerCLIForeman::SmartProxy do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::SmartProxy::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=proxy"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -76,11 +76,11 @@ describe HammerCLIForeman::SmartProxy do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::SmartProxy::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "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", []
@@ -90,11 +90,11 @@ describe HammerCLIForeman::SmartProxy do
 
   end
 
-  context "RefreshFeaturesCommand" do
+  describe "RefreshFeaturesCommand" do
 
     let(:cmd) { HammerCLIForeman::SmartProxy::RefreshFeaturesCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=proxy"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver

data/test/unit/subnet_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::Subnet do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:subnets, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Subnet::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
@@ -30,17 +30,17 @@ describe HammerCLIForeman::Subnet do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Subnet::InfoCommand.new("", ctx) }
 
-    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 ["--name=subnet"] do
         it_should_print_n_records 1
@@ -58,11 +58,11 @@ describe HammerCLIForeman::Subnet do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Subnet::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "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"]
@@ -73,11 +73,11 @@ describe HammerCLIForeman::Subnet do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Subnet::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=arch"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -86,11 +86,11 @@ describe HammerCLIForeman::Subnet do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Subnet::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "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", []

data/test/unit/template_test.rb

@@ -20,19 +20,19 @@ describe HammerCLIForeman::Template do
     File.stubs(:read).returns("")
   end
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:provisioning_templates, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Template::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
@@ -52,11 +52,11 @@ describe HammerCLIForeman::Template do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Template::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       before(:each) do
         template = {
           'config_template' => {
@@ -71,7 +71,7 @@ describe HammerCLIForeman::Template do
       # 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
         before(:each) do
           template = {
@@ -91,25 +91,25 @@ describe HammerCLIForeman::Template do
   end
 
 
-  context "ListKindsCommand" do
+  describe "ListKindsCommand" do
     before do
       ResourceMocks.mock_action_call(:template_kinds, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Template::ListKindsCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
     end
 
   end
 
 
-  context "DumpCommand" do
+  describe "DumpCommand" do
 
     let(:cmd) { HammerCLIForeman::Template::DumpCommand.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
@@ -117,7 +117,7 @@ describe HammerCLIForeman::Template do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Template::CreateCommand.new("", ctx) }
 
@@ -125,7 +125,7 @@ describe HammerCLIForeman::Template do
       cmd.stubs(:option_template_kind_id).returns(1)
     end
 
-    context "parameters" do
+    describe "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"]
@@ -136,7 +136,7 @@ describe HammerCLIForeman::Template do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Template::UpdateCommand.new("", ctx) }
 
@@ -144,7 +144,7 @@ describe HammerCLIForeman::Template do
       cmd.stubs(:option_template_kind_id).returns(1)
     end
 
-    context "parameters" do
+    describe "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"]
       # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -153,11 +153,11 @@ describe HammerCLIForeman::Template do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Template::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

data/test/unit/user_test.rb

@@ -9,19 +9,19 @@ describe HammerCLIForeman::User do
 
   let(:cmd_module) { HammerCLIForeman::User }
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:users, :index, [])
     end
 
     let(:cmd) { cmd_module::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
       it_should_print_columns ["Id", "Login", "Name", "Email", "Admin", "Last login", "Authorized by"]
@@ -30,20 +30,20 @@ describe HammerCLIForeman::User do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
     before do
       ResourceMocks.users_show
     end
 
     let(:cmd) { cmd_module::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "login", ["--login=admin"]
       # 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", "Login", "Name", "Email", "Admin", "Effective admin"]
@@ -54,11 +54,11 @@ describe HammerCLIForeman::User do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { cmd_module::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "all required", ["--login=login", "--mail=mail", "--password=paswd", "--auth-source-id=1"]
       it_should_accept "all required plus names", ["--login=login", "--firstname=fname", "--lastname=lname", "--mail=mail", "--password=paswd", "--auth-source-id=1"]
       # it_should_fail_with "login missing", ["--firstname=fname", "--lastname=lname", "--mail=mail", "--password=paswd", "--auth-source-id=1"]
@@ -71,11 +71,11 @@ describe HammerCLIForeman::User do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { cmd_module::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "login", ["--login=admin"]
       # it_should_fail_with "id and login missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -84,7 +84,7 @@ describe HammerCLIForeman::User do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { cmd_module::UpdateCommand.new("", ctx) }
 
@@ -92,7 +92,7 @@ describe HammerCLIForeman::User do
       HammerCLIForeman::OptionSources::UserParams.any_instance.stubs(:ask_password).returns("password")
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "password and current password interactively", ["--login=jane", "--ask-password=true"]
       it_should_accept "id", ["--id=1"]
       it_should_accept "login", ["--login=admin"]

data/test/unit/usergroup_test.rb

@@ -8,21 +8,21 @@ describe HammerCLIForeman::Usergroup do
   include CommandTestHelper
 
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:usergroups, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Usergroup::ListCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept_search_params
       it_should_accept 'organization', ['--organization-id=1']
       it_should_accept 'location', ['--location-id=1']
     end
 
-    context "output" do
+    describe "output" do
       it_should_print_column "Id"
       it_should_print_column "Name"
       it_should_print_column "Admin"
@@ -30,18 +30,18 @@ describe HammerCLIForeman::Usergroup do
 
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Usergroup::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=ug"]
       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_column "Id"
         it_should_print_column "Name"
@@ -55,11 +55,11 @@ describe HammerCLIForeman::Usergroup do
     end
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Usergroup::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=ug"]
       it_should_accept "name, role ids, user group ids and user ids", ["--name=ug", "--role-ids=1,2,3", "--user-group-ids=1,2,3", "--user-ids=1,2,3"]
       it_should_accept 'organization', %w[--name=ug --organization-id=1]
@@ -67,11 +67,11 @@ describe HammerCLIForeman::Usergroup do
     end
   end
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Usergroup::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=ug"]
       it_should_accept "id", ["--id=1"]
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
@@ -79,11 +79,11 @@ describe HammerCLIForeman::Usergroup do
     end
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Usergroup::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=ug"]
       it_should_accept "id", ["--id=1"]
       it_should_accept "name and new name", ["--name=ug", "--new-name=ug2"]