hammer_cli_foreman 3.7.0 → 3.8.0

data/test/unit/partition_table_test.rb

@@ -14,19 +14,19 @@ describe HammerCLIForeman::PartitionTable do
     ::File.stubs(:read).returns("FILE_CONTENT")
   end
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:ptables, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::PartitionTable::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
@@ -36,18 +36,18 @@ describe HammerCLIForeman::PartitionTable do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::PartitionTable::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=ptable"]
 
       # 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", "OS Family", "Created at", "Updated at"]
@@ -65,11 +65,11 @@ describe HammerCLIForeman::PartitionTable do
   end
 
 
-  context "DumpCommand" do
+  describe "DumpCommand" do
 
     let(:cmd) { HammerCLIForeman::PartitionTable::DumpCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=ptable"]
       # it_should_fail_with "id or name missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -86,7 +86,7 @@ describe HammerCLIForeman::PartitionTable do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::PartitionTable::CreateCommand.new("", ctx) }
 
@@ -94,7 +94,7 @@ describe HammerCLIForeman::PartitionTable do
       cmd.stubs(:template_kind_id).returns(1)
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, file, os family", ["--name=tpl", "--file=~/table.sh", "--os-family=RedHat"]
       # it_should_fail_with "name missing", ["--file=~/table.sh", "--os-family=RedHat"]
       # it_should_fail_with "file missing", ["--name=tpl", "--os-family=RedHat"]
@@ -108,11 +108,11 @@ describe HammerCLIForeman::PartitionTable do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::PartitionTable::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id, new-name, file, type, audit comment, os ids", ["--id=83", "--new-name=ptable","--file=~/table.sh", "--os-family=RedHat"]
       it_should_accept "name, new-name, file, type, audit comment, os ids", ["--name=ptable", "--new-name=ptable2", "--file=~/table.sh", "--os-family=RedHat"]
       # it_should_fail_with "id and name missing", ["--new-name=ptable","--file=~/table.sh", "--os-family=RedHat"]
@@ -126,11 +126,11 @@ describe HammerCLIForeman::PartitionTable do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::PartitionTable::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=ptable"]
       # it_should_fail_with "id or name missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver

data/test/unit/realm_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::Realm do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:realms, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Realm::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::Realm do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Realm::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 ["--id=1"] do
         it_should_print_n_records 1
         it_should_print_columns ["Id", "Name", "Created at", "Updated at"]
@@ -49,22 +49,22 @@ describe HammerCLIForeman::Realm do
 
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Realm::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, type, proxy", ["--name=realm", "--realm-type=FreeIPA", "--realm-proxy-id=1"]
     end
 
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Realm::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=realm"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -73,11 +73,11 @@ describe HammerCLIForeman::Realm do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Realm::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=realm", "--new-name=realm2"]
       it_should_accept "id", ["--id=1", "--new-name=realm2"]
       # it_should_fail_with "no params", []

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