hammer_cli_foreman 2.1.0 → 2.3.0

data/test/functional/compute_profile_test.rb

@@ -33,8 +33,10 @@ describe "parameters" do
           }
     end
     it 'update compute profile name' do
-      params = ['--id=1' ,'--new-name=profile2']
-      api_expects(:compute_profiles, :update, 'Update the compute profile').with_params({"name" =>"profile2"}).returns(@compute_profile)
+      params = ['--id=1', '--new-name=profile2']
+      api_expects(:compute_profiles, :update, 'Update the compute profile').with_params(
+        { 'compute_profile' => { 'name' => 'profile2' } }
+      ).returns(@compute_profile)
       result = run_cmd(@cmd + params)
       assert_cmd(success_result("Compute profile updated.\n"), result)
     end

data/test/functional/host_test.rb

@@ -315,6 +315,26 @@ describe 'host update' do
     }
   end
 
+  it 'updates with passed options only' do
+    params = ['--new-name=new-name']
+    expected_result = success_result("Host updated.\n")
+
+    api_expects(:hosts, :update, 'Update host name only').with_params(
+      'id' => '1', 'organization_id' => 1, 'location_id' => 1,
+      'host' => { 'name' => 'new-name' }
+    ).returns(
+      {
+        'id' => '1',
+        'name' => 'new-name',
+        'organization_id' => '1',
+        'location_id' => '1'
+      }
+    )
+
+    result = run_cmd(cmd + minimal_params + params)
+    assert_cmd(expected_result, result)
+  end
+
   it 'ensures helper methods are invoked' do
     params = ['--image-id=1']
     expected_result = success_result("Host updated.\n")
@@ -350,12 +370,9 @@ describe 'host update' do
     )
     api_expects(:hosts, :update, 'Update host with new org').with_params(
       'id' => '1', 'location_id' => 1, 'organization_id' => 1, 'host' => {
-        'organization_id' => '5', 'compute_attributes' => {}
-      }
+        'organization_id' => '5' }
     ) do |par|
-      par['id'] == '1' &&
-        par['host']['organization_id'] == '5' &&
-        par['host']['compute_attributes'] == {}
+      par['id'] == '1' && par['host']['organization_id'] == '5'
     end.returns(updated_host)
 
     expected_result = success_result("Host updated.\n")
@@ -373,12 +390,9 @@ describe 'host update' do
     )
     api_expects(:hosts, :update, 'Update host with new loc').with_params(
       'id' => '1', 'location_id' => 1, 'organization_id' => 1, 'host' => {
-        'location_id' => '5', 'compute_attributes' => {}
-      }
+        'location_id' => '5' }
     ) do |par|
-      par['id'] == '1' &&
-        par['host']['location_id'] == '5' &&
-        par['host']['compute_attributes'] == {}
+      par['id'] == '1' && par['host']['location_id'] == '5'
     end.returns(updated_host)
 
     expected_result = success_result("Host updated.\n")
@@ -396,12 +410,9 @@ describe 'host update' do
     )
     api_expects(:hosts, :update, 'Update host with new owner').with_params(
         'id' => '1', 'location_id' => 1, 'organization_id' => 1, 'host' => {
-        'owner_id' => '1', 'compute_attributes' => {}
-    }
+        'owner_id' => '1' }
     ) do |par|
-      par['id'] == '1' &&
-          par['host']['owner_id'] == '1' &&
-          par['host']['compute_attributes'] == {}
+      par['id'] == '1' && par['host']['owner_id'] == '1'
     end.returns(updated_host)
 
     expected_result = success_result("Host updated.\n")
@@ -428,7 +439,7 @@ describe 'host config reports' do
     end.returns(index_response([report15]))
 
     result = run_cmd(['host', 'config-reports', '--id=1'])
-    result.exit_code.must_equal HammerCLI::EX_OK
+    _(result.exit_code).must_equal HammerCLI::EX_OK
   end
 
   it 'filters reports by --name' do
@@ -437,7 +448,7 @@ describe 'host config reports' do
     end.returns(index_response([report15]))
 
     result = run_cmd(['host', 'config-reports', '--name=host.example.com'])
-    result.exit_code.must_equal HammerCLI::EX_OK
+    _(result.exit_code).must_equal HammerCLI::EX_OK
   end
 
   it 'prints error or missing --id and --name' do
@@ -462,7 +473,7 @@ describe 'host config reports' do
     end.returns(index_response([report15]))
 
     result = run_cmd(['host', 'config-reports', '--name=host.example.com', '--search=reported > "2 hour ago"'])
-    result.exit_code.must_equal HammerCLI::EX_OK
+    _(result.exit_code).must_equal HammerCLI::EX_OK
   end
 end
 describe 'disassociate host from vm' do

data/test/functional/http_proxy_test.rb

@@ -64,9 +64,6 @@ describe 'httpproxy' do
     expected_result = success_result("Http proxy deleted.\n")
 
     result = run_cmd(%w(http-proxy delete --id 1))
-    # Skip this assertion until a version of awesome_print without warnings on Ruby 2.7 is available
-    if RUBY_VERSION < '2.7'
-      assert_cmd(expected_result, result)
-    end
+    assert_cmd(expected_result, result)
   end
 end

data/test/functional/location_test.rb

@@ -139,3 +139,137 @@ describe "parameters" do
     end
   end
 end
+
+describe 'create' do
+  before do
+    @cmd = %w(location create)
+  end
+
+  it 'should print error missing argument name' do
+    expected_result = "Could not create the location:\n  Missing arguments for '--name'.\n"
+
+    api_expects(:locations, :index)
+
+    result = run_cmd(@cmd)
+    assert_match(expected_result, result.err)
+  end
+
+  it 'should create a location' do
+    params = ['--name=test-location']
+
+    api_expects(:locations, :index)
+    api_expects(:locations, :create, 'Create a locations') do |params|
+      (params['location']['name'] == 'test-location')
+    end
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(success_result("Location created.\n"), result)
+  end
+end
+
+describe 'delete' do
+  before do
+    @cmd = %w(location delete)
+  end
+
+  it 'should print error missing argument id' do
+    expected_result = "Could not delete the location:\n  Missing arguments for '--id'.\n"
+
+    api_expects(:locations, :index)
+
+    result = run_cmd(@cmd)
+    assert_match(expected_result, result.err)
+  end
+
+  it 'should delete a location' do
+    params = ['--id=1']
+
+    api_expects(:locations, :index)
+    api_expects(:locations, :destroy, 'Delete a location').with_params(id: '1')
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(success_result("Location deleted.\n"), result)
+  end
+end
+
+describe 'info' do
+  before do
+    @cmd = ['location', 'info']
+    @location = {
+        id: 1,
+        title: 'Default Location',
+        name: 'Default Location',
+        users: [],
+        smart_proxies: [],
+        subnets: [],
+        compute_resources: [],
+        media: [],
+        ptables: [],
+        provisioning_templates: [],
+        domains: [],
+        realms: [],
+        environments: [],
+        hostgroups: [],
+        organizations: [],
+        parameters: []
+    }
+  end
+
+  it 'should return the info of a location' do
+    params = ['--id', 1]
+    api_expects(:locations, :index)
+    api_expects(:locations, :show, 'Info location').returns(@location)
+
+    output = OutputMatcher.new([
+                                 'Id:                 1',
+                                 'Title:              Default Location',
+                                 'Name:               Default Location',
+                                 'Users:',
+                                 '',
+                                 'Smart proxies:',
+                                 '',
+                                 'Subnets:',
+                                 '',
+                                 'Compute resources:',
+                                 '',
+                                 'Installation media:',
+                                 '',
+                                 'Templates:',
+                                 '',
+                                 'Partition Tables:',
+                                 '',
+                                 'Domains:',
+                                 '',
+                                 'Realms:',
+                                 '',
+                                 'Environments:',
+                                 '',
+                                 'Hostgroups:',
+                                 '',
+                                 'Parameters:',
+    ])
+
+    expected_result = success_result(output)
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end
+
+describe 'update' do
+  before do
+    @cmd = ['location', 'update']
+  end
+
+  it 'should update a location' do
+    params = ['--id=1', '--new-name=Default Location test']
+
+    api_expects(:locations, :index)
+    api_expects(:locations, :update, 'Update a location') do |params|
+      (params['location']['id'] == '1')
+      (params['location']['name'] == 'Default Location test')
+    end
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(success_result("Location updated.\n"), result)
+  end
+end

data/test/functional/media_test.rb

@@ -0,0 +1,130 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'medium' do
+  describe 'list' do
+    before do
+      @cmd = %w[medium list]
+      @media = [{
+        id: 1,
+        name: 'CentOS mirror',
+        path: 'http://mirror.centos.org/centos/$major/os/$arch',
+        os_family: 'Redhat',
+        operating_systems: [],
+        locations: [],
+        organizations: []
+      }]
+    end
+
+    it 'should return a list of media' do
+      api_expects(:media, :index, 'List media').returns(@media)
+
+      output = IndexMatcher.new([
+                                  ['ID', 'NAME', 'PATH'],
+                                  ['1', 'CentOS mirror', 'http://mirror.centos.org/centos/$major/os/$arch']
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    before do
+      @cmd = ['medium', 'info']
+      @medium = {
+        id: 1,
+        name: 'CentOS mirror',
+        path: 'http://mirror.centos.org/centos/$major/os/$arch',
+        os_family: 'Redhat',
+        operating_systems: [],
+        locations: [],
+        organizations: []
+      }
+    end
+
+    it 'should return the info of a medium' do
+      params = ['--id', 1]
+      api_expects(:media, :show, 'Info medium').returns(@medium)
+
+      output = OutputMatcher.new([
+                                   'Id:        1',
+                                   'Name:      CentOS mirror',
+                                   'Path:      http://mirror.centos.org/centos/$major/os/$arch',
+                                   'OS Family: Redhat'
+                                 ])
+
+      expected_result = success_result(output)
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'create' do
+    before do
+      @cmd = %w(medium create)
+    end
+
+    it 'should print error missing argument name and path' do
+      expected_result = "Could not create the installation medium:\n  Missing arguments for '--name', '--path'.\n"
+
+
+      result = run_cmd(@cmd)
+      assert_match(expected_result, result.err)
+    end
+
+    it 'should create a media' do
+      params = ['--name=CentOS mirror', '--path=http://mirror.centos.org/centos/$major/os/$arch']
+
+      api_expects(:media, :create) do |params|
+        (params['medium']['name'] == 'CentOS mirror')
+        (params['medium']['path'] == 'http://mirror.centos.org/centos/$major/os/$arch')
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Installation medium created.\n"), result)
+    end
+  end
+
+  describe 'delete' do
+    before do
+      @cmd = %w(medium delete)
+    end
+
+    it 'should print error missing argument id' do
+      expected_result = "Could not delete the installation media:\n  Missing arguments for '--id'.\n"
+
+      api_expects_no_call
+
+      result = run_cmd(@cmd)
+      assert_match(expected_result, result.err)
+    end
+
+    it 'should delete a media' do
+      params = ['--id=1']
+
+      api_expects(:media, :destroy, 'Delete a media').with_params(id: '1')
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Installation medium deleted.\n"), result)
+    end
+  end
+
+  describe 'update' do
+    before do
+      @cmd = ['medium', 'update']
+    end
+
+    it 'should update a media' do
+      params = ['--id=1', '--new-name=CentOS mirror test']
+
+      api_expects(:media, :update, 'Update a media') do |params|
+        (params['medium']['id'] == '1')
+        (params['medium']['name'] == 'CentOS mirror test')
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Installation medium updated.\n"), result)
+    end
+  end
+end

data/test/functional/organization_test.rb

@@ -210,3 +210,55 @@ describe 'associating commands' do
     end
   end
 end
+
+describe 'create' do
+  before do
+    @cmd = %w(organization create)
+  end
+
+  it 'should print error missing argument name' do
+    expected_result = "Could not create the organization:\n  Missing arguments for '--name'.\n"
+
+    api_expects(:organizations, :index)
+
+    result = run_cmd(@cmd)
+    assert_match(expected_result, result.err)
+  end
+
+  it 'should create an organization' do
+    params = ['--name=test-organization']
+
+    api_expects(:organizations, :index)
+    api_expects(:organizations, :create, 'Create an organization') do |params|
+      (params['organization']['name'] == 'test-organization')
+    end
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(success_result("Organization created.\n"), result)
+  end
+end
+
+describe 'delete' do
+  before do
+    @cmd = %w(organization delete)
+  end
+
+  it 'should print error missing argument id' do
+    expected_result = "Could not delete the organization:\n  Missing arguments for '--id'.\n"
+
+    api_expects(:organizations, :index)
+
+    result = run_cmd(@cmd)
+    assert_match(expected_result, result.err)
+  end
+
+  it 'should delete an organization' do
+    params = ['--id=1']
+
+    api_expects(:organizations, :index)
+    api_expects(:organizations, :destroy, 'Delete an organization').with_params(id: '1')
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(success_result("Organization deleted.\n"), result)
+  end
+end

data/test/functional/realm_test.rb

@@ -0,0 +1,103 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'realm' do
+  describe 'list' do
+    before do
+      @cmd = %w[realm list]
+      @realms =
+        [{
+           id: 1,
+           name: 'test-realm' }]
+    end
+
+    it 'should return a list of realms' do
+      api_expects(:realms, :index, 'List realms').returns(@realms)
+
+      result = run_cmd(@cmd)
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+
+  describe 'info' do
+    before do
+      @cmd = %w[realm info]
+      @realm = {
+          id: 1,
+          name: 'test-realm'
+      }
+    end
+
+    it 'should return a realm' do
+      params = ['--id=1']
+      api_expects(:realms, :show, 'Show realm').returns(@realm)
+
+      result = run_cmd(@cmd + params)
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+
+  describe 'create' do
+    before do
+      @cmd = %w[realm create]
+    end
+
+    it 'should print error on missing --name, --realm-proxy-id, --realm-type' do
+      expected_result = "Could not create the realm:\n  Missing arguments for '--name', '--realm-proxy-id', '--realm-type'.\n"
+
+      api_expects_no_call
+      result = run_cmd(@cmd)
+      assert_match(expected_result, result.err)
+    end
+
+    it 'should create a realm' do
+      params = %w[--name=test-realm --realm-proxy-id=12345 --realm-type=FreeIPA]
+
+      api_expects(:realms, :create, 'Create a realm') do |params|
+        (params['realm']['name'] == 'test-realm' &&
+         params['realm']['realm_proxy_id'] == 12345 &&
+         params['realm']['realm_type'] == 'FreeIPA')
+      end
+
+      result = run_cmd(@cmd + params)
+
+      assert_cmd(success_result("Realm [%{name}] created.\n"), result)
+    end
+  end
+
+  describe 'delete' do
+    before do
+      @cmd = %w[realm delete]
+    end
+
+    it 'should delete a realm' do
+      params = ['--id=1']
+
+      api_expects(:realms, :destroy, 'Delete realm').with_params(id: '1')
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Realm [%{name}] deleted.\n"), result)
+    end
+  end
+
+  describe 'update' do
+    before do
+      @cmd = %w[realm update]
+      @realm = {
+        id: 1,
+        name: 'test-realm-update'
+      }
+    end
+
+    it 'should update a realm' do
+      params = %w[--id=1 --new-name=test-realm-update]
+
+      api_expects(:realms, :update, 'Update a realm') do |params|
+        (params['id'] == '1' &&
+         params['realm']['name'] == 'test-realm-update')
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Realm [%{name}] updated.\n"), result)
+    end
+  end
+end