hammer_cli_katello 0.16.1 → 0.17.0

data/test/functional/repository_set/enable_test.rb

@@ -0,0 +1,42 @@
+require_relative '../test_helper'
+require_relative './repository_set_helpers'
+require 'hammer_cli_katello/repository_set'
+
+module HammerCLIKatello
+  describe RepositorySetCommand::EnableCommand do
+    include RepositorySetHelpers
+
+    it 'allows minimal options' do
+      api_expects(:repository_sets, :enable)
+        .with_params('id' => 1)
+      assert_success run_cmd(%w(repository-set enable --id 1))
+    end
+
+    it 'requires repository set options' do
+      api_expects_no_call
+      assert_failure run_cmd(%w(repository-set enable)), /id.*name.*required/
+    end
+
+    describe 'resolves repository set ID' do
+      it 'by requiring organization or product options' do
+        api_expects_no_call
+        assert_failure run_cmd(%w(repository-set enable --name reposet1)),
+          /organization.*product.*required/
+      end
+
+      it 'from organization options' do
+        expect_repository_set_search('reposet1', 1, organization_id: 2)
+        api_expects(:repository_sets, :enable)
+          .with_params('id' => 1)
+        assert_success run_cmd(%w(repository-set enable --name reposet1 --organization-id 2))
+      end
+
+      it 'from product options' do
+        expect_repository_set_search('reposet1', 1, product_id: 3)
+        api_expects(:repository_sets, :enable)
+          .with_params('id' => 1)
+        assert_success run_cmd(%w(repository-set enable --name reposet1 --product-id 3))
+      end
+    end
+  end
+end

data/test/functional/repository_set/info_test.rb

@@ -0,0 +1,17 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/repository_set'
+
+module HammerCLIKatello
+  describe RepositorySetCommand::InfoCommand do
+    it 'allows minimal options' do
+      api_expects(:repository_sets, :show)
+        .with_params('id' => 1)
+      assert_success run_cmd(%w(repository-set info --id 1))
+    end
+
+    it 'requires repository set options' do
+      api_expects_no_call
+      assert_failure run_cmd(%w(repository-set info)), /id.*name.*required/
+    end
+  end
+end

data/test/functional/repository_set/list_test.rb

@@ -0,0 +1,18 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/repository_set'
+
+module HammerCLIKatello
+  describe RepositorySetCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:repository_sets, :index)
+        .with_params('organization_id' => 1)
+        .returns(index_response([]))
+      assert_success run_cmd(%w(repository-set list --organization-id 1))
+    end
+
+    it 'requires organization options' do
+      api_expects_no_call
+      assert_failure run_cmd(%w(repository-set list)), /organization.*required/
+    end
+  end
+end

data/test/functional/repository_set/repository_set_helpers.rb

@@ -0,0 +1,20 @@
+require_relative '../search_helpers'
+
+module RepositorySetHelpers
+  include SearchHelpers
+
+  def expect_repository_set_search(name, id, organization_id: nil, product_id: nil)
+    params = {'name' => name}
+    params.merge(organization_id: organization_id) if organization_id.nil?
+    params.merge(product_id: product_id) if product_id.nil?
+    expect_generic_repository_sets_search(
+      params: params,
+      returns: {'id' => id})
+  end
+
+  def expect_generic_repository_sets_search(params: {}, returns: [])
+    api_expects(:repository_sets, :index, 'Find repository sets')
+      .with_params(params)
+      .returns(index_response(returns))
+  end
+end

data/test/functional/test_helper.rb

@@ -9,3 +9,16 @@ include HammerCLI::Testing::OutputMatchers
 include HammerCLI::Testing::CommandAssertions
 include HammerCLI::Testing::DataHelpers
 include HammerCLIForeman::Testing::APIExpectations
+
+def assert_success(command_run_result)
+  assert command_run_result.exit_code.zero?,
+    "Nonzero exit code (#{command_run_result.exit_code}): #{command_run_result.err}"
+end
+
+def assert_failure(command_run_result, msg = nil)
+  assert !command_run_result.exit_code.zero?, "Success exit code (0) when expecting failure"
+  if msg
+    assert command_run_result.err[msg],
+      "Actual:\n#{command_run_result.err}\nExpected:\n#{msg.inspect}"
+  end
+end

data/test/test_helper.rb

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

data/test/unit/id_name_options_validator_test.rb

@@ -6,7 +6,7 @@ module HammerCLIKatello
     before(:each) do
       @cmd = Object.new
       @cmd.extend(IdNameOptionsValidator)
-      @cmd.class.send(:define_method, :validate_options) do |&block|
+      @cmd.class.send(:define_method, :validate_options) do |*_args, &block|
         block.call
       end
     end

data/test/unit/search_options_creators_test.rb

@@ -110,6 +110,12 @@ describe HammerCLIKatello::SearchOptionsCreators do
       end
     end
 
+    describe '#create_compute_profiles_search_options(options, mode = nil)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_compute_profiles_search_options(:anything)
+      end
+    end
+
     describe '#create_search_options_with_katello_api(options, resource)' do
       it 'does not use the katello api' do
         search_options_creators.create_organizations_search_options(:anything)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_katello
 version: !ruby/object:Gem::Version
-  version: 0.16.1
+  version: 0.17.0
 platform: ruby
 authors:
 - Adam Price
@@ -34,7 +34,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-04-17 00:00:00.000000000 Z
+date: 2019-01-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman
@@ -42,14 +42,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.15.0
+        version: 0.16.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.15.0
+        version: 0.16.0
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman_tasks
   requirement: !ruby/object:Gem::Requirement
@@ -243,6 +243,7 @@ files:
 - lib/hammer_cli_katello/content_view_version.rb
 - lib/hammer_cli_katello/cv_import_export_helper.rb
 - lib/hammer_cli_katello/erratum.rb
+- lib/hammer_cli_katello/erratum_info_command.rb
 - lib/hammer_cli_katello/exception_handler.rb
 - lib/hammer_cli_katello/file.rb
 - lib/hammer_cli_katello/filter.rb
@@ -310,6 +311,7 @@ files:
 - test/data/2.5/foreman_api.json
 - test/data/3.0/foreman_api.json
 - test/data/3.10/foreman_api.json
+- test/data/3.11/foreman_api.json
 - test/data/3.2/foreman_api.json
 - test/data/3.4/foreman_api.json
 - test/data/3.5/foreman_api.json
@@ -369,6 +371,8 @@ files:
 - test/functional/content_view/version/list_test.rb
 - test/functional/content_view/version/promote_test.rb
 - test/functional/content_view/version/republish_repositories_test.rb
+- test/functional/erratum/erratum_helpers.rb
+- test/functional/erratum/info_test.rb
 - test/functional/erratum/list_test.rb
 - test/functional/file/file_helpers.rb
 - test/functional/file/info_test.rb
@@ -380,6 +384,7 @@ files:
 - test/functional/filter_rule/list_test.rb
 - test/functional/filter_rule/update_test.rb
 - test/functional/host/errata/apply_test.rb
+- test/functional/host/errata/info_test.rb
 - test/functional/host/errata/recalculate_test.rb
 - test/functional/host/extensions/create_test.rb
 - test/functional/host/extensions/data/host.json
@@ -440,6 +445,12 @@ files:
 - test/functional/repository/synchronize_test.rb
 - test/functional/repository/update_test.rb
 - test/functional/repository/upload_test.rb
+- test/functional/repository_set/available_repositories_test.rb
+- test/functional/repository_set/disable_test.rb
+- test/functional/repository_set/enable_test.rb
+- test/functional/repository_set/info_test.rb
+- test/functional/repository_set/list_test.rb
+- test/functional/repository_set/repository_set_helpers.rb
 - test/functional/search_helpers.rb
 - test/functional/subscription/list_test.rb
 - test/functional/sync_plan/create_test.rb
@@ -475,7 +486,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 2.6.14
 signing_key: 
 specification_version: 4
 summary: Katello commands for Hammer
@@ -483,6 +494,7 @@ test_files:
 - test/data/2.5/foreman_api.json
 - test/data/3.0/foreman_api.json
 - test/data/3.10/foreman_api.json
+- test/data/3.11/foreman_api.json
 - test/data/3.2/foreman_api.json
 - test/data/3.4/foreman_api.json
 - test/data/3.5/foreman_api.json
@@ -542,6 +554,8 @@ test_files:
 - test/functional/content_view/version/list_test.rb
 - test/functional/content_view/version/promote_test.rb
 - test/functional/content_view/version/republish_repositories_test.rb
+- test/functional/erratum/erratum_helpers.rb
+- test/functional/erratum/info_test.rb
 - test/functional/erratum/list_test.rb
 - test/functional/file/file_helpers.rb
 - test/functional/file/info_test.rb
@@ -553,6 +567,7 @@ test_files:
 - test/functional/filter_rule/list_test.rb
 - test/functional/filter_rule/update_test.rb
 - test/functional/host/errata/apply_test.rb
+- test/functional/host/errata/info_test.rb
 - test/functional/host/errata/recalculate_test.rb
 - test/functional/host/extensions/create_test.rb
 - test/functional/host/extensions/data/host.json
@@ -613,6 +628,12 @@ test_files:
 - test/functional/repository/synchronize_test.rb
 - test/functional/repository/update_test.rb
 - test/functional/repository/upload_test.rb
+- test/functional/repository_set/available_repositories_test.rb
+- test/functional/repository_set/disable_test.rb
+- test/functional/repository_set/enable_test.rb
+- test/functional/repository_set/info_test.rb
+- test/functional/repository_set/list_test.rb
+- test/functional/repository_set/repository_set_helpers.rb
 - test/functional/search_helpers.rb
 - test/functional/subscription/list_test.rb
 - test/functional/sync_plan/create_test.rb