hammer_cli_foreman 3.7.0 → 3.8.0

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'

data/test/unit/media_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::Medium do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:media, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Medium::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::Medium do
 
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Medium::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=medium_x"]
       # 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"
@@ -54,11 +54,11 @@ describe HammerCLIForeman::Medium do
     end
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Medium::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, path, os ids", ["--name=media", "--path=http://some.path/abc/$major/Fedora/$arch/", "--operatingsystem-ids=1,2"]
       # it_should_fail_with "name missing", ["--path=http://some.path/abc/$major/Fedora/$arch/"]
       # it_should_fail_with "path missing", ["--name=media"]
@@ -70,22 +70,22 @@ describe HammerCLIForeman::Medium do
     end
   end
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Medium::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=media"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
     end
   end
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Medium::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=medium"]
       it_should_accept "id", ["--id=1"]
       it_should_accept "os ids", ["--id=1", "--operatingsystem-ids=1,2"]

data/test/unit/model_test.rb

@@ -8,21 +8,21 @@ describe HammerCLIForeman::Model do
 
   include CommandTestHelper
 
-  context 'ListCommand' do
+  describe 'ListCommand' do
     before do
       ResourceMocks.mock_action_call(:models, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Model::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,11 +34,11 @@ describe HammerCLIForeman::Model do
   end
 
 
-  context 'InfoCommand' do
+  describe 'InfoCommand' do
 
     let(:cmd) { HammerCLIForeman::Model::InfoCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'id', ['--id=1']
       it_should_accept 'name', ['--name=model']
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
@@ -46,7 +46,7 @@ describe HammerCLIForeman::Model 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
         it_should_print_n_records 1
         it_should_print_column 'Name'
@@ -62,11 +62,11 @@ describe HammerCLIForeman::Model do
   end
 
 
-  context 'CreateCommand' do
+  describe 'CreateCommand' do
 
     let(:cmd) { HammerCLIForeman::Model::CreateCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'name, info, vendor-class, hardware-model, organization, location',
                        %w[--name=model --info=description --vendor-class=class --hardware-model=model --organization-id=1 --location-id=1]
       # it_should_fail_with "name missing", ["--info=description", "--vendor-class=class", "--hardware-model=model"]
@@ -76,11 +76,11 @@ describe HammerCLIForeman::Model do
   end
 
 
-  context 'DeleteCommand' do
+  describe 'DeleteCommand' do
 
     let(:cmd) { HammerCLIForeman::Model::DeleteCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'name', ['--name=model']
       it_should_accept 'id', ['--id=1']
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
@@ -91,11 +91,11 @@ describe HammerCLIForeman::Model do
   end
 
 
-  context 'UpdateCommand' do
+  describe 'UpdateCommand' do
 
     let(:cmd) { HammerCLIForeman::Model::UpdateCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'name', ['--name=model', '--new-name=model2', '--info=description', '--vendor-class=class', '--hardware-model=model']
       it_should_accept 'id', ['--id=1', '--new-name=model2', '--info=description', '--vendor-class=class', '--hardware-model=model']
       it_should_accept 'organization', %w[--id=1 --organization-id=1]

data/test/unit/operating_system_test.rb

@@ -7,7 +7,7 @@ describe HammerCLIForeman::OperatingSystem do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     before :each do
       ResourceMocks.operatingsystems
@@ -15,14 +15,14 @@ describe HammerCLIForeman::OperatingSystem do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::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
@@ -35,11 +35,11 @@ describe HammerCLIForeman::OperatingSystem do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "title", ["--title=Rhel 6.5"]
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
@@ -47,7 +47,7 @@ describe HammerCLIForeman::OperatingSystem 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
         it_should_print_n_records 1
         it_should_print_column "Name"
@@ -68,11 +68,11 @@ describe HammerCLIForeman::OperatingSystem do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, major, minor, family, release name, organization, location",
                        ["--name=media", "--major=1", "--minor=2", "--family=Red Hat", "--release-name=awesome", "--organization-id=1", "--location-id=1"]
       # it_should_fail_with "name missing", ["--major=1", "--minor=2", "--family=Red Hat", "--release-name=awesome"]
@@ -85,11 +85,11 @@ describe HammerCLIForeman::OperatingSystem do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "title", ["--title=Rhel 6.5"]
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
@@ -101,11 +101,11 @@ describe HammerCLIForeman::OperatingSystem do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "title", ["--title=Rhel 6.5"]
       it_should_accept "name, major, minor, family, release name, organization, location",
@@ -122,7 +122,7 @@ describe HammerCLIForeman::OperatingSystem do
   end
 
 
-  context "SetParameterCommand" do
+  describe "SetParameterCommand" do
 
     before :each do
       ResourceMocks.operatingsystems
@@ -130,7 +130,7 @@ describe HammerCLIForeman::OperatingSystem do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::SetParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, value and os id", ["--name=domain", "--value=val", "--operatingsystem-id=1"]
       it_should_accept "name, value and os title", ["--name=domain", "--value=val", "--operatingsystem=Rhel 6.5"]
       it_should_accept "name, value, type and os id", ["--name=domain", "--value=val", "--parameter-type=string", "--operatingsystem-id=1"]
@@ -143,11 +143,11 @@ describe HammerCLIForeman::OperatingSystem do
   end
 
 
-  context "DeleteParameterCommand" do
+  describe "DeleteParameterCommand" do
 
     let(:cmd) { HammerCLIForeman::OperatingSystem::DeleteParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name and os id", ["--name=domain", "--operatingsystem-id=1"]
       it_should_accept "name and os title", ["--name=domain", "--operatingsystem=Rhel 6.5"]
       # it_should_fail_with "name missing", ["--operatingsystem-id=id"]

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