hammer_cli_katello 1.5.0 → 1.5.1

data/test/functional/host_collection/content_update_test.rb

@@ -0,0 +1,37 @@
+require_relative '../test_helper'
+require_relative './content_api_expectations'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::UpdateContentBaseCommand do
+    include HammerCLIKatello::ContentAPIExpectations
+
+    def api_expects_content_update(content_type, expected_params)
+      api_expects_content_action(:update_content, content_type, expected_params)
+    end
+
+    it 'updates packages on hosts in a host collection' do
+      api_expects_content_update('package', :content => ['wget'])
+      run_cmd(%w(host-collection package update --id 3 --packages wget --organization-id 1))
+    end
+
+    it 'updates packages on hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_update('package', :content => ['wget'])
+      run_cmd(%w(host-collection package update --name Test --packages wget --organization-id 1))
+    end
+
+    it 'updates package groups on hosts in a host collection' do
+      api_expects_content_update('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group update --id 3
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'updates package groups on hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_update('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group update --name Test
+                 --package-groups birds --organization-id 1))
+    end
+  end
+end

data/test/functional/host_collection/copy_test.rb

@@ -0,0 +1,64 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::CopyCommand do
+    it 'requires a new name' do
+      result = run_cmd(%w(host-collection copy --id 1))
+      expected_error = "Option '--new-name' is required."
+      assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+      assert_equal(result.err[/#{expected_error}/], expected_error)
+    end
+
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :copy)
+      run_cmd(%w(host-collection copy --id 1 --new-name foo))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection copy --name hc1 --new-name foo))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :copy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection copy --name hc1 --organization-id 1 --new-name foo))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :copy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection copy --name hc1 --organization org1 --new-name foo))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :copy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection copy --name hc1 --organization-label org1 --new-name foo))
+    end
+  end
+end

data/test/functional/host_collection/create_test.rb

@@ -0,0 +1,54 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::CreateCommand do
+    it 'requires organization options' do
+      result = run_cmd(%w(host-collection create --name hc1))
+      expected_error = "Could not find organization"
+      assert_equal(result.exit_code, HammerCLI::EX_SOFTWARE)
+      assert_equal(result.err[/#{expected_error}/], expected_error)
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :create) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :create) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization org1))
+    end
+
+    it 'allows unlimited-hosts flag with no arguments' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :create) do |par|
+        par['unlimited_hosts'] == true
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization org1 --unlimited-hosts))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :create) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/delete_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::DeleteCommand do
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :destroy)
+      run_cmd(%w(host-collection delete --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection delete --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :destroy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection delete --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :destroy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection delete --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :destroy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection delete --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/hosts_test.rb

@@ -0,0 +1,39 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::HostsCommand do
+    it 'allows host collection id' do
+      api_expects(:hosts, :index) do |par|
+        par['search'] == "host_collection_id=1"
+      end
+
+      run_cmd(%w(host-collection hosts --id 1))
+    end
+
+    it 'allows host collection name' do
+      ex = api_expects(:host_collections, :index)
+      ex.returns(index_response([{'id' => 2}]))
+
+      api_expects(:hosts, :index) do |par|
+        par['search'] == "host_collection_id=2"
+      end
+
+      run_cmd(%w(host-collection hosts --name collection --organization-id 1))
+    end
+
+    it 'requires organization with host collection name' do
+      result = run_cmd(%w(host-collection hosts --name collection))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'requires host collection name or id' do
+      result = run_cmd(%w(host-collection hosts))
+      expected_error = "Error: At least one of options --id, --name is required"
+      assert_equal(HammerCLI::EX_USAGE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+  end
+end

data/test/functional/host_collection/info_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::InfoCommand do
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :show)
+      run_cmd(%w(host-collection info --id 1))
+    end
+
+    it 'requires organization options if id is not specified' do
+      result = run_cmd(%w(host-collection info --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :show) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection info --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :show) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection info --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :show) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection info --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/list_test.rb

@@ -0,0 +1,63 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::ListCommand do
+    it 'does not require organization options' do
+      api_expects(:host_collections, :index)
+
+      run_cmd(%w(host-collection list))
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection list --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection list --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection list --organization-label org1))
+    end
+
+    it 'allows host id' do
+      api_expects(:host_collections, :index) do |par|
+        par['host_id'] == 1
+      end
+
+      run_cmd(%w(host-collection list --host-id 1 --organization-id 1))
+    end
+
+    it 'allows host name' do
+      ex = api_expects(:hosts, :index) do |par|
+        par[:search] == "name = \"host1\""
+      end
+      ex.returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) do |par|
+        par['host_id'] == 1
+      end
+
+      run_cmd(%w(host-collection list --host host1 --organization-id 1))
+    end
+  end
+end

data/test/functional/host_collection/remove_host_test.rb

@@ -0,0 +1,90 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::RemoveHostCommand do
+    describe 'handles individual host errors' do
+      it 'for successful results' do
+        api_expects(:host_collections, :remove_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => ["Successfully removed 2 Host(s)."],
+                                         "error" => []})
+        result = run_cmd(%w(host-collection remove-host --id 1 --host-ids 2,3))
+        assert_match("The host(s) has been removed", result.out)
+      end
+
+      it 'for mixed results' do
+        api_expects(:host_collections, :remove_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => ["Successfully removed 1 Host(s)."],
+                                         "error" => ["Host with ID 3 not found."]})
+        result = run_cmd(%w(host-collection remove-host --id 1 --host-ids 2,3))
+        assert_match("Could not remove host(s)", result.out)
+        assert_match("Host with ID 3 not found.", result.out)
+      end
+
+      it 'for errored results' do
+        api_expects(:host_collections, :remove_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => [], "error" => [
+            "Host with ID 1 does not exist in the host collection.",
+            "Host with ID 3 not found."]})
+        result = run_cmd(%w(host-collection remove-host --id 1 --host-ids 2,3))
+        assert_match("Could not remove host(s)", result.out)
+        assert_match("Host with ID 1 does not exist in the host collection.", result.out)
+        assert_match("Host with ID 3 not found.", result.out)
+      end
+    end
+
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :remove_hosts)
+      run_cmd(%w(host-collection remove-host --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection remove-host --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :remove_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection remove-host --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :remove_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection remove-host --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :remove_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection remove-host --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/update_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::UpdateCommand do
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :update)
+      run_cmd(%w(host-collection update --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection update --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :update) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection update --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :update) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection update --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :update) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection update --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/hostgroup/create_test.rb

@@ -0,0 +1,112 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/hostgroup'
+
+module HammerCLIForeman
+  describe Hostgroup do
+    # These tests are only for the extensions Katello adds to the hostgroup command
+    # See hammer-cli-foreman for the core hostgroup tests
+    describe CreateCommand do
+      it 'allows content source id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['content_source_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-source-id 1))
+      end
+
+      it 'allows content source name' do
+        content_source_name = "life_of_cs"
+        content_source_id = 111
+        hg_name = "mercury"
+
+        api_expects(:smart_proxies, :index).
+          with_params(:search => "name = \"#{content_source_name}\"").
+          returns(index_response([{'id' => content_source_id}]))
+
+        api_expects(:hostgroups, :create).
+          with_params('hostgroup' => {'name' => hg_name,
+                                      'content_source_id' => content_source_id
+                                     })
+
+        cmd = "hostgroup create --name #{hg_name} --content-source #{content_source_name}"
+        run_cmd(cmd.split)
+      end
+
+      it 'allows content view id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-view-id 1))
+      end
+
+      it 'allows content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p[:search] = "name = \"cv1\"" && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['content_view_id'] == 1 && p['hostgroup']['name'] == 'hg1'
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-view cv1 --query-organization-id 1))
+      end
+
+      it 'requires organization options to resolve content view name' do
+        api_expects_no_call
+        result = run_cmd(%w(hostgroup create --name hg1 --content-view cv1))
+        assert_match(/--query-organization/, result.err)
+      end
+
+      it 'allows lifecycle environment id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['lifecycle_environment_id'] == 1 &&
+            p['hostgroup']['organization_ids'] == %w(1 2)
+        end
+        run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment-id 1 --organization-ids 1,2))
+      end
+
+      it 'allows lifecycle environment name' do
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p['name'] == 'le1' && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['lifecycle_environment_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment le1
+                   --query-organization-id 1 --organization-ids 1,2))
+      end
+
+      it 'requires organization options to resolve lifecycle environment name' do
+        api_expects_no_call
+        result = run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment le1))
+        assert_match(/--query-organization/, result.err)
+      end
+
+      it 'allows kickstart repository name' do
+        env_id = 100
+        cv_id = 200
+        repo_name = "life_of_kickstart"
+        repo_id = 111
+        hg_name = "mercury"
+
+        api_expects(:repositories, :index).
+          with_params('name' => repo_name,
+                      'environment_id' => env_id,
+                      'content_view_id' => cv_id).
+          returns(index_response([{'id' => repo_id}]))
+
+        expected_params = {'hostgroup' => {
+          'name' => hg_name,
+          'lifecycle_environment_id' => env_id,
+          'content_view_id' => cv_id,
+          'kickstart_repository_id' => repo_id
+        }
+                          }
+        api_expects(:hostgroups, :create).with_params(expected_params)
+
+        cmd = "hostgroup create --name #{hg_name} --lifecycle-environment-id #{env_id}"\
+              " --content-view-id #{cv_id} --kickstart-repository #{repo_name}"
+        run_cmd(cmd.split)
+      end
+    end
+  end
+end