hammer_cli_foreman 3.7.0 → 3.9.0

data/test/unit/option_builders_test.rb

@@ -175,7 +175,7 @@ describe HammerCLIForeman::ForemanOptionBuilder do
   end
 
 
-  context "primary searchables options expansion" do
+  describe "primary searchables options expansion" do
     let(:option_switches) { container.build(@build_options).map(&:switches) }
 
     it "can remove original searchable builder" do
@@ -208,7 +208,7 @@ describe HammerCLIForeman::ForemanOptionBuilder do
 
   end
 
-  context "dependent searchables options expansion" do
+  describe "dependent searchables options expansion" do
     let(:option_switches) do
       container.build(@build_options.merge(command: Class.new(HammerCLIForeman::Command))).map(&:switches)
     end

data/test/unit/organization_test.rb

@@ -9,7 +9,7 @@ describe HammerCLIForeman::Organization do
   include CommandTestHelper
   extend ResourceDisabled
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     before :each do
       ResourceMocks.organizations_index
@@ -17,12 +17,12 @@ describe HammerCLIForeman::Organization do
 
     let(:cmd) { HammerCLIForeman::Organization::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
@@ -35,7 +35,7 @@ describe HammerCLIForeman::Organization do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     before :each do
       ResourceMocks.organizations_show
@@ -43,13 +43,13 @@ describe HammerCLIForeman::Organization do
 
     let(:cmd) { HammerCLIForeman::Organization::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_column "Name"
@@ -66,11 +66,11 @@ describe HammerCLIForeman::Organization do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Organization::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
@@ -82,11 +82,11 @@ describe HammerCLIForeman::Organization do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Organization::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=org"]
       it_should_accept "id", ["--id=1"]
 
@@ -100,11 +100,11 @@ describe HammerCLIForeman::Organization do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Organization::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", [] # TODO: temporarily disabled, parameters are checked in the id resolver

data/test/unit/output/formatters_test.rb

@@ -16,7 +16,7 @@ describe HammerCLIForeman::Output::Formatters::StructuredReferenceFormatter do
     end
   end
 
-  context 'with symbol keys' do
+  describe 'with symbol keys' do
     it 'formats name' do
       options = {:display_field_key => :server_name}
       _(formatter.format(reference, options)).must_equal('Name' => "#{reference_str_keys['server_name']}")
@@ -30,7 +30,7 @@ describe HammerCLIForeman::Output::Formatters::StructuredReferenceFormatter do
     end
   end
 
-  context 'with string keys' do
+  describe 'with string keys' do
     it 'formats name' do
       options = {:display_field_key => :server_name}
       _(formatter.format(reference_str_keys, options)).must_equal('Name' => "#{reference_str_keys['server_name']}")
@@ -62,7 +62,7 @@ describe HammerCLIForeman::Output::Formatters::ReferenceFormatter do
     _(formatter.format(nil)).must_equal ''
   end
 
-  context 'with symbol keys' do
+  describe 'with symbol keys' do
     let(:reference_sym_keys) do
       reference
     end
@@ -106,7 +106,7 @@ describe HammerCLIForeman::Output::Formatters::ReferenceFormatter do
     end
   end
 
-  context 'with string keys' do
+  describe 'with string keys' do
     let(:reference_str_keys) do
       reference.inject({}) do |new_ref, (key, value)|
         new_ref.update(key.to_s => value)

data/test/unit/param_filters_test.rb

@@ -74,7 +74,7 @@ describe HammerCLIForeman::IdParamsFilter do
       _(filter.for_action(action)).must_equal required_params
     end
 
-    context "with hash params" do
+    describe "with hash params" do
 
       let(:action) {
         stub(:params => [hash_param])
@@ -139,7 +139,7 @@ describe HammerCLIForeman::IdArrayParamsFilter do
       _(filter.for_action(action)).must_equal required_params
     end
 
-    context "with hash params" do
+    describe "with hash params" do
 
       let(:action) {
         stub(:params => [hash_param])

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