hammer_cli_foreman 3.7.0 → 3.8.0

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" => {
@@ -96,7 +96,7 @@ describe HammerCLIForeman do
     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" }]],

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

data/test/unit/domain_test.rb

@@ -7,19 +7,19 @@ describe HammerCLIForeman::Domain do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     before do
       ResourceMocks.mock_action_call(:domains, :index, [])
     end
 
     let(:cmd) { HammerCLIForeman::Domain::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::Domain do
   end
 
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
 
     let(:cmd) { HammerCLIForeman::Domain::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"]
@@ -50,11 +50,11 @@ describe HammerCLIForeman::Domain do
   end
 
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
 
     let(:cmd) { HammerCLIForeman::Domain::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, fullname", ["--name=domain", "--description=full_domain_name"]
       # it_should_fail_with "name missing", ["--full-name=full_domain_name"]
       # TODO: temporarily disabled, parameters are checked in the api
@@ -63,11 +63,11 @@ describe HammerCLIForeman::Domain do
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
 
     let(:cmd) { HammerCLIForeman::Domain::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=domain"]
       it_should_accept "id", ["--id=1"]
       # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
@@ -76,11 +76,11 @@ describe HammerCLIForeman::Domain do
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
 
     let(:cmd) { HammerCLIForeman::Domain::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name", ["--name=domain", "--new-name=domain2", "--description=full_domain_name"]
       it_should_accept "id", ["--id=1", "--new-name=domain2", "--description=full_domain_name"]
       # it_should_fail_with "no params", []
@@ -91,7 +91,7 @@ describe HammerCLIForeman::Domain do
   end
 
 
-  context "SetParameterCommand" do
+  describe "SetParameterCommand" do
 
     before :each do
       ResourceMocks.parameters_index
@@ -99,7 +99,7 @@ describe HammerCLIForeman::Domain do
 
     let(:cmd) { HammerCLIForeman::Domain::SetParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name, value and domain name", ["--name=name", "--value=val", "--domain=name"]
       it_should_accept "name, value and domain id", ["--name=name", "--value=val", "--domain-id=1"]
       it_should_accept "name, value, parameter type and domain name",
@@ -113,11 +113,11 @@ describe HammerCLIForeman::Domain do
   end
 
 
-  context "DeleteParameterCommand" do
+  describe "DeleteParameterCommand" do
 
     let(:cmd) { HammerCLIForeman::Domain::DeleteParameterCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "name and domain name", ["--name=param", "--domain=name"]
       it_should_accept "name and domain id", ["--name=param", "--domain-id=1"]
 

data/test/unit/exception_handler_test.rb

@@ -125,7 +125,7 @@ describe HammerCLIForeman::ExceptionHandler do
     _(err_code).must_equal HammerCLI::EX_SOFTWARE
   end
 
-  context "redirects" do
+  describe "redirects" do
     let(:response) { HammerCLIForeman.foreman_api.api.send(:create_fake_response, 301, '', 'GET', 'http://foreman.example.com/api/architectures') }
     it "should detect http to https redirection error" do
       response.headers[:location] = 'https://foreman.example.com/api/architectures'

data/test/unit/external_usergroup_test.rb

@@ -7,19 +7,19 @@ require 'hammer_cli_foreman/usergroup'
 describe HammerCLIForeman::ExternalUsergroup do
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::ListCommand.new("", ctx) }
 
     before :each do
       ResourceMocks.mock_action_call(:external_usergroups, :index, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "user group name", ["--user-group=cr"]
       it_should_accept "user group id", ["--user-group-id=1"]
     end
 
-    context "output" do
+    describe "output" do
       let(:expected_record_count) { count_records(cmd.resource.call(:index, :usergroup_id => 1)) }
 
       with_params ["--user-group-id=1"] do
@@ -31,19 +31,19 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
   end
 
-  context "InfoCommand" do
+  describe "InfoCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::InfoCommand.new("", ctx) }
 
     before :each do
       ResourceMocks.mock_action_call(:external_usergroups, :show, {})
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "user group name and external usergroup's id", ["--user-group=cr", "--id=1"]
       it_should_accept "user group id and external usergroup's id", ["--user-group-id=1", "--id=1"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--user-group-id=1", "--id=1"] do
         it_should_print_column "Name"
         it_should_print_column "Auth source"
@@ -51,19 +51,19 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
   end
 
-  context "RefreshExternalUsergroupsCommand" do
+  describe "RefreshExternalUsergroupsCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::RefreshExternalUsergroupsCommand.new("", ctx) }
 
     before :each do
       ResourceMocks.mock_action_call(:external_usergroups, :refresh, [])
     end
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "user group name and external usergroup's id", ["--user-group=cr", "--id=1"]
       it_should_accept "user group id and external usergroup's id", ["--user-group-id=1", "--id=1"]
     end
 
-    context "output" do
+    describe "output" do
       with_params ["--user-group-id=1", "--id=1"] do
         it_should_print_column "Name"
         it_should_print_column "Auth source"
@@ -71,30 +71,30 @@ describe HammerCLIForeman::ExternalUsergroup do
     end
   end
 
-  context "CreateCommand" do
+  describe "CreateCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::CreateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       # it_should_fail_with "no name", ["--auth-source-id=1", "--user-group-id=1"]
       it_should_accept "all required params", ["--name=aabbcc123", "--auth-source-id=1", "--user-group-id=1"]
     end
   end
 
 
-  context "DeleteCommand" do
+  describe "DeleteCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::DeleteCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id and user group's id", ["--id=1", "--user-group-id=1"]
       it_should_accept "id and user group's name", ["--id=1", "--user-group=ec2"]
     end
   end
 
 
-  context "UpdateCommand" do
+  describe "UpdateCommand" do
     let(:cmd) { HammerCLIForeman::ExternalUsergroup::UpdateCommand.new("", ctx) }
 
-    context "parameters" do
+    describe "parameters" do
       it_should_accept "id and user group's id", ["--id=1", "--user-group-id=1"]
       it_should_accept "id and user group's name", ["--id=1", "--user-group=ec2"]
       it_should_accept "all available params", ["--id=1", "--name=img", "--auth-source-id=1", "--user-group-id=1"]

data/test/unit/fact_test.rb

@@ -7,7 +7,7 @@ describe HammerCLIForeman::Fact do
 
   include CommandTestHelper
 
-  context "ListCommand" do
+  describe "ListCommand" do
 
     let(:cmd) { HammerCLIForeman::Fact::ListCommand.new("", ctx) }
 
@@ -15,12 +15,12 @@ describe HammerCLIForeman::Fact do
       ResourceMocks.facts_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 "Host"
       it_should_print_column "Fact"
       it_should_print_column "Value"