hammer_cli_katello 1.5.0 → 1.5.1

data/test/functional/content_import/version_test.rb

@@ -0,0 +1,85 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_katello/content_import'
+
+describe 'content-import version' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-import version)
+  end
+
+  let(:task_id) { '5' }
+
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'planned'
+    }
+  end
+
+  let(:path) do
+    File.dirname(__FILE__)
+  end
+
+  let(:metadata) do
+    JSON.parse(File.read("#{path}/metadata.json"))
+  end
+
+  let(:organization_id) { 3 }
+
+  it "performs import with required options and async" do
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{path}",
+      '--async'
+    ]
+    api_expects(:content_imports, :version)
+      .with_params('organization_id' => organization_id, 'path' => path, 'metadata' => metadata)
+      .returns(response)
+
+    result = run_cmd(@cmd + params)
+
+    assert_equal("Archive is being imported in task #{task_id}.\n", result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "performs import with required options" do
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{path}"
+    ]
+
+    api_expects(:content_imports, :version)
+      .with_params('organization_id' => organization_id, 'path' => path, 'metadata' => metadata)
+      .returns(response)
+
+    expect_foreman_task(task_id)
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails on missing required params' do
+    params = [
+      '--id=2'
+    ]
+
+    result = run_cmd(@cmd + params)
+    expected_error = "Could not import the archive."
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_equal(result.err[/#{expected_error}/], expected_error)
+  end
+
+  it 'fails on missing metadata.json' do
+    bad_path = "/nosuchdir"
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{bad_path}"
+    ]
+    result = run_cmd(@cmd + params)
+    expected_error = "Unable to find '#{bad_path}/metadata.json'."
+
+    assert_match(/#{expected_error}/, result.err)
+  end
+end

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

data/test/functional/content_view/add_content_view_version_test.rb

@@ -0,0 +1,88 @@
+require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::AddContentViewVersionCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
+    it 'allows minimal options' do
+      ex = api_expects(:content_views, :show) do |p|
+        p[:id] == '1'
+      end
+      ex.returns('id' => 1, 'component_ids' => [1, 3])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3 6)
+      end
+      run_cmd(%w(content-view add-version --id 1 --content-view-version-id 6))
+    end
+
+    it 'resolves content view version ID' do
+      ex = api_expects(:content_view_versions, :index) do |p|
+        p['content_view_id'] == 3 && p['version'] == '2.1'
+      end
+      ex.returns(index_response([{'id' => 6}]))
+      ex.returns('id' => 1, 'component_ids' => [1, 3])
+      ex = api_expects(:content_views, :show) do |p|
+        p[:id] == '1'
+      end
+      ex.returns('id' => 1, 'component_ids' => [1, 3])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3 6)
+      end
+      run_cmd(%w(content-view add-version --id 1 --content-view-id 3 --content-view-version 2.1))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view add-version --name cv1))
+        assert(result.err[/--organization-id, --organization, --organization-label is required/],
+               "Organization option requirements are validated")
+      end
+
+      it 'allows organization id' do
+        expect_content_view_search('1', 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization-id 1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization org1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization-label org1
+                   --content-view-version-id 3))
+      end
+    end
+  end
+end

data/test/functional/content_view/add_repository_test.rb

@@ -0,0 +1,21 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView do
+    describe 'AddRepositoryCommand' do
+      it 'allows adding a repository by ID' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\""
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv' && p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view add-repository --organization org1 --name cv --repository-id 1))
+      end
+    end
+  end
+end

data/test/functional/content_view/component/add_test.rb

@@ -0,0 +1,177 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+# rubocop:disable Metrics/ModuleLength
+module HammerCLIKatello
+  describe ContentViewComponent::AddComponents do
+    before do
+      @cmd = %w(content-view component add)
+      @composite = OpenStruct.new(:id => 2, :name => "composite")
+      @component_version = OpenStruct.new(:id => 666, :version => "1.0")
+      @component = OpenStruct.new(:id => 6, :name => "component",
+                                  :content_view_version => @component_version)
+      @organization = OpenStruct.new(:id => 1, :name => "great", :label => "label")
+    end
+
+    def setup_org_expectations
+      org_expect = api_expects(:organizations, :index) do |par|
+        par[:search] == "name = \"#{@organization.name}\""
+      end
+      org_expect.at_least_once.returns(index_response([{'id' => @organization.id.to_s}]))
+    end
+
+    def setup_content_view_expectations(content_view)
+      cv_expect = api_expects(:content_views, :index) do |p|
+        p['name'] == content_view.name && p["organization_id"].to_s == @organization.id.to_s
+      end
+      cv_expect.at_least_once.returns(index_response([{'id' => content_view.id}]))
+    end
+
+    describe 'allows adding a component with' do
+      it 'component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--latest"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view-version-id and latest=false' do
+        latest = false
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-version-id=#{@component_version.id}"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_version_id] == @component_version.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+
+    describe 'allows adding a component with composite name and org' do
+      before do
+        setup_org_expectations
+        setup_content_view_expectations(@composite)
+      end
+
+      it 'component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--organization=#{@organization.name}",
+                  "--latest"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view-version-id and latest=false' do
+        latest = false
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--organization=#{@organization.name}",
+                  "--component-content-view-version-id=#{@component_version.id}"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_version_id] == @component_version.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+
+    describe 'allows adding a component with component name and org' do
+      it 'component-content-view and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view=#{@component.name}",
+                  "--organization=#{@organization.name}",
+                  "--latest"]
+        setup_org_expectations
+        setup_content_view_expectations(@component)
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+
+    describe 'allows adding a component with component version number org component content_view' do
+      before do
+        cvv_expect = api_expects(:content_view_versions, :index) do |p|
+          p['content_view_id'].to_s == @component.id.to_s &&
+            p["version"] == @component.content_view_version.version
+        end
+        cvv_expect.at_least_once.
+          returns(index_response([{'id' => @component.content_view_version.id}]))
+      end
+
+      it 'component-content-view-id and component version number' do
+        latest = false
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--component-content-view-version=#{@component.content_view_version.version}"]
+
+        # 1) Should fetch the component_content_view_version_id
+        #    using cv specified in --component-content-view-id and version specified in
+        #    --component-content-view-version
+        # 2) Should set the component[:content_view_id] && component[:content_view_version_id]
+        #    correctly
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id.to_s &&
+            component[:content_view_version_id] == @component.content_view_version.id
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view and component version number' do
+        latest = false
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-view=#{@component.name}",
+                  "--component-content-view-version=#{@component.content_view_version.version}",
+                  "--organization=#{@organization.name}"]
+        # 1) Should fetch the org using org name specified in --organization
+        # 2) Should fetch composite content view using name in --composite-content-view
+        # 3) Should fetch component content view using name in --component-content-view
+        # 4) Should fetch the component_content_view_version_id
+        #    using cv specified in --component-content-view-id and version specified in
+        #    --component-content-view-version
+        # 5) Should set the component[:content_view_id] && component[:content_view_version_id]
+        #    correctly
+        setup_org_expectations
+        setup_content_view_expectations(@component)
+        setup_content_view_expectations(@composite)
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id &&
+            component[:content_view_version_id] == @component.content_view_version.id
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+  end
+end

data/test/functional/content_view/component/list_test.rb

@@ -0,0 +1,55 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+module HammerCLIKatello
+  describe ContentViewComponent::ListCommand do
+    before do
+      @cmd = %w(content-view component list)
+    end
+
+    it 'expects composite_content_view_id as minimal options' do
+      api_expects(:content_view_components, :index, "component list") do |p|
+        p["composite_content_view_id"].to_s == "1"
+      end
+      run_cmd(@cmd + ["--composite-content-view-id=1"])
+    end
+
+    it 'expects composite_content_view and organization_id as other options' do
+      api_expects(:content_views, :index) do |p|
+        p['name'] == "lol" && p["organization_id"].to_s == "1"
+      end
+      run_cmd(@cmd + %w(--composite-content-view=lol --organization-id=1))
+    end
+
+    it 'label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .at_least_once.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_components, :index) do |p|
+        p["composite_content_view_id"].to_s == "1"
+      end
+
+      api_expects(:content_views, :index).
+        at_least_once.returns(index_response([{'id' => 1}])) do |p|
+        p['name'] == "lol" && p["organization_id"].to_s == "1"
+      end
+
+      run_cmd(@cmd + %w(--composite-content-view=lol --organization-label=org1))
+    end
+
+    it 'name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .at_least_once.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_components, :index) do |p|
+        p["composite_content_view_id"].to_s == "1"
+      end
+
+      api_expects(:content_views, :index).
+        at_least_once.returns(index_response([{'id' => 1}])) do |p|
+        p['name'] == "lol" && p["organization_id"].to_s == "1"
+      end
+
+      run_cmd(@cmd + %w(--composite-content-view=lol --organization=org1))
+    end
+  end
+end

data/test/functional/content_view/component/remove_test.rb

@@ -0,0 +1,107 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+module HammerCLIKatello
+  describe ContentViewComponent::RemoveComponents do
+    before do
+      @cmd = %w(content-view component remove)
+
+      @component = OpenStruct.new(:id => 6, :name => "component")
+      @component2 = OpenStruct.new(:id => 7, :name => "component2")
+
+      @content_view_component = OpenStruct.new(:id => 1444,
+                                               :content_view => @component)
+
+      @content_view_component2 = OpenStruct.new(:id => 1445,
+                                                :content_view => @component2)
+
+      @composite = OpenStruct.new(:id => 2, :name => "composite",
+                                  :content_view_components => [@content_view_component])
+      @organization = OpenStruct.new(:id => 1, :name => "great", :label => "label")
+      @content_view_components = [@content_view_component, @content_view_component2]
+    end
+
+    def setup_org_expectations
+      org_expect = api_expects(:organizations, :index) do |par|
+        par[:search] == "name = \"#{@organization.name}\""
+      end
+      org_expect.at_least_once.returns(index_response([{'id' => @organization.id.to_s}]))
+    end
+
+    def setup_content_view_expectations(content_view)
+      cv_expect = api_expects(:content_views, :index) do |p|
+        p['name'] == content_view.name && p["organization_id"].to_s == @organization.id.to_s
+      end
+      cv_expect.at_least_once.returns(index_response([{"id" => content_view.id}]))
+    end
+
+    def setup_content_view_components_expectations
+      cvc_expect = api_expects(:content_view_components, :index) do |p|
+        p[:composite_content_view_id].to_s == @composite.id.to_s
+      end
+      ret = @content_view_components.map do |cvc|
+        { "id" => cvc.id,
+          "content_view" => {"id" => cvc.content_view.id,
+                             "name" => cvc.content_view.name}}
+      end
+      cvc_expect.at_least_once.returns(index_response(ret))
+    end
+
+    describe 'allows deleting component with' do
+      it 'id and component-ids' do
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-ids=#{@content_view_components.map(&:id).join(',')}"]
+
+        api_expects(:content_view_components, :remove_components) do |p|
+          component_ids = p['component_ids'].map(&:to_s)
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            component_ids.include?(@content_view_component.id.to_s) &&
+            component_ids.include?(@content_view_component2.id.to_s)
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'id and component-content-view-id' do
+        component_cv_ids = @content_view_components.map do |cvc|
+          cvc["content_view"]["id"]
+        end
+
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-ids=#{component_cv_ids.join(',')}"]
+        setup_content_view_components_expectations
+
+        api_expects(:content_view_components, :remove_components) do |p|
+          component_ids = p['component_ids'].map(&:to_s)
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            component_ids.include?(@content_view_component.id.to_s) &&
+            component_ids.include?(@content_view_component2.id.to_s)
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'orgs and component-content-view-names' do
+        component_cv_names = @content_view_components.map do |cvc|
+          cvc["content_view"]["name"]
+        end
+
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-views=#{component_cv_names.join(',')}",
+                  "--organization=#{@organization.name}"]
+        # 1) Should fetch the org using org name specified in --organization
+        # 2) Should fetch composite content view using name in --composite-content-view
+        # 3) Should fetch component content view using name in --component-content-view
+
+        setup_org_expectations
+        setup_content_view_expectations(@composite)
+        setup_content_view_components_expectations
+
+        api_expects(:content_view_components, :remove_components) do |p|
+          component_ids = p['component_ids'].map(&:to_s)
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            component_ids.include?(@content_view_component.id.to_s) &&
+            component_ids.include?(@content_view_component2.id.to_s)
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+  end
+end