hammer_cli_katello 1.5.0 → 1.5.1

data/test/functional/content_view/version/promote_test.rb

@@ -0,0 +1,75 @@
+require_relative '../../test_helper'
+
+module HammerCLIKatello
+  describe ContentViewVersion::PromoteCommand do
+    it 'allows promoting a content view' do
+      ex = api_expects(:organizations, :index) do |p|
+        p[:search] == "name = \"org1\""
+      end
+      ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+      ex = api_expects(:content_views, :index) do |p|
+        p['name'] == 'cv' && p['organization_id'] == 1
+      end
+      ex.returns(index_response([{'id' => 3}]))
+
+      ex = api_expects(:content_view_versions, :index) do |p|
+        p['version'] == '1' && p['content_view_id'] == 3
+      end
+      ex.returns(index_response([{'id' => 6}]))
+
+      ex = api_expects(:lifecycle_environments, :index) do |p|
+        p['organization_id'] == 1 && p['name'] == 'test'
+      end
+      ex.returns(index_response([{'id' => 9}]))
+
+      api_expects(:content_view_versions, :promote) do |p|
+        p['id'] == 6 && p['environment_ids'] == [9]
+      end
+
+      run_cmd(%w(content-view version promote --organization org1 --content-view cv
+                 --to-lifecycle-environment test --version 1 --async))
+    end
+
+    it 'allows promoting a content view without version name/id' do
+      ex = api_expects(:organizations, :index) do |p|
+        p[:search] == "name = \"org1\""
+      end
+      ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+      ex = api_expects(:content_views, :index) do |p|
+        p['name'] == 'cv' && p['organization_id'] == 1
+      end
+      ex.returns(index_response([{'id' => 3}]))
+
+      ex = api_expects(:content_view_versions, :index) do |p|
+        p['content_view_id'] == 3
+      end
+      ex.returns(
+        index_response(
+          [
+            {'id' => 6, 'name' => 'cv', 'environments' => [{'id' => 1}]},
+            {'id' => 7, 'name' => 'cv2', 'environments' => [{'id' => 9}]}
+          ]
+        )
+      )
+
+      ex = api_expects(:lifecycle_environments, :index) do |p|
+        p['organization_id'] == 1 && p['name'] == 'Library'
+      end
+      ex.returns(index_response([{'id' => 1}]))
+
+      ex = api_expects(:lifecycle_environments, :index) do |p|
+        p['organization_id'] == 1 && p['name'] == 'test'
+      end
+      ex.returns(index_response([{'id' => 9}]))
+
+      api_expects(:content_view_versions, :promote) do |p|
+        p['id'] == 6 && p['environment_ids'] == [9]
+      end
+
+      run_cmd(%w(content-view version promote --organization org1 --content-view cv
+                 --to-lifecycle-environment test --from-lifecycle-environment Library --async))
+    end
+  end
+end

data/test/functional/content_view/version/republish_repositories_test.rb

@@ -0,0 +1,34 @@
+require_relative '../../test_helper'
+
+module HammerCLIKatello
+  describe ContentViewVersion::RepublishRepositoriesCommand do
+    include ForemanTaskHelpers
+    include OrganizationHelpers
+
+    it 'allows republishing repositories on  a content view version' do
+      expect_organization_search('org1', 1)
+
+      ex = api_expects(:content_views, :index) do |p|
+        p['name'] == 'cv' && p['organization_id'] == 1
+      end
+      ex.returns(index_response([{'id' => 3}]))
+
+      ex = api_expects(:content_view_versions, :index) do |p|
+        p['version'] == '1' && p['content_view_id'] == 3
+      end
+      ex.returns(index_response([{'id' => 6}]))
+
+      ex = api_expects(:content_view_versions, :republish_repositories) do |p|
+        p['id'] == 6
+      end
+      ex.returns('state' => 'pending', 'id' => '3')
+
+      expect_foreman_task('3')
+
+      result = run_cmd(%w(content-view version republish-repositories --organization org1
+                          --content-view cv --version 1 --force true))
+
+      assert_equal(result.exit_code, 0)
+    end
+  end
+end

data/test/functional/content_view/version/update_test.rb

@@ -0,0 +1,48 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_version'
+
+module HammerCLIKatello
+  describe ContentViewVersion::UpdateCommand do
+    include OrganizationHelpers
+    it 'allows minimal options' do
+      ex = api_expects(:content_view_versions, :update).with_params('description' => 'pizza')
+      ex.returns('id' => '2', 'description' => 'pizza')
+
+      result = run_cmd(%w(content-view version update --id 2 --description pizza))
+      assert_equal(result.exit_code, 0)
+    end
+  end
+
+  describe 'content view options' do
+    it 'allows content view id with one version' do
+      ex = api_expects(:content_view_versions, :index)
+      ex.returns('id' => '1', 'content_view' => [{'id' => '2'}])
+
+      ex = api_expects(:content_view_versions, :update).with_params('description' => 'pizza')
+      ex.returns('id' => '1', 'description' => 'pizza', 'content_view' => [{'id' => '2'}])
+
+      result = run_cmd(%w(content-view version update --description pizza --content-view-id 2))
+      assert_equal(result.exit_code, 0)
+    end
+
+    it 'allows content view id with multiple versions' do
+      ex = api_expects(:content_view_versions, :index)
+      ex.returns('id' => '2', 'content_view' => [{'id' => '3'}])
+
+      ex = api_expects(:content_view_versions, :update).with_params('description' => 'pizza')
+      ex.returns('id' => '2', 'description' => 'pizza', 'content_view' => [{'id' => '3'}])
+
+      result = run_cmd(%w(content-view version update --description pizza
+                          --content-view-id 3 --version 2))
+      assert_equal(result.exit_code, 0)
+    end
+
+    it 'blocks update without description' do
+      ex = api_expects(:content_view_versions, :index)
+      ex.returns('id' => '1', 'content_view' => [{'id' => '2'}])
+
+      result = run_cmd(%w(content-view version update --content-view-id 2))
+      assert_equal(result.exit_code, 64)
+    end
+  end
+end

data/test/functional/erratum/erratum_helpers.rb

@@ -0,0 +1,91 @@
+module ErratumHelpers
+  # rubocop:disable MethodLength
+  def make_erratum_response(with_packages = true, with_module_streams = true)
+    packs = with_packages ? packages : []
+    if with_module_streams
+      module_streams = [
+        {
+          "name" => module_stream[:name],
+          "stream" => module_stream[:stream],
+          "version" => "20180730223407",
+          "context" => "deadbeef",
+          "arch" => "noarch",
+          "id" => 4,
+          "packages" => packs
+        }
+      ]
+    else
+      module_streams = []
+    end
+    {
+      "id" => 6,
+      "pulp_id" => "a4999071-2e2a-4293-9a72-219924032633",
+      "title" => "Duck_Kangaroo_Erratum",
+      "errata_id" => "RHEA-2012:0059",
+      "issued" => "2018-01-27",
+      "updated" => "2018-07-20",
+      "severity" => "",
+      "description" => "Duck_Kangaro_Erratum description",
+      "solution" => "",
+      "summary" => "",
+      "reboot_suggested" => false,
+      "uuid" => "a4999071-2e2a-4293-9a72-219924032633",
+      "name" => "Duck_Kangaroo_Erratum",
+      "type" => "enhancement",
+      "cves" => [],
+      "bugs" => [],
+      "hosts_available_count" => 0,
+      "hosts_applicable_count" => 0,
+      "packages" => packages,
+      "module_streams" => module_streams
+    }
+  end
+
+  def packages
+    ["kangaroo-0.3-1.noarch", "bar"]
+  end
+
+  def module_stream
+    {name: "MyMOdule", stream: "cool-100"}
+  end
+
+  def erratum_id
+    "1"
+  end
+
+  def host_id
+    1
+  end
+
+  def verify_module_packages_and_orphan_packages(result)
+    result_out = result.out
+    # output looks like
+    # => "Title:            Duck_Kangaroo_Erratum\n
+    # Description:      Duck_Kangaro_Erratum description\n
+    # ID:               6
+    # \nErrata ID:        RHEA-2012:0059\nReboot Suggested: false\n
+    # Updated:          2018-07-20\nIssued:           2018-01-27\nSolution:
+    # \nPackages:         kangaroo-0.3-1.noarch
+    #\nModule Streams:   \n 1) Name:     kangaroo\n    Stream:   0
+    # \n    Packages: kangaroo-0.3-1.noarch\n\n
+    module_streams_out = result_out.slice(result_out.index("Module Streams")..-1)
+    package_out = result_out.slice(0..result_out.index("Module Streams"))
+
+    assert_includes(module_streams_out, "Name")
+    assert_includes(module_streams_out, module_stream[:name])
+    assert_includes(module_streams_out, "Stream")
+    assert_includes(module_streams_out, module_stream[:stream])
+    assert_includes(module_streams_out, packages.first)
+    assert_includes(module_streams_out, packages.last)
+
+    assert_includes(package_out, packages.first)
+    assert_includes(package_out, packages.last)
+  end
+
+  def verify_no_modules_packages_with_orphan_packages(result)
+    result_out = result.out
+    refute_includes(result_out, "Module Streams")
+    assert_includes(result_out, packages.first)
+    assert_includes(result_out, packages.last)
+  end
+end

data/test/functional/erratum/info_test.rb

@@ -0,0 +1,29 @@
+require_relative '../test_helper'
+require_relative 'erratum_helpers'
+
+describe 'Erratum Info' do
+  include ErratumHelpers
+
+  before do
+    @cmd = %w(erratum info)
+  end
+
+  it "prints module package info on run" do
+    api_expects(:errata, :show)
+      .with_params('id' => erratum_id)
+      .returns(make_erratum_response)
+    params = ["--id=#{erratum_id}"]
+    result = run_cmd(@cmd + params)
+    verify_module_packages_and_orphan_packages(result)
+  end
+
+  it "prints package info on run" do
+    api_expects(:errata, :show)
+      .with_params('id' => erratum_id)
+      .returns(make_erratum_response(true, false))
+
+    params = ["--id=#{erratum_id}"]
+    result = run_cmd(@cmd + params)
+    verify_no_modules_packages_with_orphan_packages(result)
+  end
+end

data/test/functional/erratum/list_test.rb

@@ -0,0 +1,108 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/erratum'
+
+module HammerCLIKatello
+  describe ErratumCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:errata, :index)
+
+      run_cmd(%w(erratum list))
+    end
+
+    describe 'product options' do
+      it 'may be specified by ID' do
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 2}]))
+
+        api_expects(:errata, :index) do |p|
+          p['repository_id'] = 2
+        end
+
+        run_cmd(%w(erratum list --product-id 1))
+      end
+
+      it 'fail if more than one repository is found' do
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 2}, {'id' => 3}]))
+
+        r = run_cmd(%w(erratum list --product-id 1))
+        assert r.err.include?("Found more than one repository."), r.err
+      end
+
+      it 'requires organization options to resolve ID by name' do
+        api_expects_no_call
+
+        r = run_cmd(%w(erratum list --product product1))
+        assert r.err.include? "--organization-id, --organization, --organization-label is required"
+      end
+
+      it 'allows organization ID when resolving ID by name' do
+        ex = api_expects(:products, :index) do |p|
+          p['name'] == 'product1' && p['organization_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 2}]))
+
+        api_expects(:errata, :index) do |p|
+          p['repository_id'] = 2
+        end
+
+        run_cmd(%w(erratum list --product product1 --organization-id 3))
+      end
+
+      it 'allows organization name when resolving ID by name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org3\""
+        end
+        ex.at_least_once.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:products, :index) do |p|
+          p['name'] == 'product1' && p['organization_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 2}]))
+
+        api_expects(:errata, :index) do |p|
+          p['repository_id'] = 2
+        end
+
+        run_cmd(%w(erratum list --product product1 --organization org3))
+      end
+
+      it 'allows organization label when resolving ID by name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "label = \"org3\""
+        end
+        ex.at_least_once.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:products, :index) do |p|
+          p['name'] == 'product1' && p['organization_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 2}]))
+
+        api_expects(:errata, :index) do |p|
+          p['repository_id'] = 2
+        end
+
+        run_cmd(%w(erratum list --product product1 --organization-label org3))
+      end
+    end
+  end
+end

data/test/functional/file/file_helpers.rb

@@ -0,0 +1,13 @@
+require_relative '../search_helpers'
+
+module FileHelpers
+  include SearchHelpers
+
+  def expect_file_search(params, returns)
+    expect_generic_search(:file_units, params: params, returns: returns)
+  end
+
+  def expect_file_show(params)
+    api_expects(:file_units, :show).with_params(params)
+  end
+end

data/test/functional/file/info_test.rb

@@ -0,0 +1,162 @@
+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_relative 'file_helpers'
+require 'hammer_cli_katello/file'
+
+# rubocop:disable ModuleLength
+module HammerCLIKatello
+  describe FileCommand::InfoCommand do
+    include FileHelpers
+    include ContentViewHelpers
+    include RepositoryHelpers
+    include ProductHelpers
+    include OrganizationHelpers
+
+    it 'allows minimal options' do
+      expect_file_show('id' => '1')
+      run_cmd(%w(file info --id 1))
+    end
+
+    describe 'requires' do
+      describe 'organization options' do
+        it 'to resolve product ID' do
+          api_expects_no_call
+          result = run_cmd(%w(file list --product product1))
+          assert_match(/--organization-id/, result.err)
+        end
+
+        it 'to resolve content view ID' do
+          api_expects_no_call
+          result = run_cmd(%w(file list --content-view cv1))
+          assert_match(/--organization-id/, result.err)
+        end
+      end
+
+      describe 'product options' do
+        it 'to resolve repository ID' do
+          api_expects_no_call
+          result = run_cmd(%w(file list --repository repo1))
+          assert_match(/--product-id/, result.err)
+        end
+      end
+    end
+
+    describe 'allows filtering by' do
+      it 'repository ID' do
+        expect_file_search({'repository_id' => 2}, {})
+        run_cmd(%w(file list --repository-id 2))
+      end
+
+      it 'content view version ID' do
+        expect_file_search({'content_view_version_id' => 2}, {})
+        run_cmd(%w(file list --content-view-version-id 2))
+      end
+    end
+
+    describe 'resolves ID from file name' do
+      it 'and repository ID' do
+        expect_file_search({'repository_id' => '2', search: "name = \"foo\""}, 'id' => 1)
+        expect_file_show('id' => 1)
+        run_cmd(%w(file info --name foo --repository-id 2))
+      end
+
+      describe 'repository name' do
+        it 'and product ID' do
+          expect_generic_repositories_search({'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+          expect_file_search({'repository_id' => '2', search: "name = \"foo\""}, 'id' => 1)
+          expect_file_show('id' => 1)
+          run_cmd(%w(file info --name foo --repository repo2 --product-id 1))
+        end
+
+        describe 'product name, and organization' do
+          it 'ID' do
+            expect_generic_product_search({'name' => 'product1', 'organization_id' => 3}, 'id' => 1)
+            expect_generic_repositories_search({'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+            expect_file_search({'repository_id' => '2', search: "name = \"foo\""}, 'id' => 1)
+            expect_file_show('id' => 1)
+            run_cmd(%w(file info --name foo --repository repo2 --product product1
+                       --organization-id 3))
+          end
+
+          it 'name' do
+            expect_organization_search('org3', 3)
+            expect_generic_product_search({'name' => 'product1', 'organization_id' => 3}, 'id' => 1)
+            expect_generic_repositories_search({'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+            expect_file_search({'repository_id' => '2', search: "name = \"foo\""}, 'id' => 1)
+            expect_file_show('id' => 1)
+            run_cmd(%w(file info --name foo --repository repo2 --product product1
+                       --organization org3))
+          end
+
+          it 'label' do
+            expect_organization_search('org3', 3, field: 'label')
+            expect_generic_product_search({'name' => 'product1', 'organization_id' => 3}, 'id' => 1)
+            expect_generic_repositories_search({'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+            expect_file_search({'repository_id' => '2', search: "name = \"foo\""}, 'id' => 1)
+            expect_file_show('id' => 1)
+            run_cmd(%w(file info --name foo --repository repo2 --product product1
+                       --organization-label org3))
+          end
+        end
+      end
+
+      it 'and content view version ID' do
+        expect_file_search({'content_view_version_id' => '2', 'name' => 'foo'}, 'id' => 1)
+        expect_file_show('id' => 1)
+        run_cmd(%w(file info --name foo --content-view-version-id 2))
+      end
+
+      describe 'content view version' do
+        it 'and content view ID' do
+          expect_content_view_version_search({'version' => '2.0',
+                                              'content_view_id' => 3}, 'id' => 2)
+          expect_file_search({'content_view_version_id' => '2',
+                              search: "name = \"foo\""}, 'id' => 1)
+          expect_file_show('id' => 1)
+          run_cmd(%w(file info --name foo --content-view-version 2.0 --content-view-id 3))
+        end
+
+        describe 'content view name, and organization' do
+          it 'ID' do
+            expect_content_view_search(4, 'cv3', 3)
+            expect_content_view_version_search({'version' => '2.0',
+                                                'content_view_id' => 3}, 'id' => 2)
+            expect_file_search({'content_view_version_id' => '2',
+                                search: "name = \"foo\""}, 'id' => 1)
+            expect_file_show('id' => 1)
+            run_cmd(%w(file info --name foo --content-view-version 2.0 --content-view cv3
+                       --organization-id 4))
+          end
+
+          it 'name' do
+            expect_organization_search('org4', 4)
+            expect_content_view_search(4, 'cv3', 3)
+            expect_content_view_version_search({'version' => '2.0',
+                                                'content_view_id' => 3}, 'id' => 2)
+            expect_file_search({'content_view_version_id' => '2',
+                                search: "name = \"foo\""}, 'id' => 1)
+            expect_file_show('id' => 1)
+            run_cmd(%w(file info --name foo --content-view-version 2.0 --content-view cv3
+                       --organization org4))
+          end
+
+          it 'label' do
+            expect_organization_search('org4', 4, field: 'label')
+            expect_content_view_search(4, 'cv3', 3)
+            expect_content_view_version_search({'version' => '2.0',
+                                                'content_view_id' => 3}, 'id' => 2)
+            expect_file_search({'content_view_version_id' => '2',
+                                search: "name = \"foo\""}, 'id' => 1)
+            expect_file_show('id' => 1)
+            run_cmd(%w(file info --name foo --content-view-version 2.0 --content-view cv3
+                       --organization-label org4))
+          end
+        end
+      end
+    end
+  end
+end
+# rubocop:enable ModuleLength

data/test/functional/file/list_test.rb

@@ -0,0 +1,112 @@
+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_relative 'file_helpers'
+require 'hammer_cli_katello/file'
+
+module HammerCLIKatello
+  describe FileCommand::ListCommand do
+    include FileHelpers
+    include ContentViewHelpers
+    include RepositoryHelpers
+    include ProductHelpers
+    include OrganizationHelpers
+
+    it 'allows minimal options' do
+      expect_file_search({}, {})
+      run_cmd(%w(file list))
+    end
+
+    describe 'requires' do
+      describe 'organization options' do
+        it 'to resolve product ID' do
+          api_expects_no_call
+          result = run_cmd(%w(file list --product product1))
+          assert_match(/--organization-id/, result.err)
+        end
+
+        it 'to resolve content view ID' do
+          api_expects_no_call
+          result = run_cmd(%w(file list --content-view cv1))
+          assert_match(/--organization-id/, result.err)
+        end
+      end
+
+      describe 'product options' do
+        it 'to resolve repository ID' do
+          api_expects_no_call
+          result = run_cmd(%w(file list --repository repo1))
+          assert_match(/--product-id/, result.err)
+        end
+      end
+    end
+
+    describe 'allows filtering by' do
+      it 'repository ID' do
+        expect_file_search({'repository_id' => 2}, {})
+        run_cmd(%w(file list --repository-id 2))
+      end
+
+      it 'content view version ID' do
+        expect_file_search({'content_view_version_id' => 2}, {})
+        run_cmd(%w(file list --content-view-version-id 2))
+      end
+    end
+
+    describe 'resolves' do
+      describe 'repository ID' do
+        it 'from name and product ID' do
+          expect_generic_repositories_search(
+            {'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+          expect_file_search({'repository_id' => 2}, {})
+          run_cmd(%w(file list --repository repo2 --product-id 1))
+        end
+
+        it 'from name, product, and organization ID' do
+          expect_generic_product_search({'name' => 'product1', 'organization_id' => 3}, 'id' => 1)
+          expect_generic_repositories_search({'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+          expect_file_search({'repository_id' => 2}, {})
+          run_cmd(%w(file list --repository repo2 --product product1 --organization-id 3))
+        end
+
+        it 'from name, product, and organization name' do
+          expect_organization_search('org3', 3)
+          expect_generic_product_search({'name' => 'product1', 'organization_id' => 3}, 'id' => 1)
+          expect_generic_repositories_search({'name' => 'repo2', 'product_id' => 1}, 'id' => 2)
+          expect_file_search({'repository_id' => 2}, {})
+          run_cmd(%w(file list --repository repo2 --product product1 --organization org3))
+        end
+      end
+
+      describe 'content view version ID' do
+        it 'from version and content view ID' do
+          expect_content_view_version_search(
+            {'version' => '1.0', 'content_view_id' => 1}, 'id' => 2)
+          expect_file_search({'content_view_version_id' => 2}, {})
+          run_cmd(%w(file list --content-view-version 1.0 --content-view-id 1))
+        end
+
+        it 'from version, content view name, and organization ID' do
+          expect_generic_content_view_search(params: {'name' => 'cv1', 'organization_id' => 3},
+                                             returns: {'id' => 1})
+          expect_content_view_version_search(
+            {'version' => '1.0', 'content_view_id' => 1}, 'id' => 2)
+          expect_file_search({'content_view_version_id' => 2}, {})
+          run_cmd(%w(file list --content-view-version 1.0 --content-view cv1 --organization-id 3))
+        end
+
+        it 'from version, content view name, and organization name' do
+          expect_organization_search('org3', 3)
+          expect_generic_content_view_search(params: {'name' => 'cv1', 'organization_id' => 3},
+                                             returns: {'id' => 1})
+          expect_content_view_version_search(
+            {'version' => '1.0', 'content_view_id' => 1}, 'id' => 2)
+          expect_file_search({'content_view_version_id' => 2}, {})
+          run_cmd(%w(file list --content-view-version 1.0 --content-view cv1 --organization org3))
+        end
+      end
+    end
+  end
+end