hammer_cli_katello 1.1.3 → 1.2.0

data/test/functional/content_view/remove_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::RemoveContentViewVersionCommand 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, 6])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3)
+      end
+      run_cmd(%w(content-view remove-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, 6])
+      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)
+      end
+      run_cmd(%w(content-view remove-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 remove-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, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-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, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-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, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-version --name cv2 --organization-label org1
+                   --content-view-version-id 3))
+      end
+    end
+  end
+end

data/test/functional/content_view/remove_test.rb

@@ -0,0 +1,79 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::RemoveCommand do
+    include ForemanTaskHelpers
+
+    describe 'content view version options' do
+      it 'allows removing versions by ID' do
+        ex = api_expects(:content_views, :remove) do |p|
+          p['id'] == 1 && p['content_view_version_ids'] == %w(6 7 8)
+        end
+        ex.returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --content-view-version-ids 6,7,8))
+      end
+
+      it 'allows removing versions by version number' do
+        versions = %w(6.0 7.0 8.0)
+        ids = [6, 7, 8]
+
+        api_expects(:content_view_versions, :index).with_params(
+          'search' => %(version = "#{versions.join('" or version = "')}"),
+          'content_view_id' => 1
+        ).returns(index_response(ids.map { |v| { 'id' => v } }))
+
+        api_expects(:content_views, :remove).with_params(
+          'id' => 1, 'content_view_version_ids' => ids
+        ).returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --content-view-versions 6.0,7.0,8.0))
+      end
+    end
+
+    describe 'environment options' do
+      it 'allows removing versions by ID' do
+        ex = api_expects(:content_views, :remove) do |p|
+          p['id'] == 1 && p['environment_ids'] == %w(6 7 8)
+        end
+        ex.returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --environment-ids 6,7,8))
+      end
+
+      it 'requires organization options when removing environments by name' do
+        api_expects_no_call
+
+        run_cmd(%w(content-view remove --id 1 --environments env6,env7,env8))
+      end
+
+      it 'allows removing environments by name' do
+        environment_ids = [6, 7, 8]
+        all_environment_ids = [3, 4, 5, 6, 7, 8]
+
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p['organization_id'] == '1'
+        end
+        ex.returns(index_response(all_environment_ids.map do |id|
+          {'id' => id, 'name' => "env#{id}"}
+        end))
+
+        ex = api_expects(:content_views, :remove) do |p|
+          p['id'] == 1 && p['environment_ids'] == environment_ids
+        end
+        ex.returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --environments env6,env7,env8 --organization-id 1))
+      end
+    end
+  end
+end

data/test/functional/content_view/update_test.rb

@@ -0,0 +1,53 @@
+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::UpdateCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
+    it 'allows minimal options' do
+      api_expects(:content_views, :update) do |p|
+        p['id'] == 2
+      end
+      run_cmd(%w(content-view update --id 2))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view update --name cv2))
+        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)
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2
+        end
+        run_cmd(%w(content-view update --name cv2 --organization-id 1))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2
+        end
+        run_cmd(%w(content-view update --name cv2 --organization org1))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2
+        end
+        run_cmd(%w(content-view update --name cv2 --organization-label org1))
+      end
+    end
+  end
+end

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

@@ -0,0 +1,108 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../../organization/organization_helpers')
+require File.join(File.dirname(__FILE__),
+                  '../../lifecycle_environment/lifecycle_environment_helpers')
+
+describe 'content-view version incremental-update' do
+  include OrganizationHelpers
+  include LifecycleEnvironmentHelpers
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-view version incremental-update)
+  end
+
+  it "performs incremental update with no updates" do
+    params = ['--errata-ids=FOO2012', '--lifecycle-environment-ids=1,2,3',
+              '--content-view-version-id=5']
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts'].nil? &&
+        par[:content_view_version_environments][0][:environment_ids] == %w(1 2 3) &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with update all hosts" do
+    params = ['--update-all-hosts=true', '--errata-ids=FOO2012',
+              '--lifecycle-environment-ids=1,2,3', '--content-view-version-id=5']
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts']['included'][:search] == '' &&
+        par[:content_view_version_environments][0][:environment_ids] == %w(1 2 3) &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with no environment" do
+    params = ['--update-all-hosts=true', '--errata-ids=FOO2012', '--content-view-version-id=5']
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts']['included'][:search] == '' &&
+        par[:content_view_version_environments][0][:environment_ids].nil? &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with names" do
+    params = ['--update-all-hosts=true', '--errata-ids=FOO2012',
+              '--lifecycle-environments=trump,cruz,bernie',
+              '--content-view-version-id=5', '--organization=USA']
+
+    expect_organization_search('USA', 5)
+    expect_lifecycle_environments_request(5, [{'name' => 'trump', 'id' => 1},
+                                              {'name' => 'cruz', 'id' => 2},
+                                              {'name' => 'bernie', 'id' => 3}])
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts']['included'][:search] == '' &&
+        par[:content_view_version_environments][0][:environment_ids] == [1, 2, 3] &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with packages" do
+    params = ['--content-view-version-id=5', '--packages=bla', '--lifecycle-environment-ids=1']
+
+    expect_generic_search(
+      :packages, params: {search: "name = \"bla\""}, returns: {'id' => 15}
+    )
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par["add_content"]["package_ids"] == [15]
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+    expect_foreman_task('3')
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+end

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

@@ -0,0 +1,44 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_version'
+
+module HammerCLIKatello
+  describe ContentViewVersion::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:content_view_versions, :index)
+
+      run_cmd(%w(content-view version list))
+    end
+
+    describe 'allows organization' do
+      it 'id' do
+        api_expects(:content_view_versions, :index) do |p|
+          p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view version list --organization-id 1))
+      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_versions, :index) do |p|
+          p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view version list --organization org1))
+      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_versions, :index) do |p|
+          p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view version list --organization-label org1))
+      end
+    end
+  end
+end

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

@@ -0,0 +1,34 @@
+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
+  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))
+
+      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