hammer_cli_katello 1.4.1 → 1.5.0

data/test/functional/host_collection/content_update_test.rb

@@ -1,37 +0,0 @@
-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

@@ -1,64 +0,0 @@
-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

@@ -1,54 +0,0 @@
-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

@@ -1,57 +0,0 @@
-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

@@ -1,39 +0,0 @@
-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

@@ -1,57 +0,0 @@
-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

@@ -1,63 +0,0 @@
-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

@@ -1,90 +0,0 @@
-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

@@ -1,57 +0,0 @@
-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

@@ -1,112 +0,0 @@
-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