hammer_cli_foreman 2.1.2 → 2.4.0

data/test/functional/config_group_test.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'config_group' do
+  describe 'list' do
+    before do
+      @cmd = %w[config-group list]
+      @config_groups = [{
+                          id: 1,
+                          name: 'config-group-test',
+                        }]
+    end
+
+    it 'should return a list of config groups' do
+      api_expects(:config_groups, :index, 'List config groups').returns(@config_groups)
+
+      output = IndexMatcher.new([
+                                  %w[ID NAME],
+                                  %w[1 config-group-test]
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+
+    it 'should run list command with defaults' do
+      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+      defaults = HammerCLI::Defaults.new(
+        {
+          organization_id: {
+            provider: 'foreman'
+          },
+          location_id: {
+            provider: 'foreman'
+          }
+        }
+      )
+      defaults.stubs(:write_to_file).returns(true)
+      defaults.stubs(:providers).returns(providers)
+      api_expects(:config_groups, :index, 'List config groups').returns(@config_groups)
+
+      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+      _(result.exit_code).must_equal HammerCLI::EX_OK
+    end
+  end
+end
+
+

data/test/functional/filter_test.rb

@@ -1,18 +1,21 @@
+# frozen_string_literal: true
+
 require File.join(File.dirname(__FILE__), 'test_helper')
 
 describe 'filter' do
   def api_expects_filter_info(options)
     override = !!options[:override]
-    api_expects(:filters, :show, 'Get filter info').with_params({
-      :id => '1'
-    }).returns(@filter.merge('override?' => override))
+    api_expects(:filters, :show, 'Get filter info').with_params(
+      id: '1'
+    ).returns(@filter.merge('override?' => override))
   end
 
   def taxonomy_usage_error(action, cmd)
     usage_error_result(
       cmd,
       'Organizations and locations can be set only for overriding filters.',
-      "Could not #{action} the permission filter")
+      "Could not #{action} the permission filter"
+    )
   end
 
   def assert_update_success(result)
@@ -21,7 +24,7 @@ describe 'filter' do
 
   describe 'create' do
     before do
-      @cmd = %w(filter create)
+      @cmd = %w[filter create]
     end
 
     it 'prints error when taxonomies are used for a not-overriding filter' do
@@ -32,41 +35,105 @@ describe 'filter' do
       result = run_cmd(@cmd + params)
       assert_cmd(taxonomy_usage_error('create', @cmd), result)
     end
+
+    it 'should create a filter' do
+      params = ['--role-id=1', '--permission-ids=[1]']
+
+      api_expects(:filters, :create) do |params|
+        (params['filter']['role_id'] == 1)
+        (params['filter']['permission_ids'] == [1])
+      end
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Permission filter for [%<resource_type>s] created.\n"), result)
+    end
+  end
+
+  describe 'list' do
+    before do
+      @cmd = %w[filter list]
+      @filters = [{
+        id: 1,
+        resource_type: 'Architecture',
+        search: 'none',
+        unlimited?: true,
+        override?: false,
+        role: { name: 'Manager', id: 2, description: 'Role granting all available permissions.', origin: 'foreman' },
+        permissions: 'view_architectures'
+      }]
+    end
+
+    it 'should return a list of filters' do
+      api_expects(:filters, :index, 'List filters').returns(@filters)
+
+      output = IndexMatcher.new([
+                                  ['ID', 'RESOURCE TYPE', 'SEARCH', 'UNLIMITED?', 'OVERRIDE?', 'ROLE', 'PERMISSIONS'],
+                                  ['1', 'Architecture', 'none', 'yes', 'no', 'Manager', 'view_architectures']
+                                ])
+      expected_result = success_result(output)
+
+      result = run_cmd(@cmd)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'delete' do
+    before do
+      @cmd = %w(filter delete)
+    end
+
+    it 'should print error missing argument id' do
+      expected_result = "Could not delete the permission filter:\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 filter' do
+      params = ['--id=1']
+
+      api_expects(:filters, :destroy, 'Delete a filter').with_params(id: '1')
+
+      result = run_cmd(@cmd + params)
+      assert_cmd(success_result("Permission filter deleted.\n"), result)
+    end
   end
 
   describe 'update' do
     before do
-      @cmd = %w(filter update)
+      @cmd = %w[filter update]
       @filter = {
-        "search" => nil,
-        "resource_type" => "User",
-        "unlimited?" => false,
-        "created_at" => "2017-07-18 14:34:09 UTC",
-        "updated_at" => "2017-07-18 14:34:09 UTC",
-        "override?" => true,
-        "id" => 404,
-        "role" => {
-          "name" => "Some Role",
-          "id" => 28,
-          "description" => "Description\nof the new\nrole",
-          "origin" => nil
+        'search' => nil,
+        'resource_type' => 'User',
+        'unlimited?' => false,
+        'created_at' => '2017-07-18 14:34:09 UTC',
+        'updated_at' => '2017-07-18 14:34:09 UTC',
+        'override?' => true,
+        'id' => 404,
+        'role' => {
+          'name' => 'Some Role',
+          'id' => 28,
+          'description' => "Description\nof the new\nrole",
+          'origin' => nil
         },
-        "permissions" => [{
-            "name" => "view_users",
-            "id" => 164,
-            "resource_type" => "User"
+        'permissions' => [{
+          'name' => 'view_users',
+          'id' => 164,
+          'resource_type' => 'User'
         }],
-        "locations" => [{
-          "id" => 28,
-          "name" => "location74",
-          "title" => "location74",
-          "description" => nil
+        'locations' => [{
+          'id' => 28,
+          'name' => 'location74',
+          'title' => 'location74',
+          'description' => nil
         }],
-        "organizations" => [{
-          "id" => 27,
-          "name" => "organization74",
-          "title" => "organization74",
-          "description" => nil
+        'organizations' => [{
+          'id' => 27,
+          'name' => 'organization74',
+          'title' => 'organization74',
+          'description' => nil
         }]
       }
     end
@@ -74,13 +141,13 @@ describe 'filter' do
     it 'resets taxonomies when a filter is not-overriding' do
       params = ['--id=1']
 
-      api_expects_filter_info(:override => false)
-      api_expects(:filters, :update, 'Update the filter').with_params({
+      api_expects_filter_info(override: false)
+      api_expects(:filters, :update, 'Update the filter').with_params(
         'filter' => {
           'organization_ids' => [],
           'location_ids' => []
         }
-      }).returns(@filter)
+      ).returns(@filter)
 
       assert_update_success(run_cmd(@cmd + params))
     end
@@ -88,12 +155,12 @@ describe 'filter' do
     it 'resets taxonomies when switching a filter to not-overriding' do
       params = ['--id=1', '--override=false']
 
-      api_expects(:filters, :update, 'Update the filter').with_params({
+      api_expects(:filters, :update, 'Update the filter').with_params(
         'filter' => {
           'organization_ids' => [],
           'location_ids' => []
         }
-      }).returns(@filter)
+      ).returns(@filter)
 
       assert_update_success(run_cmd(@cmd + params))
     end
@@ -101,13 +168,13 @@ describe 'filter' do
     it 'can add taxonomies when a filter is overriding' do
       params = ['--id=1', '--organization-ids=1,2', '--location-ids=3,4']
 
-      api_expects_filter_info(:override => true)
-      api_expects(:filters, :update, 'Update the filter').with_params({
+      api_expects_filter_info(override: true)
+      api_expects(:filters, :update, 'Update the filter').with_params(
         'filter' => {
-          'organization_ids' => ['1', '2'],
-          'location_ids' => ['3', '4']
+          'organization_ids' => %w[1 2],
+          'location_ids' => %w[3 4]
         }
-      }).returns(@filter)
+      ).returns(@filter)
 
       assert_update_success(run_cmd(@cmd + params))
     end
@@ -115,12 +182,12 @@ describe 'filter' do
     it 'can add taxonomies when switching a filter to overriding' do
       params = ['--id=1', '--organization-ids=1,2', '--location-ids=3,4', '--override=true']
 
-      api_expects(:filters, :update, 'Update the filter').with_params({
+      api_expects(:filters, :update, 'Update the filter').with_params(
         'filter' => {
-          'organization_ids' => ['1', '2'],
-          'location_ids' => ['3', '4']
+          'organization_ids' => %w[1 2],
+          'location_ids' => %w[3 4]
         }
-      }).returns(@filter)
+      ).returns(@filter)
 
       assert_update_success(run_cmd(@cmd + params))
     end
@@ -128,7 +195,7 @@ describe 'filter' do
     it 'prints error when taxonomies are used on not-overriding' do
       params = ['--id=1', '--organization-ids=1,2', '--location-ids=3,4']
 
-      api_expects_filter_info(:override => false)
+      api_expects_filter_info(override: false)
 
       result = run_cmd(@cmd + params)
       assert_cmd(taxonomy_usage_error('update', @cmd), result)

data/test/functional/host_test.rb

@@ -297,6 +297,21 @@ describe "host create" do
       assert_cmd(expected_result, result)
   end
 
+  it 'should create a host with an owner' do
+    params = ['--owner-id=1']
+
+    api_expects(:hosts, :create, 'Create a host with an owner').with_params(
+      'location_id' => 1, 'organization_id' => 1, 'host' => {
+        'owner_id' => '1', 'name' => 'test'
+      }
+    ).returns(results: { 'owner_id' => '1', 'name' => 'test' })
+
+    expected_result = success_result("Host created.\n")
+
+    result = run_cmd(cmd + minimal_params_without_hostgroup + params)
+
+    assert_cmd(expected_result, result)
+  end
 end
 
 describe 'host update' do
@@ -315,6 +330,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 +385,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 +405,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 +425,27 @@ 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'
+    end.returns(updated_host)
+
+    expected_result = success_result("Host updated.\n")
+
+    result = run_cmd(cmd + minimal_params + params)
+
+    assert_cmd(expected_result, result)
+  end
+
+  it 'should update the host owner with id' do
+    params = ['--owner-id=1']
+
+    api_expects(:hosts, :update, 'Update host with new owner').with_params(
+      'id' => '1', 'location_id' => 1, 'organization_id' => 1, 'host' => {
+        '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")
@@ -415,7 +459,7 @@ end
 describe 'host config reports' do
   let(:report15) do
     {
-        "id" => 15,
+      "id" => 15,
         "host_id" => 1,
         "host_name" => "host.example.com",
         "reported_at" => "2017-11-13 03:04:53 UTC",
@@ -428,7 +472,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 +481,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 +506,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
@@ -491,3 +535,45 @@ describe 'disassociate host from vm' do
     assert_cmd(expected_result, result)
   end
 end
+
+describe 'run puppetrun for host' do
+  let(:cmd) { ['host', 'puppetrun'] }
+  let(:params) { ['--id=1'] }
+
+  it "does nothing for puppetrun" do
+    expected_result = CommandExpectation.new
+    expected_result.expected_err = "The puppetrun feature has been removed, however you can use the Remote Execution Plugin to run Puppet commands\n"
+    expected_result.expected_exit_code = HammerCLI::EX_SOFTWARE
+
+    result = run_cmd(cmd)
+    assert_cmd(expected_result, result)
+  end
+end
+
+describe 'list' do
+  before do
+    @cmd = %w[host list]
+  end
+
+  it 'should run list command with defaults' do
+    providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
+    defaults = HammerCLI::Defaults.new(
+      {
+        organization_id: {
+          provider: 'foreman'
+        },
+        location_id: {
+          provider: 'foreman'
+        }
+      }
+    )
+    defaults.stubs(:write_to_file).returns(true)
+    defaults.stubs(:providers).returns(providers)
+    api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_organization' => { 'id' => 2 } }]))
+    api_expects(:users, :index, 'Find user').with_params(search: 'login=admin').returns(index_response([{ 'default_location' => { 'id' => 1 } }]))
+    api_expects(:hosts, :index, 'List hosts').returns(index_response([{ 'id' => '42' }]))
+
+    result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
+    _(result.exit_code).must_equal HammerCLI::EX_OK
+  end
+end