hammer_cli_katello 1.9.1 → 1.10.0

data/test/functional/host/errata/apply_test.rb

@@ -1,45 +1,15 @@
 require File.join(File.dirname(__FILE__), '../../test_helper')
-
 describe 'apply an errata' do
-  include ForemanTaskHelpers
-
   before do
     @cmd = %w(host errata apply)
   end
 
-  let(:errata_id) { "RHEA-1111:1111" }
-  let(:host_id) { 1 }
-  let(:task_id) { '5' }
-  let(:response) do
-    {
-      'id' => task_id,
-      'state' => 'stopped'
-    }
-  end
-
-  it "applies an errata to a host" do
-    params = ["--errata-ids=#{errata_id}", "--host-id=#{host_id}"]
-
-    ex = api_expects(:host_errata, :apply, 'Host errata apply') do |par|
-      par['errata_ids'] == [errata_id] && par['host_id'] == host_id
-    end
-
-    ex.returns(response)
-
-    expect_foreman_task(task_id)
-
-    run_cmd(@cmd + params)
-  end
-
-  it "applies an errata to a host with async flag" do
-    params = ["--errata-ids=#{errata_id}", "--host-id=#{host_id}", "--async"]
-
-    ex = api_expects(:host_errata, :apply, 'Host errata apply') do |par|
-      par['errata_ids'] == [errata_id] && par['host_id'] == host_id
-    end
-
-    ex.returns(response)
-
-    run_cmd(@cmd + params)
+  it "errors out on errata applies" do
+    params = ["--help"]
+    expected = "Not supported. Use the remote execution equivalent "\
+            "`hammer job-invocation create --feature katello_errata_install`.\n"
+    result = run_cmd(@cmd + params)
+    assert_exit_code_equal(HammerCLI::EX_UNAVAILABLE, result.exit_code)
+    assert_equal expected, result.out
   end
 end

data/test/functional/host_collection/content_install_test.rb

@@ -1,62 +1,21 @@
 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))
+  describe 'Install Content on a host-collection' do
+    { "package" => "katello_package_install",
+      "package-group" => "katello_group_install",
+      "erratum" => "katello_errata_install"
+     }.each do |command, feature|
+      it "errors out on #{command} install" do
+        cmd = ["host-collection", command,  "install"]
+        params = ["--help"]
+        result = run_cmd(cmd + params)
+        assert_exit_code_equal(HammerCLI::EX_UNAVAILABLE, result.exit_code)
+        assert_match(/Not supported. Use the remote execution equivalent/, result.out)
+        assert_match(/feature #{feature}/, result.out)
+        assert_match(/Specify the host collection with the --search-query/, result.out)
+      end
     end
   end
 end

data/test/functional/host_collection/content_remove_test.rb

@@ -1,37 +1,20 @@
 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))
+  describe 'Remove Content on a host-collection' do
+    { "package" => "katello_package_remove",
+      "package-group" => "katello_group_remove"
+     }.each do |command, feature|
+      it "errors out on #{command} install" do
+        cmd = ["host-collection", command, "remove"]
+        params = ["--help"]
+        result = run_cmd(cmd + params)
+        assert_exit_code_equal(HammerCLI::EX_UNAVAILABLE, result.exit_code)
+        assert_match(/Not supported. Use the remote execution equivalent/, result.out)
+        assert_match(/feature #{feature}/, result.out)
+        assert_match(/Specify the host collection with the --search-query/, result.out)
+      end
     end
   end
 end

data/test/functional/host_collection/content_update_test.rb

@@ -1,37 +1,20 @@
 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))
+  describe 'update Content on a host-collection' do
+    { "package" => "katello_package_update",
+      "package-group" => "katello_group_update"
+     }.each do |command, feature|
+      it "errors out on #{command} update" do
+        cmd = ["host-collection", command, "update"]
+        params = ["--help"]
+        result = run_cmd(cmd + params)
+        assert_exit_code_equal(HammerCLI::EX_UNAVAILABLE, result.exit_code)
+        assert_match(/Not supported. Use the remote execution equivalent/, result.out)
+        assert_match(/feature #{feature}/, result.out)
+        assert_match(/Specify the host collection with the --search-query/, result.out)
+      end
     end
   end
 end

data/test/test_helper.rb

@@ -4,7 +4,7 @@ require 'minitest/spec'
 require 'mocha/minitest'
 require 'hammer_cli'
 
-KATELLO_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '4.9')
+KATELLO_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '4.10')
 
 if HammerCLI.context[:api_connection]
   HammerCLI.context[:api_connection].create('foreman') do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_katello
 version: !ruby/object:Gem::Version
-  version: 1.9.1
+  version: 1.10.0
 platform: ruby
 authors:
 - Adam Price
@@ -37,7 +37,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-06-21 00:00:00.000000000 Z
+date: 2023-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman
@@ -204,6 +204,7 @@ files:
 - config/katello.yml
 - lib/hammer_cli_katello.rb
 - lib/hammer_cli_katello/acs.rb
+- lib/hammer_cli_katello/acs_bulk_actions.rb
 - lib/hammer_cli_katello/activation_key.rb
 - lib/hammer_cli_katello/apipie_helper.rb
 - lib/hammer_cli_katello/associating_commands.rb
@@ -282,104 +283,153 @@ files:
 - lib/hammer_cli_katello/status.rb
 - lib/hammer_cli_katello/subscription.rb
 - lib/hammer_cli_katello/sync_plan.rb
+- lib/hammer_cli_katello/unsupported_katello_agent_command_helper.rb
 - lib/hammer_cli_katello/version.rb
 - locale/Makefile
 - locale/README.md
 - locale/bn/LC_MESSAGES/hammer-cli-katello.mo
+- locale/bn/hammer-cli-katello.edit.po
 - locale/bn/hammer-cli-katello.po
 - locale/bn_IN/LC_MESSAGES/hammer-cli-katello.mo
+- locale/bn_IN/hammer-cli-katello.edit.po
 - locale/bn_IN/hammer-cli-katello.po
+- locale/bqi/LC_MESSAGES/hammer-cli-katello.mo
+- locale/bqi/hammer-cli-katello.edit.po
 - locale/ca/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ca/hammer-cli-katello.edit.po
 - locale/ca/hammer-cli-katello.po
 - locale/cs_CZ/LC_MESSAGES/hammer-cli-katello.mo
+- locale/cs_CZ/hammer-cli-katello.edit.po
 - locale/cs_CZ/hammer-cli-katello.po
 - locale/de/LC_MESSAGES/hammer-cli-katello.mo
+- locale/de/hammer-cli-katello.edit.po
 - locale/de/hammer-cli-katello.po
 - locale/de_AT/LC_MESSAGES/hammer-cli-katello.mo
+- locale/de_AT/hammer-cli-katello.edit.po
 - locale/de_AT/hammer-cli-katello.po
 - locale/de_DE/LC_MESSAGES/hammer-cli-katello.mo
+- locale/de_DE/hammer-cli-katello.edit.po
 - locale/de_DE/hammer-cli-katello.po
 - locale/el/LC_MESSAGES/hammer-cli-katello.mo
+- locale/el/hammer-cli-katello.edit.po
 - locale/el/hammer-cli-katello.po
 - locale/en/LC_MESSAGES/hammer-cli-katello.mo
 - locale/en/hammer-cli-katello.po
 - locale/en_GB/LC_MESSAGES/hammer-cli-katello.mo
+- locale/en_GB/hammer-cli-katello.edit.po
 - locale/en_GB/hammer-cli-katello.po
 - locale/en_US/LC_MESSAGES/hammer-cli-katello.mo
+- locale/en_US/hammer-cli-katello.edit.po
 - locale/en_US/hammer-cli-katello.po
 - locale/es/LC_MESSAGES/hammer-cli-katello.mo
+- locale/es/hammer-cli-katello.edit.po
 - locale/es/hammer-cli-katello.po
 - locale/et_EE/LC_MESSAGES/hammer-cli-katello.mo
+- locale/et_EE/hammer-cli-katello.edit.po
 - locale/et_EE/hammer-cli-katello.po
 - locale/fr/LC_MESSAGES/hammer-cli-katello.mo
+- locale/fr/hammer-cli-katello.edit.po
 - locale/fr/hammer-cli-katello.po
 - locale/gl/LC_MESSAGES/hammer-cli-katello.mo
+- locale/gl/hammer-cli-katello.edit.po
 - locale/gl/hammer-cli-katello.po
 - locale/gu/LC_MESSAGES/hammer-cli-katello.mo
+- locale/gu/hammer-cli-katello.edit.po
 - locale/gu/hammer-cli-katello.po
 - locale/hammer-cli-katello.pot
 - locale/he_IL/LC_MESSAGES/hammer-cli-katello.mo
+- locale/he_IL/hammer-cli-katello.edit.po
 - locale/he_IL/hammer-cli-katello.po
 - locale/hi/LC_MESSAGES/hammer-cli-katello.mo
+- locale/hi/hammer-cli-katello.edit.po
 - locale/hi/hammer-cli-katello.po
 - locale/id/LC_MESSAGES/hammer-cli-katello.mo
+- locale/id/hammer-cli-katello.edit.po
 - locale/id/hammer-cli-katello.po
 - locale/it/LC_MESSAGES/hammer-cli-katello.mo
+- locale/it/hammer-cli-katello.edit.po
 - locale/it/hammer-cli-katello.po
 - locale/ja/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ja/hammer-cli-katello.edit.po
 - locale/ja/hammer-cli-katello.po
 - locale/ka/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ka/hammer-cli-katello.edit.po
 - locale/ka/hammer-cli-katello.po
 - locale/kn/LC_MESSAGES/hammer-cli-katello.mo
+- locale/kn/hammer-cli-katello.edit.po
 - locale/kn/hammer-cli-katello.po
 - locale/ko/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ko/hammer-cli-katello.edit.po
 - locale/ko/hammer-cli-katello.po
 - locale/ml_IN/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ml_IN/hammer-cli-katello.edit.po
 - locale/ml_IN/hammer-cli-katello.po
 - locale/mr/LC_MESSAGES/hammer-cli-katello.mo
+- locale/mr/hammer-cli-katello.edit.po
 - locale/mr/hammer-cli-katello.po
 - locale/nl_NL/LC_MESSAGES/hammer-cli-katello.mo
+- locale/nl_NL/hammer-cli-katello.edit.po
 - locale/nl_NL/hammer-cli-katello.po
 - locale/or/LC_MESSAGES/hammer-cli-katello.mo
+- locale/or/hammer-cli-katello.edit.po
 - locale/or/hammer-cli-katello.po
 - locale/pa/LC_MESSAGES/hammer-cli-katello.mo
+- locale/pa/hammer-cli-katello.edit.po
 - locale/pa/hammer-cli-katello.po
 - locale/pl/LC_MESSAGES/hammer-cli-katello.mo
+- locale/pl/hammer-cli-katello.edit.po
 - locale/pl/hammer-cli-katello.po
 - locale/pl_PL/LC_MESSAGES/hammer-cli-katello.mo
+- locale/pl_PL/hammer-cli-katello.edit.po
 - locale/pl_PL/hammer-cli-katello.po
 - locale/pt/LC_MESSAGES/hammer-cli-katello.mo
+- locale/pt/hammer-cli-katello.edit.po
 - locale/pt/hammer-cli-katello.po
 - locale/pt_BR/LC_MESSAGES/hammer-cli-katello.mo
+- locale/pt_BR/hammer-cli-katello.edit.po
 - locale/pt_BR/hammer-cli-katello.po
 - locale/ro/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ro/hammer-cli-katello.edit.po
 - locale/ro/hammer-cli-katello.po
 - locale/ro_RO/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ro_RO/hammer-cli-katello.edit.po
 - locale/ro_RO/hammer-cli-katello.po
 - locale/ru/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ru/hammer-cli-katello.edit.po
 - locale/ru/hammer-cli-katello.po
 - locale/sl/LC_MESSAGES/hammer-cli-katello.mo
+- locale/sl/hammer-cli-katello.edit.po
 - locale/sl/hammer-cli-katello.po
 - locale/sv_SE/LC_MESSAGES/hammer-cli-katello.mo
+- locale/sv_SE/hammer-cli-katello.edit.po
 - locale/sv_SE/hammer-cli-katello.po
 - locale/ta/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ta/hammer-cli-katello.edit.po
 - locale/ta/hammer-cli-katello.po
 - locale/ta_IN/LC_MESSAGES/hammer-cli-katello.mo
+- locale/ta_IN/hammer-cli-katello.edit.po
 - locale/ta_IN/hammer-cli-katello.po
 - locale/te/LC_MESSAGES/hammer-cli-katello.mo
+- locale/te/hammer-cli-katello.edit.po
 - locale/te/hammer-cli-katello.po
 - locale/tr/LC_MESSAGES/hammer-cli-katello.mo
+- locale/tr/hammer-cli-katello.edit.po
 - locale/tr/hammer-cli-katello.po
 - locale/vi/LC_MESSAGES/hammer-cli-katello.mo
+- locale/vi/hammer-cli-katello.edit.po
 - locale/vi/hammer-cli-katello.po
 - locale/vi_VN/LC_MESSAGES/hammer-cli-katello.mo
+- locale/vi_VN/hammer-cli-katello.edit.po
 - locale/vi_VN/hammer-cli-katello.po
 - locale/zanata.xml
 - locale/zh/LC_MESSAGES/hammer-cli-katello.mo
+- locale/zh/hammer-cli-katello.edit.po
 - locale/zh/hammer-cli-katello.po
 - locale/zh_CN/LC_MESSAGES/hammer-cli-katello.mo
+- locale/zh_CN/hammer-cli-katello.edit.po
 - locale/zh_CN/hammer-cli-katello.po
 - locale/zh_TW/LC_MESSAGES/hammer-cli-katello.mo
+- locale/zh_TW/hammer-cli-katello.edit.po
 - locale/zh_TW/hammer-cli-katello.po
 - test/data/2.5/foreman_api.json
 - test/data/3.0/foreman_api.json
@@ -402,6 +452,7 @@ files:
 - test/data/3.9/foreman_api.json
 - test/data/4.0/foreman_api.json
 - test/data/4.1/foreman_api.json
+- test/data/4.10/foreman_api.json
 - test/data/4.3/foreman_api.json
 - test/data/4.4/foreman_api.json
 - test/data/4.5/foreman_api.json
@@ -519,7 +570,6 @@ files:
 - test/functional/host/traces/list_test.rb
 - test/functional/host/traces/resolve_test.rb
 - test/functional/host_collection/add_host_test.rb
-- test/functional/host_collection/content_api_expectations.rb
 - test/functional/host_collection/content_install_test.rb
 - test/functional/host_collection/content_remove_test.rb
 - test/functional/host_collection/content_update_test.rb
@@ -640,6 +690,7 @@ test_files:
 - test/data/3.9/foreman_api.json
 - test/data/4.0/foreman_api.json
 - test/data/4.1/foreman_api.json
+- test/data/4.10/foreman_api.json
 - test/data/4.3/foreman_api.json
 - test/data/4.4/foreman_api.json
 - test/data/4.5/foreman_api.json
@@ -757,7 +808,6 @@ test_files:
 - test/functional/host/traces/list_test.rb
 - test/functional/host/traces/resolve_test.rb
 - test/functional/host_collection/add_host_test.rb
-- test/functional/host_collection/content_api_expectations.rb
 - test/functional/host_collection/content_install_test.rb
 - test/functional/host_collection/content_remove_test.rb
 - test/functional/host_collection/content_update_test.rb

data/test/functional/host_collection/content_api_expectations.rb

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