hammer_cli_foreman 2.2.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")

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

@@ -93,7 +93,7 @@ describe 'realm' do
 
       api_expects(:realms, :update, 'Update a realm') do |params|
         (params['id'] == '1' &&
-         params['name'] == 'test-realm-update')
+         params['realm']['name'] == 'test-realm-update')
       end
 
       result = run_cmd(@cmd + params)
@@ -101,4 +101,3 @@ describe 'realm' do
     end
   end
 end
-

data/test/functional/role_test.rb

@@ -40,9 +40,9 @@ describe 'role' do
     it 'should clone a role by id' do
       params = ['--id=1', '--new-name=zzz']
 
-      api_expects(:roles, :clone, 'Clone role').with_params({
-        'id' => '1', 'role' => {'name' => 'zzz'}, 'name' => 'zzz'
-      })
+      api_expects(:roles, :clone, 'Clone role').with_params(
+        { 'id' => '1', 'role' => { 'name' => 'zzz' } }
+      )
 
       result = run_cmd(@cmd + params)
       assert_cmd(success_result("User role cloned.\n"), result)
@@ -51,16 +51,13 @@ describe 'role' do
     it 'should clone a role by name' do
       params = ['--name=old', '--new-name=zzz']
 
-      api_expects_search(:roles, { :name => 'old' }, 'Attempt find role').returns(
-        index_response(
-          [{
-              'name' => 'old',
-              'id' => 1
-          }]))
+      api_expects_search(:roles, { name: 'old' }, 'Attempt find role').returns(
+        index_response([{ 'name' => 'old', 'id' => 1 }])
+      )
 
-      api_expects(:roles, :clone, 'Clone role').with_params({
-        'id' => 1, 'role' => {'name' => 'zzz'}, 'name' => 'zzz'
-      })
+      api_expects(:roles, :clone, 'Clone role').with_params(
+        { 'id' => 1, 'role' => { 'name' => 'zzz' } }
+      )
 
       result = run_cmd(@cmd + params)
       assert_cmd(success_result("User role cloned.\n"), result)