hammer_cli_katello 1.1.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d8787337433eea048dbc889beeb9d09dd0264d9f22838851544f08230e4fdb01
-  data.tar.gz: 280fbe62f124439debe0bcb7e19cf13f041857801d06eaf0ad2e162cdc53b36b
+  metadata.gz: 2ec840ea7e9516ce7268b7d2ddce6bebce2a4b4a71ac4963c475fc472a6e5dab
+  data.tar.gz: 2388188ed12e64cc36d20192ccd1d70c2e53547efef35a5f40b02d1a190fcb77
 SHA512:
-  metadata.gz: 611b6017e7bb314e31dcfa9138f37c9171a6e118e87b9174664bfe4f0569bc6b77b5b1e5cb2682ede8716c1f62aab4b9b046e6aad8e518a5dec83d0d2ddb4402
-  data.tar.gz: 61acf6ec874572401e818494534a264476b4300efead8781fe64f22ddbd350b33eed22eb764558fc80e5e8e3517c4df43bd24613dc84800dc7f094a333f091f2
+  metadata.gz: 190ede71c0bb873d5ded32b884de84137da2a367d987f055505fcfde3995a3419fd1e5c919630569b18266bba2efd5c5a496b471803de659f63dd547299c67e1
+  data.tar.gz: 3f96f683fb9a0ab6f0f92c6451aa528dd6929c6a16172dd1a4dfbfc63c7dcb6f1d7b2b0f49dc1fff72f039a08aaa5374d31d122a7222e053d985c78755f44e97

data/lib/hammer_cli_katello/host_collection.rb

@@ -50,6 +50,12 @@ module HammerCLIKatello
       success_message _("Host collection created.")
       failure_message _("Could not create the host collection")
 
+      option "--unlimited-hosts", :flag, "Set hosts max to unlimited"
+
+      validate_options :before, 'IdResolution' do
+        all(:option_unlimited_hosts, :option_max_hosts).rejected
+      end
+
       build_options
     end
 

data/lib/hammer_cli_katello/host_errata.rb

@@ -11,7 +11,7 @@ module HammerCLIKatello
       success_message _("Errata is being applied with task %{id}.")
       failure_message _("Could not apply errata")
 
-      build_options
+      build_options(without: ['ids'])
 
       def execute
         warn "This command uses katello agent and will be removed in favor of remote execution " \

data/lib/hammer_cli_katello/repository.rb

@@ -541,33 +541,6 @@ module HammerCLIKatello
       end
     end
 
-    class ExportCommand < HammerCLIKatello::SingleResourceCommand
-      include HammerCLIForemanTasks::Async
-      include OrganizationOptions
-      extend RepositoryScopedToProduct
-
-      validate_repo_name_requires_product_options
-      action :export
-      command_name "export"
-      desc _("Export content from a repository to the configured directory")
-
-      success_message _("Repository is being exported in task %{id}.")
-      failure_message _("Could not export the repository")
-
-      validate_options :before, 'IdResolution' do
-        organization_options = [:option_organization_id, :option_organization_name, \
-                                :option_organization_label]
-
-        if option(:option_product_name).exist?
-          any(*organization_options).required
-        end
-      end
-
-      build_options do |o|
-        o.expand.including(:products)
-      end
-    end
-
     autoload_subcommands
   end
 end

data/lib/hammer_cli_katello/version.rb

@@ -1,5 +1,5 @@
 module HammerCLIKatello
   def self.version
-    @version ||= Gem::Version.new('1.1.2')
+    @version ||= Gem::Version.new('1.2.0')
   end
 end

data/test/functional/host_collection/create_test.rb

@@ -29,6 +29,17 @@ module HammerCLIKatello
       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}]))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_katello
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.2.0
 platform: ruby
 authors:
 - Adam Price
@@ -32,10 +32,10 @@ authors:
 - Tom McKay
 - Walden Raines
 - Zach Huntington-Meath
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-01 00:00:00.000000000 Z
+date: 2021-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman
@@ -456,7 +456,6 @@ files:
 - test/functional/product/update_test.rb
 - test/functional/repository/create_test.rb
 - test/functional/repository/delete_test.rb
-- test/functional/repository/export_test.rb
 - test/functional/repository/info_test.rb
 - test/functional/repository/list_test.rb
 - test/functional/repository/remove_content_test.rb
@@ -492,7 +491,7 @@ homepage: https://github.com/Katello/hammer-cli-katello
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -507,8 +506,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Katello commands for Hammer
 test_files:
@@ -672,7 +671,6 @@ test_files:
 - test/functional/product/update_test.rb
 - test/functional/repository/create_test.rb
 - test/functional/repository/delete_test.rb
-- test/functional/repository/export_test.rb
 - test/functional/repository/info_test.rb
 - test/functional/repository/list_test.rb
 - test/functional/repository/remove_content_test.rb

data/test/functional/repository/export_test.rb

@@ -1,121 +0,0 @@
-require_relative '../test_helper'
-require_relative '../organization/organization_helpers'
-require 'hammer_cli_katello/repository'
-
-module HammerCLIKatello
-  describe Repository::ExportCommand do
-    include ForemanTaskHelpers
-    include OrganizationHelpers
-
-    it 'allows minimal options' do
-      ex = api_expects(:repositories, :export) do |p|
-        p['id'] == 1
-      end
-      ex.returns(id: '2')
-
-      expect_foreman_task('2')
-
-      run_cmd(%w(repository export --id 1))
-    end
-
-    describe 'resolves repository ID' do
-      it 'by requiring product' do
-        api_expects_no_call
-        result = run_cmd(%w(repository export --name repo1))
-        assert(result.err[/--product, --product-id is required/], 'Incorrect error message')
-      end
-
-      it 'by product ID' do
-        ex = api_expects(:repositories, :index) do |p|
-          p['name'] == 'repo1' && p['product_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        ex = api_expects(:repositories, :export) do |p|
-          p['id'] == 1
-        end
-        ex.returns(id: '2')
-
-        expect_foreman_task('2')
-
-        run_cmd(%w(repository export --name repo1 --product-id 3))
-      end
-    end
-
-    describe 'resolves product ID' do
-      it 'by requiring organization options' do
-        api_expects_no_call
-        result = run_cmd(%w(repository export --name repo1 --product prod1))
-        assert(result.err[/--organization-id, --organization, --organization-label is required/],
-               "Organization option requirements must be validated")
-      end
-
-      it 'by organization ID' do
-        ex = api_expects(:products, :index) do |p|
-          p['name'] == 'prod3' && p['organization_id'] == '5'
-        end
-        ex.returns(index_response([{'id' => 3}]))
-
-        ex = api_expects(:repositories, :index) do |p|
-          p['name'] == 'repo1' && p['product_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        ex = api_expects(:repositories, :export) do |p|
-          p['id'] == 1
-        end
-        ex.returns(id: '2')
-
-        expect_foreman_task('2')
-
-        run_cmd(%w(repository export --name repo1 --product prod3 --organization-id 5))
-      end
-
-      it 'by organization name' do
-        expect_organization_search('org5', 5)
-
-        ex = api_expects(:products, :index) do |p|
-          p['name'] == 'prod3' && p['organization_id'] == 5
-        end
-        ex.returns(index_response([{'id' => 3}]))
-
-        ex = api_expects(:repositories, :index) do |p|
-          p['name'] == 'repo1' && p['product_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        ex = api_expects(:repositories, :export) do |p|
-          p['id'] == 1
-        end
-        ex.returns(id: '2')
-
-        expect_foreman_task('2')
-
-        run_cmd(%w(repository export --name repo1 --product prod3 --organization org5))
-      end
-
-      it 'by organization label' do
-        expect_organization_search('org5', 5, field: 'label')
-
-        ex = api_expects(:products, :index) do |p|
-          p['name'] == 'prod3' && p['organization_id'] == 5
-        end
-        ex.returns(index_response([{'id' => 3}]))
-
-        ex = api_expects(:repositories, :index) do |p|
-          p['name'] == 'repo1' && p['product_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        ex = api_expects(:repositories, :export) do |p|
-          p['id'] == 1
-        end
-        ex.returns(id: '2')
-
-        expect_foreman_task('2')
-
-        run_cmd(%w(repository export --name repo1 --product prod3 --organization-label org5))
-      end
-    end
-  end
-end