hammer_cli_katello 1.3.0 → 1.3.1

data/test/functional/content_units/info_test.rb

@@ -0,0 +1,29 @@
+require_relative '../test_helper.rb'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require_relative '../repository/repository_helpers'
+require_relative '../product/product_helpers'
+require 'hammer_cli_katello/content_units'
+
+module HammerCLIKatello
+  describe ContentUnitsCommand::InfoCommand do
+    include ContentViewHelpers
+    include RepositoryHelpers
+    include ProductHelpers
+    include OrganizationHelpers
+
+    it 'allows minimal options' do
+      api_expects(:generic_content_units, :show)
+        .with_params('content_type' => 'python_package', 'id' => '1492')
+
+      run_cmd(%w(content-units info --content-type python_package --id 1492))
+    end
+
+    it 'requires content_type param' do
+      api_expects_no_call
+
+      r = run_cmd(%w(content-units info --id 1492))
+      assert(r.err.include?("Missing arguments for '--content-type'"), "Invalid error message")
+    end
+  end
+end

data/test/functional/content_units/list_test.rb

@@ -0,0 +1,106 @@
+require_relative '../test_helper.rb'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require_relative '../repository/repository_helpers'
+require_relative '../product/product_helpers'
+require 'hammer_cli_katello/content_units'
+
+module HammerCLIKatello
+  describe ContentUnitsCommand::ListCommand do
+    include ContentViewHelpers
+    include RepositoryHelpers
+    include ProductHelpers
+    include OrganizationHelpers
+
+    it 'allows minimal options' do
+      api_expects(:generic_content_units, :index)
+        .with_params('content_type' => 'python_package')
+
+      run_cmd(%w(content-units list --content-type python_package))
+    end
+
+    it 'requires content_type param' do
+      api_expects_no_call
+
+      r = run_cmd(%w(content-units list))
+      assert(r.err.include?("Missing arguments for '--content-type'"), "Invalid error message")
+    end
+
+    describe 'repository options' do
+      it 'may be specified by ID' do
+        api_expects(:generic_content_units, :index)
+          .with_params('content_type' => 'python_package', 'repository_id' => 1)
+
+        run_cmd(%w(content-units list --content-type python_package --repository-id 1))
+      end
+
+      it 'require product ID when given repository name' do
+        api_expects_no_call
+
+        r = run_cmd(%w(content-units list --content-type python_package --repository repo1))
+        assert(r.err.include?("--product, --product-id is required"), "Invalid error message")
+      end
+
+      # rubocop:disable LineLength
+      it 'may be specified by name and product ID' do
+        expect_repository_search(2, 'repo1', 1)
+
+        api_expects(:generic_content_units, :index)
+          .with_params('content_type' => 'python_package', 'repository_id' => 1)
+        run_cmd(%w(content-units list --content-type python_package --repository repo1 --product-id 2))
+      end
+      # rubocop:enable LineLength
+    end
+
+    describe 'organization options' do
+      it 'may be specified by ID' do
+        api_expects(:generic_content_units, :index)
+          .with_params('organization_id' => 1, 'content_type' => 'python_package')
+
+        run_cmd(%w(content-units list --organization-id 1 --content-type python_package))
+      end
+
+      it 'may be specified by name' do
+        expect_organization_search('org3', 1)
+        api_expects(:generic_content_units, :index)
+          .with_params('organization_id' => 1, 'content_type' => 'python_package')
+
+        run_cmd(%w(content-units list --organization org3 --content-type python_package))
+      end
+
+      it 'may be specified by label' do
+        expect_organization_search('org3', 1, field: 'label')
+        api_expects(:generic_content_units, :index)
+          .with_params('organization_id' => 1, 'content_type' => 'python_package')
+
+        run_cmd(%w(content-units list --organization-label org3 --content-type python_package))
+      end
+    end
+
+    describe 'content-view options' do
+      # rubocop:disable LineLength
+      it 'may be specified by ID' do
+        api_expects(:content_view_versions, :index)
+          .with_params('content_view_id' => 1, 'version' => '2.1')
+          .returns(index_response([{'id' => 5}]))
+        api_expects(:generic_content_units, :index)
+          .with_params('content_view_version_id' => 5, 'content_type' => 'python_package')
+
+        run_cmd(
+          %w(
+            content-units list --content-view-id 1 --content-view-version 2.1 --content-type python_package
+          )
+        )
+      end
+
+      it 'requires organization ID when given content view name' do
+        api_expects_no_call
+
+        r = run_cmd(%w(content-units list --content-view cv1 --content-view-version 2.1 --content-type python_package))
+        expected_error = "--organization-id, --organization, --organization-label is required"
+        assert(r.err.include?(expected_error), "Invalid error message")
+      end
+      # rubocop:enable LineLength
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_katello
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Adam Price
@@ -35,7 +35,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-11-11 00:00:00.000000000 Z
+date: 2021-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman
@@ -220,6 +220,7 @@ files:
 - lib/hammer_cli_katello/content_export_incremental.rb
 - lib/hammer_cli_katello/content_import.rb
 - lib/hammer_cli_katello/content_override.rb
+- lib/hammer_cli_katello/content_units.rb
 - lib/hammer_cli_katello/content_view.rb
 - lib/hammer_cli_katello/content_view_component.rb
 - lib/hammer_cli_katello/content_view_name_resolvable.rb
@@ -262,7 +263,6 @@ files:
 - lib/hammer_cli_katello/option_sources/lifecycle_environment_params.rb
 - lib/hammer_cli_katello/organization.rb
 - lib/hammer_cli_katello/organization_options.rb
-- lib/hammer_cli_katello/ostree_branch.rb
 - lib/hammer_cli_katello/output/fields.rb
 - lib/hammer_cli_katello/output/formatters.rb
 - lib/hammer_cli_katello/package.rb
@@ -358,6 +358,8 @@ files:
 - test/functional/content_import/list_test.rb
 - test/functional/content_import/metadata.json
 - test/functional/content_import/version_test.rb
+- test/functional/content_units/info_test.rb
+- test/functional/content_units/list_test.rb
 - test/functional/content_view/add_content_view_version_test.rb
 - test/functional/content_view/add_repository_test.rb
 - test/functional/content_view/component/add_test.rb
@@ -575,6 +577,8 @@ test_files:
 - test/functional/content_import/list_test.rb
 - test/functional/content_import/metadata.json
 - test/functional/content_import/version_test.rb
+- test/functional/content_units/info_test.rb
+- test/functional/content_units/list_test.rb
 - test/functional/content_view/add_content_view_version_test.rb
 - test/functional/content_view/add_repository_test.rb
 - test/functional/content_view/component/add_test.rb

data/lib/hammer_cli_katello/ostree_branch.rb

@@ -1,34 +0,0 @@
-module HammerCLIKatello
-  class OstreeBranchCommand < HammerCLIKatello::Command
-    resource :ostree_branches
-
-    class ListCommand < HammerCLIKatello::ListCommand
-      extend RepositoryScopedToProduct
-
-      validate_repo_name_requires_product_options(:option_repository_name)
-
-      output do
-        field :id, _("Id")
-        field :name, _("Name")
-        field :version, _("version")
-      end
-
-      build_options do |o|
-        o.expand.including(:products, :organizations, :content_views)
-      end
-    end
-
-    class InfoCommand < HammerCLIKatello::InfoCommand
-      output do
-        field :id, _("Id")
-        field :name, _("Name")
-        field :version, _("Version")
-        field :commit, _("Commit")
-      end
-
-      build_options
-    end
-
-    autoload_subcommands
-  end
-end