hammer_cli_katello 1.1.3 → 1.2.0

data/test/functional/host/subscription/content_override_test.rb

@@ -0,0 +1,96 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../host_helpers')
+require 'hammer_cli_katello/host_subscription'
+
+describe 'host subscription content-override' do
+  include HostHelpers
+
+  before do
+    @cmd = %w(host subscription content-override)
+  end
+
+  it "attaches a content label" do
+    label = "foo"
+    value = 'default'
+    id = '20'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--value=#{value}"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['value'] == value &&
+        par['content_overrides'][0]['name'] == "enabled"
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override.\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "attaches a content label with name" do
+    label = "foo"
+    value = '1'
+    id = '20'
+    name = 'protected'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--value=#{value}",
+              "--override-name=#{name}"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['value'] == value &&
+        par['content_overrides'][0]['name'] == name
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override.\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "removes override" do
+    label = "foo"
+    id = '20'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--remove"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['remove'] == true &&
+        par['content_overrides'][0]['name'] == "enabled"
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override.\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "removes override with name" do
+    label = "foo"
+    id = '20'
+    name = 'protected'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--override-name=#{name}", "--remove"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['remove'] == true &&
+        par['content_overrides'][0]['name'] == name
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override.\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "validation fails on no override value or remove" do
+    api_expects_no_call
+    label = "foo"
+    id = '20'
+    name = 'protected'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--override-name=#{name}"]
+    result = run_cmd(@cmd + params)
+
+    assert(result.err[/At least one of options --remove, --value is required/],
+               "Remove or Value must be provided")
+  end
+end

data/test/functional/host/subscription/product_content_test.rb

@@ -0,0 +1,27 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../host_helpers')
+require 'hammer_cli_katello/host_subscription'
+
+describe 'listing available product content' do
+  include HostHelpers
+
+  let(:host_id) { 1 }
+  let(:empty_response_table) do
+    <<eostring
+---|------|------|-----|---------|-------|------------------|---------
+ID | NAME | TYPE | URL | GPG KEY | LABEL | DEFAULT ENABLED? | OVERRIDE
+---|------|------|-----|---------|-------|------------------|---------
+eostring
+  end
+
+  it "lists content available for a host" do
+    ex = api_expects(:host_subscriptions, :product_content) do |p|
+      p['id'] = host_id
+    end
+    ex.returns(index_response([]))
+    assert_cmd(
+      success_result(empty_response_table),
+      run_cmd(%w(host subscription product-content --host-id 1))
+    )
+  end
+end

data/test/functional/host/subscription/register_test.rb

@@ -0,0 +1,53 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../../content_view/content_view_helpers')
+require File.join(File.dirname(__FILE__), '../../organization/organization_helpers')
+require File.join(File.dirname(__FILE__),
+                  '../../lifecycle_environment/lifecycle_environment_helpers')
+
+describe 'host subscription register' do
+  include OrganizationHelpers
+  include ContentViewHelpers
+  include LifecycleEnvironmentHelpers
+
+  before do
+    @cmd = %w(host subscription register)
+  end
+
+  it "subscribe a host" do
+    params = ['--name=trump.wall.com', '--content-view-id=1', '--lifecycle-environment-id=2']
+    ex = api_expects(:host_subscriptions, :create, 'Host subscription register') do |par|
+      par['name'] == 'trump.wall.com' && par['content_view_id'] == 1 &&
+        par['lifecycle_environment_id'] == 2
+    end
+    ex.returns({})
+
+    expected_result = success_result(
+      'Host successfully registered.
+'
+    )
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "resolves ids from names" do
+    params = ['--name=trump.wall.com', '--content-view=someview', '--lifecycle-environment=someenv',
+              '--organization=trumporg']
+
+    api_expects(:host_subscriptions, :create, 'Host subscription register') do |par|
+      par['name'] == 'trump.wall.com' && par['content_view_id'] == 1 &&
+        par['lifecycle_environment_id'] == 2
+    end
+
+    expect_organization_search('trumporg', 3)
+    expect_content_view_search(3, 'someview', 1)
+    expect_lifecycle_environment_search(3, 'someenv', 2)
+
+    expected_result = success_result(
+      'Host successfully registered.
+'
+    )
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/host/subscription/remove_test.rb

@@ -0,0 +1,52 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../host_helpers')
+
+describe 'host subscription remove' do
+  include HostHelpers
+
+  before do
+    @cmd = %w(host subscription remove)
+  end
+  it "remove a subscription to a host No Quantity" do
+    params = ['--host-id=3', '--subscription-id=100']
+    ex = api_expects(:host_subscriptions, :remove_subscriptions, "remove subs host") do |par|
+      par['host_id'] == 3 && par[:subscriptions][0][:id].to_s == '100'
+    end
+    ex.returns({})
+
+    expected_result = success_result(
+      'Subscription removed from the host successfully.
+'
+    )
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "remove a subscription to a host with Quantity" do
+    params = ['--host-id=3', '--subscription-id=100', "--quantity=10"]
+    ex = api_expects(:host_subscriptions, :remove_subscriptions, "remove subs host") do |par|
+      par['host_id'] == 3 && par[:subscriptions][0][:id].to_s == '100' &&
+        par[:subscriptions][0][:quantity].to_s == '10'
+    end
+    ex.returns({})
+
+    expected_result = success_result(
+      'Subscription removed from the host successfully.
+'
+    )
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "resolves id from name" do
+    params = ['--host=boo', '--subscription-id=100']
+    api_expects(:host_subscriptions, :remove_subscriptions, "remove subs host") do |par|
+      par['host_id'].to_s == "3" && par[:subscriptions][0][:id].to_s == '100'
+    end
+    expect_host_search('boo', '3')
+
+    run_cmd(@cmd + params)
+  end
+end

data/test/functional/host/subscription/unregister_test.rb

@@ -0,0 +1,37 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../host_helpers')
+
+describe 'host subscription unregister' do
+  include HostHelpers
+
+  before do
+    @cmd = %w(host subscription unregister)
+  end
+
+  it "unregisters the host" do
+    params = ['--host-id=3']
+    ex = api_expects(:host_subscriptions, :destroy, 'Host unregister') do |par|
+      par['host_id'] == 3
+    end
+    ex.returns({})
+
+    expected_result = success_result(
+      'Host unregistered.
+'
+    )
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "resolves id from name" do
+    params = ['--host=host1']
+
+    api_expects(:host_subscriptions, :destroy, 'Host unregister') do |par|
+      par['host_id'].to_i == 3
+    end
+    expect_host_search('host1', '3')
+
+    run_cmd(@cmd + params)
+  end
+end

data/test/functional/host/traces/list_test.rb

@@ -0,0 +1,37 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+
+describe 'host trace listing' do
+  before do
+    @cmd = %w(host traces list)
+  end
+
+  let(:host_id) { 2 }
+  let(:empty_response) do
+    {
+      "total" => 0,
+      "subtotal" => 0,
+      "page" => "1",
+      "per_page" => "1000",
+      "error" => nil,
+      "search" => nil,
+      "sort" => {
+        "by" => nil,
+        "order" => nil
+      },
+      "results" => []
+    }
+  end
+
+  it 'allows listing by host' do
+    params = ["--host-id=#{host_id}"]
+    ex = api_expects(:host_tracer, :index, 'host traces list')
+
+    ex.returns(empty_response)
+    expected_result = success_result("---------|-------------|--------|-----
+TRACE ID | APPLICATION | HELPER | TYPE
+---------|-------------|--------|-----
+")
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/host/traces/resolve_test.rb

@@ -0,0 +1,31 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+
+describe 'host trace resolve' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(host traces resolve)
+  end
+
+  let(:host_id) { '2' }
+  let(:task_id) { '5' }
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'stopped'
+    }
+  end
+
+  it "resolves traces on a host" do
+    trace_ids = [3]
+    params = ["--host-id=#{host_id}", "--trace-ids=#{trace_ids}"]
+
+    ex = api_expects(:host_tracer, :resolve)
+
+    ex.returns(response)
+
+    expect_foreman_task(task_id)
+
+    run_cmd(@cmd + params)
+  end
+end

data/test/functional/host_collection/add_host_test.rb

@@ -0,0 +1,90 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::AddHostCommand do
+    describe 'handles individual host errors' do
+      it 'for successful results' do
+        api_expects(:host_collections, :add_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => ["Successfully added 2 Host(s)."],
+                                         "error" => []})
+        result = run_cmd(%w(host-collection add-host --id 1 --host-ids 2,3))
+        assert_match("The host(s) has been added", result.out)
+      end
+
+      it 'for mixed results' do
+        api_expects(:host_collections, :add_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => ["Successfully added 1 Host(s)."],
+                                         "error" => ["Host with ID 3 not found."]})
+        result = run_cmd(%w(host-collection add-host --id 1 --host-ids 2,3))
+        assert_match("Could not add host(s)", result.out)
+        assert_match("Host with ID 3 not found.", result.out)
+      end
+
+      it 'for errored results' do
+        api_expects(:host_collections, :add_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => [], "error" => [
+            "Host with ID 1 already exists in the host collection.",
+            "Host with ID 3 not found."]})
+        result = run_cmd(%w(host-collection add-host --id 1 --host-ids 2,3))
+        assert_match("Could not add host(s)", result.out)
+        assert_match("Host with ID 1 already exists 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, :add_hosts)
+      run_cmd(%w(host-collection add-host --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection add-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, :add_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection add-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, :add_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection add-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, :add_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection add-host --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/content_api_expectations.rb

@@ -0,0 +1,35 @@
+module HammerCLIKatello
+  module ContentAPIExpectations
+    ORGANIZATION_ID = 1
+    HOST_COLLECTION_ID = 3
+    HOST_COLLECTION_NAME = 'Test'.freeze
+
+    def host_collection
+      {
+        'id' => HOST_COLLECTION_ID,
+        'name' => HOST_COLLECTION_NAME
+      }
+    end
+
+    def api_expects_content_action(action, content_type, expected_params)
+      expected_params[:organization_id] ||= ORGANIZATION_ID
+      expected_params[:host_collection_id] ||= HOST_COLLECTION_ID
+
+      api_expects(:hosts_bulk_actions, action) do |p|
+        search_query = "host_collection_id=\"#{expected_params[:host_collection_id]}\""
+        p['organization_id'] == expected_params[:organization_id] &&
+          p['included'] == { search: search_query } &&
+          p['content'] == expected_params[:content] &&
+          p['content_type'] == content_type
+      end
+    end
+
+    def api_expects_collection_search
+      expectation = api_expects(:host_collections, :index) do |p|
+        p["name"] == HOST_COLLECTION_NAME &&
+          p["organization_id"] == ORGANIZATION_ID
+      end
+      expectation.returns(index_response([host_collection]))
+    end
+  end
+end

data/test/functional/host_collection/content_install_test.rb

@@ -0,0 +1,62 @@
+require_relative '../test_helper'
+require_relative './content_api_expectations'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::InstallContentBaseCommand do
+    include HammerCLIKatello::ContentAPIExpectations
+
+    def api_expects_content_install(content_type, expected_params)
+      api_expects_content_action(:install_content, content_type, expected_params)
+    end
+
+    it 'installs packages to hosts in a host collection' do
+      api_expects_content_install('package', :content => ['wget'])
+      run_cmd(%w(host-collection package install --id 3 --packages wget --organization-id 1))
+    end
+
+    it 'installs packages to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('package', :content => ['wget'])
+      run_cmd(%w(host-collection package install --name Test --packages wget --organization-id 1))
+    end
+
+    it 'installs package groups to hosts in a host collection' do
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --id 3
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'installs package groups to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --name Test
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'installs package groups to hosts in a host collection' do
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --id 3
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'installs package groups to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --name Test
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'installs erratum to hosts in a host collection' do
+      api_expects_content_install('errata', :content => ['Bird_Erratum'])
+      run_cmd(%w(host-collection erratum install --id 3 --errata Bird_Erratum --organization-id 1))
+    end
+
+    it 'installs erratum to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('errata', :content => ['Bird_Erratum'])
+      run_cmd(%w(host-collection erratum install --name Test --errata Bird_Erratum
+                 --organization-id 1))
+    end
+  end
+end

data/test/functional/host_collection/content_remove_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::RemoveContentBaseCommand do
+    include HammerCLIKatello::ContentAPIExpectations
+
+    def api_expects_content_remove(content_type, expected_params)
+      api_expects_content_action(:remove_content, content_type, expected_params)
+    end
+
+    it 'removes packages from hosts in a host collection' do
+      api_expects_content_remove('package', :content => ['wget'])
+      run_cmd(%w(host-collection package remove --id 3 --packages wget --organization-id 1))
+    end
+
+    it 'removes packages from hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_remove('package', :content => ['wget'])
+      run_cmd(%w(host-collection package remove --name Test --packages wget --organization-id 1))
+    end
+
+    it 'removes package groups from hosts in a host collection' do
+      api_expects_content_remove('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group remove --id 3
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'removes package groups from hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_remove('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group remove --name Test
+                 --package-groups birds --organization-id 1))
+    end
+  end
+end

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