hammer_cli_foreman 3.6.0 → 3.8.0

data/test/unit/auth_source_external.rb

@@ -7,20 +7,20 @@ describe HammerCLIForeman::AuthSourceExternal do
 
   include CommandTestHelper
 
-  context 'ListCommand' do
+  describe 'ListCommand' do
     before :each do
       ResourceMocks.auth_source_external_index
     end
 
     let(:cmd) { HammerCLIForeman::AuthSourceExternal::ListCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'no arguments'
       it_should_accept 'per page', ['--per-page=1']
       it_should_accept 'page', ['--page=2']
     end
 
-    context 'output' do
+    describe 'output' do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records 1
@@ -29,10 +29,10 @@ describe HammerCLIForeman::AuthSourceExternal do
     end
   end
 
-  context 'UpdateCommand' do
+  describe 'UpdateCommand' do
     let(:cmd) { HammerCLIForeman::AuthSourceExternal::UpdateCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'name', ['--name=External', '--new-name=auth-external-1']
       it_should_accept 'id', ['--id=11', '--new-name=auth-external-2']
     end

data/test/unit/auth_source_ldap_test.rb

@@ -7,20 +7,20 @@ describe HammerCLIForeman::AuthSourceLdap do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before :each do
       ResourceMocks.auth_source_ldap_index
     end
 
     let(:cmd) { HammerCLIForeman::AuthSourceLdap::ListCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "no arguments"
       it_should_accept "per page", ["--per-page=1"]
       it_should_accept "page", ["--page=2"]
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records 1
@@ -34,17 +34,17 @@ describe HammerCLIForeman::AuthSourceLdap do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::AuthSourceLdap::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 ["Server", "Account", "Attribute mappings", "Locations", "Organizations"]
@@ -54,11 +54,11 @@ describe HammerCLIForeman::AuthSourceLdap do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::AuthSourceLdap::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "all required params", ["--name=arch", "--host=my.host"]
       # it_should_fail_with "name missing", []
       # TODO: temporarily disabled, parameters are checked in the api
@@ -67,11 +67,11 @@ describe HammerCLIForeman::AuthSourceLdap do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::AuthSourceLdap::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
@@ -80,11 +80,11 @@ describe HammerCLIForeman::AuthSourceLdap do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::AuthSourceLdap::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", [] # TODO: temporarily disabled, parameters are checked in the id resolver

data/test/unit/bookmark_test.rb

@@ -8,20 +8,20 @@ require 'hammer_cli_foreman/bookmark'
 describe HammerCLIForeman::Bookmark do
   include CommandTestHelper
 
-  context 'ListCommand' do
+  describe 'ListCommand' do
     before :each do
       ResourceMocks.bookmarks
     end
 
     let(:cmd) { HammerCLIForeman::Bookmark::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
@@ -35,17 +35,17 @@ describe HammerCLIForeman::Bookmark do
     end
   end
 
-  context 'InfoCommand' do
+  describe 'InfoCommand' do
     let(:cmd) { HammerCLIForeman::Bookmark::InfoCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'id', ['--id=1']
       it_should_accept 'name', ['--name=active']
       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'
@@ -59,10 +59,10 @@ describe HammerCLIForeman::Bookmark do
     end
   end
 
-  context 'CreateCommand' do
+  describe 'CreateCommand' do
     let(:cmd) { HammerCLIForeman::Bookmark::CreateCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'name, public, controller, query, organization, location',
                        ['--name=active', '--public=1', '--controller=hosts', '--organization-id=1', '--location-id=1',
                         '--query=last_report > "35 minutes ago" and (status.applied > 0 or status.restarted > 0)']
@@ -70,10 +70,10 @@ describe HammerCLIForeman::Bookmark do
 
   end
 
-  context 'DeleteCommand' do
+  describe 'DeleteCommand' do
     let(:cmd) { HammerCLIForeman::Bookmark::DeleteCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'id', ['--id=1']
       it_should_accept 'name', ['--name=active']
       it_should_accept 'organization', %w[--id=1 --organization-id=1]
@@ -81,10 +81,10 @@ describe HammerCLIForeman::Bookmark do
     end
   end
 
-  context 'UpdateCommand' do
+  describe 'UpdateCommand' do
     let(:cmd) { HammerCLIForeman::Bookmark::UpdateCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'id', ['--id=1']
       it_should_accept 'name', ['--name=active']
       it_should_accept 'name, public, controller, query, organization, location',

data/test/unit/commands_test.rb

@@ -7,7 +7,7 @@ describe HammerCLIForeman do
     HammerCLI::Settings.load({:_params => {:interactive => false}})
   end
 
-  context "collection_to_common_format" do
+  describe "collection_to_common_format" do
 
     let(:kind) { { "name" => "PXELinux", "id" => 1 } }
 
@@ -57,7 +57,7 @@ describe HammerCLIForeman do
   end
 
 
-  context "record_to_common_format" do
+  describe "record_to_common_format" do
 
     let(:arch) { { "name" => "x86_64", "id" => 1 } }
 
@@ -79,7 +79,7 @@ describe HammerCLIForeman do
     end
   end
 
-  context "Create command" do
+  describe "Create command" do
     it "should format created entity in csv output" do
       ResourceMocks.mock_action_call(:architectures, :create, {
           "architecture" => {
@@ -91,12 +91,12 @@ describe HammerCLIForeman do
           }
       })
       arch = HammerCLIForeman::Architecture::CreateCommand.new("", { :adapter => :csv, :interactive => false })
-      out, err = capture_io { arch.run(["--name='i386'"]) }
+      out, _ = capture_io { arch.run(["--name='i386'"]) }
       _(out).must_match("Message,Id,Name\nArchitecture created.,3,i386\n")
     end
   end
 
-  context "AddAssociatedCommand" do
+  describe "AddAssociatedCommand" do
     it "should associate resource" do
       ResourceMocks.mock_action_calls(
           [:organizations, :show, { "id" => 1, "domain_ids" => [2] }],
@@ -132,7 +132,7 @@ describe HammerCLIForeman do
     end
   end
 
-  context "ListSearchCommand" do
+  describe "ListSearchCommand" do
     it "should find correct results" do
       ResourceMocks.mock_action_calls(
         [:hosts, :index, [{ "id" => 2, "name" => "random-host",  "ip" => "192.168.100.112", "mac" => "6e:4b:3c:2c:8a:0a" }]],
@@ -151,7 +151,7 @@ describe HammerCLIForeman do
         end
       end
       comm = DomainOuter::HostsCommand.new("", { :adapter => :csv, :interactive => false })
-      out, err = capture_io { comm.run(["--id=5"]) }
+      out, _ = capture_io { comm.run(["--id=5"]) }
       _(out).must_equal "Id,Name,Operating System,Host Group,IP,MAC\n2,random-host,,,192.168.100.112,6e:4b:3c:2c:8a:0a\n"
     end
   end
@@ -180,7 +180,7 @@ describe HammerCLIForeman::Command do
         HammerCLIForeman.foreman_api_connection.api.resource(:locations)
       )
     )
-    out, err = capture_io do
+    _, err = capture_io do
       _(com.run(['--location', 'loc'])).wont_equal HammerCLI::EX_OK
     end
     _(err).must_equal "Error: Could not find location, please set one of options --location-id, --location, --location-title.\n"
@@ -203,7 +203,7 @@ describe HammerCLIForeman::Command do
 
     ResourceMocks.mock_action_call(:domains, :index, [])
 
-    out, err = capture_io do
+    _out, _err = capture_io do
       _(com.run([])).must_equal HammerCLI::EX_OK
     end
 

data/test/unit/common_parameter_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::CommonParameter do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:common_parameters, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::CommonParameter::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,21 +29,21 @@ describe HammerCLIForeman::CommonParameter do
   end
 
 
-  context "SetCommand" do
+  describe "SetCommand" do
     before do
       ResourceMocks.common_parameter_list
     end
 
     let(:cmd) { HammerCLIForeman::CommonParameter::SetCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, value, parameter-type and hidden-value", ["--name=param", "--value=val", "--parameter-type=string", "--hidden-value=true"]
       # it_should_fail_with "name missing", ["--value=val"]
       # it_should_fail_with "value missing", ["--name=param"]
       # TODO: temporarily disabled, parameters are checked by the api
     end
 
-    context "adding params" do
+    describe "adding params" do
       before :each do
         ResourceMocks.mock_action_calls(
           [:common_parameters, :index, []],
@@ -56,7 +56,7 @@ describe HammerCLIForeman::CommonParameter do
       end
     end
 
-    context "updating params" do
+    describe "updating params" do
       before :each do
         ResourceMocks.mock_action_calls(
           [:common_parameters, :index, [{'name' => 'param', 'value' => 'test'}]],
@@ -70,7 +70,7 @@ describe HammerCLIForeman::CommonParameter do
       end
     end
 
-    context "adding params with parameter type" do
+    describe "adding params with parameter type" do
       before :each do
         ResourceMocks.mock_action_calls(
           [:common_parameters, :index, []],
@@ -85,11 +85,11 @@ describe HammerCLIForeman::CommonParameter do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::CommonParameter::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=arch"]
       # it_should_fail_with "name missing", []
       # TODO: temporarily disabled, parameters are checked in the id resolver

data/test/unit/compute_profile_test.rb

@@ -8,20 +8,20 @@ require 'hammer_cli_foreman/compute_profile'
 describe HammerCLIForeman::ComputeProfile do
   include CommandTestHelper
 
-  context 'ListCommand' do
+  describe 'ListCommand' do
     before :each do
       ResourceMocks.compute_profiles
     end
 
     let(:cmd) { HammerCLIForeman::ComputeProfile::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::ComputeProfile do
     end
   end
 
-  context 'InfoCommand' do
+  describe 'InfoCommand' do
     let(:cmd) { HammerCLIForeman::ComputeProfile::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'
@@ -52,10 +52,10 @@ describe HammerCLIForeman::ComputeProfile do
     end
   end
 
-  context 'CreateCommand' do
+  describe 'CreateCommand' do
     let(:cmd) { HammerCLIForeman::ComputeProfile::CreateCommand.new('', ctx) }
 
-    context 'parameters' do
+    describe 'parameters' do
       it_should_accept 'name', ['--name=test']
       it_should_accept 'organization', %w[--name=test --organization-id=1]
       it_should_accept 'location', %w[--name=test --location-id=1]
@@ -63,10 +63,10 @@ describe HammerCLIForeman::ComputeProfile do
 
   end
 
-  context 'DeleteCommand' do
+  describe 'DeleteCommand' do
     let(:cmd) { HammerCLIForeman::ComputeProfile::DeleteCommand.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]
@@ -74,10 +74,10 @@ describe HammerCLIForeman::ComputeProfile do
     end
   end
 
-  context 'UpdateCommand' do
+  describe 'UpdateCommand' do
     let(:cmd) { HammerCLIForeman::ComputeProfile::UpdateCommand.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]

data/test/unit/compute_resource_test.rb

@@ -8,19 +8,19 @@ describe HammerCLIForeman::ComputeResource do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:compute_resources, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::ComputeResource::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,20 +30,20 @@ describe HammerCLIForeman::ComputeResource do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
     before do
       ResourceMocks.compute_resource_show
     end
 
     let(:cmd) { HammerCLIForeman::ComputeResource::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 ["Name", "Id", "Provider", "Url"]
@@ -53,11 +53,11 @@ describe HammerCLIForeman::ComputeResource do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::ComputeResource::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, url, provider", ["--name=arch", "--url=http://some.org", "--provider=Libvirt"]
       # it_should_fail_with "name missing", ["--url=http://some.org", "--provider=Libvirt"]
       # it_should_fail_with "url missing", ["--name=arch", "--provider=Libvirt"]
@@ -68,11 +68,11 @@ describe HammerCLIForeman::ComputeResource do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::ComputeResource::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
@@ -81,11 +81,11 @@ describe HammerCLIForeman::ComputeResource do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::ComputeResource::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", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -94,19 +94,19 @@ describe HammerCLIForeman::ComputeResource do
 
   end
 
-  context "AvailableClustersCommand" do
+  describe "AvailableClustersCommand" do
     before do
       ResourceMocks.compute_resources_available_clusters
     end
 
     let(:cmd) { HammerCLIForeman::ComputeResource::AvailableClustersCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       it_should_accept "name", ["--name=domain-c7"]
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:available_clusters)) }
 
       with_params ["--name=testcr"] do
@@ -116,21 +116,21 @@ describe HammerCLIForeman::ComputeResource do
     end
   end
 
-  context "AvailableNetworksCommand" do
+  describe "AvailableNetworksCommand" do
     before do
       ResourceMocks.compute_resources_available_networks
     end
 
     let(:cmd) { HammerCLIForeman::ComputeResource::AvailableNetworksCommand.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 params", [] # TODO: temporarily disabled, parameters are checked in the id resolver
       # it_should_fail_with "name or id missing", ["--new-name=arch2"] # TODO: temporarily disabled, parameters are checked in the id resolver
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:available_networks)) }
 
       with_params ["--name=testcr"] do

data/test/unit/config_report_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::ConfigReport do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:config_reports, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::ConfigReport::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
@@ -38,11 +38,11 @@ describe HammerCLIForeman::ConfigReport do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::ConfigReport::InfoCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
     end
@@ -50,11 +50,11 @@ describe HammerCLIForeman::ConfigReport do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::ConfigReport::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