hammer_cli_katello 1.5.0 → 1.5.1

data/test/unit/id_name_options_validator_test.rb

@@ -0,0 +1,96 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/id_name_options_validator'
+
+module HammerCLIKatello
+  describe IdNameOptionsValidator do
+    before(:each) do
+      @cmd = Object.new
+      @cmd.extend(IdNameOptionsValidator)
+      @cmd.class.send(:define_method, :validate_options) do |*_args, &block|
+        block.call
+      end
+    end
+
+    describe '#validate_id_or_name_with_parent' do
+      it 'accepts no record name and validates id, name, and parent opts' do
+        any_stub = stub(:required => true)
+        option_stub = stub(:exist? => true)
+        @cmd.stubs(:option).returns(option_stub)
+
+        @cmd.expects(:any).with('option_id', 'option_name').returns(any_stub)
+        @cmd.expects(:any).with('option_organization_id',
+                                'option_organization_name',
+                                'option_organization_label'
+                               ).returns(any_stub)
+
+        @cmd.validate_id_or_name_with_parent
+      end
+
+      it 'accepts content view as a record name' do
+        any_stub = stub(:required => true)
+        option_stub = stub(:exist? => true)
+        @cmd.stubs(:option).returns(option_stub)
+
+        @cmd.expects(:any).with('option_content_view_id',
+                                'option_content_view_name'
+                               ).returns(any_stub)
+        @cmd.expects(:any).with('option_organization_id',
+                                'option_organization_name',
+                                'option_organization_label'
+                               ).returns(any_stub)
+
+        @cmd.validate_id_or_name_with_parent(:content_view)
+      end
+
+      it 'does not validate org options if the name opt is not used' do
+        any_stub = stub(:required => true)
+        option_stub = stub(:exist? => false) # --name was not supplied
+        @cmd.stubs(:option).returns(option_stub)
+
+        @cmd.expects(:any).at_most_once.returns(any_stub)
+
+        @cmd.validate_id_or_name_with_parent
+      end
+
+      it 'does not validate name or id if required is false' do
+        any_stub = stub(:required => true)
+        option_stub = stub(:exist? => true)
+        @cmd.stubs(:option).returns(option_stub)
+
+        @cmd.expects(:any).with('option_organization_id',
+                                'option_organization_name',
+                                'option_organization_label'
+                               ).returns(any_stub)
+
+        @cmd.validate_id_or_name_with_parent(required: false)
+      end
+
+      it 'accepts a hash of fields for the parent' do
+        any_stub = stub(:required => true)
+        option_stub = stub(:exist? => true)
+        @cmd.stubs(:option).returns(option_stub)
+
+        @cmd.expects(:any).with('option_id', 'option_name').returns(any_stub)
+        @cmd.expects(:any).with('option_product_test').returns(any_stub)
+
+        @cmd.validate_id_or_name_with_parent(parent: {:product => ['test']})
+      end
+    end
+
+    describe '#validate_id_or_name' do
+      it 'validates that id or name is required' do
+        any_stub = stub(:required => true)
+        @cmd.expects(:any).with('option_id', 'option_name').returns(any_stub)
+        @cmd.validate_id_or_name
+      end
+
+      it 'accepts a custom record name' do
+        any_stub = stub(:required => true)
+        @cmd.expects(:any).with('option_content_view_id',
+                                'option_content_view_name'
+                               ).returns(any_stub)
+        @cmd.validate_id_or_name(:content_view)
+      end
+    end
+  end
+end

data/test/unit/id_resolver_test.rb

@@ -0,0 +1,58 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/id_resolver'
+
+module HammerCLIKatello
+  describe IdResolver do
+    let(:api) { mock('api') }
+    let(:searchables) { mock('searchables') }
+    let(:id_resolver) { IdResolver.new api, searchables }
+
+    before(:each) do
+      api.stubs(:resources).returns([])
+      api.stubs(:resource).returns([])
+    end
+
+    describe '#repository_ids' do
+      it 'accepts nil repository_names' do
+        id_resolver.stubs(:find_resources).returns([])
+        id_resolver.repository_ids({}).must_equal(nil)
+      end
+
+      it 'accepts repository_ids' do
+        id_resolver.repository_ids(
+          'option_repository_ids' => [1, 2]
+        ).must_equal [1, 2]
+      end
+
+      it 'accepts repository_names' do
+        id_resolver.stubs(:find_resources).returns(
+          [
+            {'id' => 1, 'name' => 'repo1'},
+            {'id' => 2, 'name' => 'repo2'}
+          ]
+        )
+        id_resolver.repository_ids(
+          'option_names' => %w(repo1 repo2)
+        ).must_equal [1, 2]
+      end
+    end
+
+    describe '#repository_id' do
+      it 'resolves ID from name' do
+        id_resolver.expects(:find_resource).with(:repositories, name: 'repo1').returns('id' => 5)
+        options = {name: 'repo1'}
+        id_resolver.repository_id(options).must_equal 5
+      end
+    end
+
+    describe '#content_view_version_id' do
+      it 'resolves ID from version number' do
+        options = {'option_version' => '2.0', 'option_content_view_id' => 4}
+        id_resolver.expects(:find_resources)
+                   .with(:content_view_versions, options)
+                   .returns(['id' => 5])
+        id_resolver.content_view_version_id(options).must_equal 5
+      end
+    end
+  end
+end

data/test/unit/messages_test.rb

@@ -0,0 +1,7 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require 'hammer_cli/testing/messages'
+
+include HammerCLI::Testing::Messages
+describe "message formatting" do
+  check_all_command_messages(HammerCLI::MainCommand, HammerCLIKatello::Command)
+end

data/test/unit/search_options_creators_test.rb

@@ -0,0 +1,145 @@
+require_relative '../test_helper'
+
+class SearchOptionsCreatorsMock < HammerCLIKatello::IdResolver
+  def one
+  end
+
+  def two
+  end
+end
+
+describe HammerCLIKatello::SearchOptionsCreators do
+  let(:api) { mock('api') }
+  let(:searchables) { HammerCLIKatello::Searchables.new }
+  let(:search_options_creators) { SearchOptionsCreatorsMock.new(api, searchables) }
+  let(:options) { Hash.new }
+  let(:resource) { mock('ApipieBindings::Resource') }
+
+  before(:each) do
+    api.stubs(:resources).returns([resource])
+    resource.stubs(:singular_name).returns('')
+  end
+
+  describe '#create_repositories_search_options' do
+    it 'handles an array of names' do
+      search_options_creators.create_repositories_search_options(
+        'option_names' => ['repo1']
+      )['names'].must_equal ['repo1']
+    end
+
+    it 'handles a single value name' do
+      search_options_creators.create_repositories_search_options(
+        'option_name' => 'repo1'
+      )['name'].must_equal 'repo1'
+    end
+
+    it 'handles multiple parameters' do
+      search_options_creators.create_repositories_search_options(
+        'option_names' => ['repo1'], 'option_name' => 'repo2'
+      )['name'].must_equal(nil)
+    end
+
+    it 'handles a single value product_id' do
+      search_options_creators.create_repositories_search_options(
+        'option_product_id' => 3
+      )['product_id'].must_equal 3
+    end
+  end
+
+  describe '#create_content_view_versions_search_options(options)' do
+    it 'handles a single value environment_id' do
+      search_options_creators.create_content_view_versions_search_options(
+        'option_environment_id' => 3,
+        'option_content_view_id' => :required_option
+      )['environment_id'].must_equal 3
+    end
+
+    it 'handles a single value content_view_id' do
+      search_options_creators.create_content_view_versions_search_options(
+        'option_content_view_id' => 3
+      )['content_view_id'].must_equal 3
+    end
+
+    it 'handles a single value version' do
+      search_options_creators.create_content_view_versions_search_options(
+        'option_version' => 3
+      )['version'].must_equal 3
+    end
+  end
+
+  describe "#create_host_collections_search_options" do
+    it 'handles organization options' do
+      api.stubs(:resource)
+      search_options_creators.stubs(:scoped_options).returns('option_name' => 'org1')
+      search_options_creators.expects(:organization_id).with('option_name' => 'org1').returns(1)
+      search_options_creators.expects(:create_search_options_with_katello_api)
+      search_options_creators.create_host_collections_search_options(
+        'option_organization_name' => 'org1'
+      )
+    end
+  end
+
+  describe 'without the katello api' do
+    before(:each) do
+      search_options_creators.expects(:create_search_options_without_katello_api)
+      search_options_creators.stubs(:api).returns(api)
+      api.stubs(:resource)
+    end
+
+    describe '#create_organizations_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_organizations_search_options(:anything)
+      end
+    end
+
+    describe '#create_environments_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_environments_search_options(:environments)
+      end
+    end
+
+    describe '#create_smart_proxies_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_smart_proxies_search_options(:anything)
+      end
+    end
+
+    describe '#create_capsules_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_capsules_search_options(:anything)
+      end
+    end
+
+    describe '#create_usergroups_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_usergroups_search_options(:anything)
+      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)
+      end
+    end
+  end # describe 'without the katello api'
+
+  describe '#create_search_options_with_katello_api' do
+    let(:searchable1) { HammerCLIForeman::Searchable.new('one', '') }
+    let(:searchable2) { HammerCLIForeman::Searchable.new('two', '') }
+
+    it 'translates all searchable fields from options' do
+      search_options_creators.stubs(:searchables).
+        returns([searchable1, searchable2])
+
+      search_options_creators.create_search_options_with_katello_api(
+        {'option_one' => 1, 'option_two' => 2}, resource
+      ).must_equal('one' => '1', 'two' => '2')
+    end
+  end # describe '#create_search_options_with_katello_api'
+end # describe HammerCLIKatello::SearchOptionsCreators