hammer_cli_katello 1.1.3 → 1.2.0

data/test/functional/filter_rule/info_test.rb

@@ -0,0 +1,104 @@
+require_relative '../test_helper'
+require_relative 'filter_rule_helpers'
+require 'hammer_cli_katello/filter_rule'
+
+module HammerCLIKatello
+  describe FilterRule::InfoCommand do
+    include FilterRuleHelpers
+
+    it 'allows minimal options' do
+      api_expects(:content_view_filter_rules, :show) do |p|
+        p['content_view_filter_id'] == 1 && p['id'] == 9
+      end
+      run_cmd(%w(content-view filter rule info --content-view-filter-id 1 --id 9))
+    end
+
+    it 'resolves rule ID from rule name and filter ID' do
+      expect_filter_rule_search('rule9', 1, 9)
+
+      api_expects(:content_view_filter_rules, :show) do |p|
+        p['content_view_filter_id'] == 1 && p['id'] == 9
+      end
+      run_cmd(%w(content-view filter rule info --content-view-filter-id 1 --name rule9))
+    end
+
+    it 'allows name resolution of filter with content-view-id' do
+      ex = api_expects(:content_view_filters, :index) do |p|
+        p['name'] == 'cvf1' && p['content_view_id'] == 3
+      end
+      ex.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_filter_rules, :show) do |p|
+        p['content_view_filter_id'] == 1 && p['id'] == 9
+      end
+      run_cmd(%w(content-view filter rule info --content-view-filter cvf1 --content-view-id 3
+                 --id 9))
+    end
+
+    describe 'organization' do
+      it 'ID can be specified to resolve content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == '6'
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :show) do |p|
+          p['content_view_filter_id'] == 1 && p['id'] == 9
+        end
+        run_cmd(%w(content-view filter rule info --content-view-filter cvf1 --organization-id 6
+                   --content-view cv3 --id 9))
+      end
+
+      it 'name can be specified to resolve content view name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org6\""
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == 6
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :show) do |p|
+          p['content_view_filter_id'] == 1 && p['id'] == 9
+        end
+        run_cmd(%w(content-view filter rule info --content-view-filter cvf1 --organization org6
+                   --content-view cv3 --id 9))
+      end
+
+      it 'label can be specified to resolve content view name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "label = \"org6\""
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == 6
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :show) do |p|
+          p['content_view_filter_id'] == 1 && p['id'] == 9
+        end
+        run_cmd(%w(content-view filter rule info --content-view-filter cvf1 --organization-label
+                   org6 --content-view cv3 --id 9))
+      end
+    end
+  end
+end

data/test/functional/filter_rule/list_test.rb

@@ -0,0 +1,91 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/filter_rule'
+
+module HammerCLIKatello
+  describe FilterRule::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:content_view_filter_rules, :index) do |p|
+        p['content_view_filter_id'] == 1
+      end
+      run_cmd(%w(content-view filter rule list --content-view-filter-id 1))
+    end
+
+    it 'allows name resolution with content-view-id' do
+      ex = api_expects(:content_view_filters, :index) do |p|
+        p['name'] == 'cvf1' && p['content_view_id'] == 3
+      end
+      ex.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_filter_rules, :index) do |p|
+        p['content_view_filter_id'] == 1
+      end
+      run_cmd(%w(content-view filter rule list --content-view-filter cvf1 --content-view-id 3))
+    end
+
+    describe 'organization' do
+      it 'ID can be specified to resolve content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == '6'
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :index) do |p|
+          p['content_view_filter_id'] == 1
+        end
+        run_cmd(%w(content-view filter rule list --content-view-filter cvf1 --organization-id 6
+                   --content-view cv3))
+      end
+
+      it 'name can be specified to resolve content view name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org6\""
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == 6
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :index) do |p|
+          p['content_view_filter_id'] == 1
+        end
+        run_cmd(%w(content-view filter rule list --content-view-filter cvf1 --organization org6
+                   --content-view cv3))
+      end
+
+      it 'label can be specified to resolve content view name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "label = \"org6\""
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == 6
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :index) do |p|
+          p['content_view_filter_id'] == 1
+        end
+        run_cmd(%w(content-view filter rule list --content-view-filter cvf1 --organization-label
+                   org6 --content-view cv3))
+      end
+    end
+  end
+end

data/test/functional/filter_rule/update_test.rb

@@ -0,0 +1,104 @@
+require_relative '../test_helper'
+require_relative 'filter_rule_helpers'
+require 'hammer_cli_katello/filter_rule'
+
+module HammerCLIKatello
+  describe FilterRule::UpdateCommand do
+    include FilterRuleHelpers
+
+    it 'allows minimal options' do
+      api_expects(:content_view_filter_rules, :update) do |p|
+        p['content_view_filter_id'] == 1 && p['id'] == 9
+      end
+      run_cmd(%w(content-view filter rule update --content-view-filter-id 1 --id 9))
+    end
+
+    it 'resolves rule ID from rule name and filter ID' do
+      expect_filter_rule_search('rule9', 1, 9)
+
+      api_expects(:content_view_filter_rules, :update) do |p|
+        p['content_view_filter_id'] == 1 && p['id'] == 9
+      end
+      run_cmd(%w(content-view filter rule update --content-view-filter-id 1 --name rule9))
+    end
+
+    it 'allows name resolution of filter with content-view-id' do
+      ex = api_expects(:content_view_filters, :index) do |p|
+        p['name'] == 'cvf1' && p['content_view_id'] == 3
+      end
+      ex.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_filter_rules, :update) do |p|
+        p['content_view_filter_id'] == 1 && p['id'] == 9
+      end
+      run_cmd(%w(content-view filter rule update --content-view-filter cvf1 --content-view-id 3
+                 --id 9))
+    end
+
+    describe 'organization' do
+      it 'ID can be specified to resolve content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == '6'
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :update) do |p|
+          p['content_view_filter_id'] == 1 && p['id'] == 9
+        end
+        run_cmd(%w(content-view filter rule update --content-view-filter cvf1 --organization-id 6
+                   --content-view cv3 --id 9))
+      end
+
+      it 'name can be specified to resolve content view name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org6\""
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == 6
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :update) do |p|
+          p['content_view_filter_id'] == 1 && p['id'] == 9
+        end
+        run_cmd(%w(content-view filter rule update --content-view-filter cvf1 --organization org6
+                   --content-view cv3 --id 9))
+      end
+
+      it 'label can be specified to resolve content view name' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "label = \"org6\""
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv3' && p['organization_id'] == 6
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:content_view_filters, :index) do |p|
+          p['name'] == 'cvf1' && p['content_view_id'] == 3
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_view_filter_rules, :update) do |p|
+          p['content_view_filter_id'] == 1 && p['id'] == 9
+        end
+        run_cmd(%w(content-view filter rule update --content-view-filter cvf1 --organization-label
+                   org6 --content-view cv3 --id 9))
+      end
+    end
+  end
+end

data/test/functional/host/errata/apply_test.rb

@@ -0,0 +1,45 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+
+describe 'apply an errata' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(host errata apply)
+  end
+
+  let(:errata_id) { "RHEA-1111:1111" }
+  let(:host_id) { 1 }
+  let(:task_id) { '5' }
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'stopped'
+    }
+  end
+
+  it "applies an errata to a host" do
+    params = ["--errata-ids=#{errata_id}", "--host-id=#{host_id}"]
+
+    ex = api_expects(:host_errata, :apply, 'Host errata apply') do |par|
+      par['errata_ids'] == [errata_id] && par['host_id'] == host_id
+    end
+
+    ex.returns(response)
+
+    expect_foreman_task(task_id)
+
+    run_cmd(@cmd + params)
+  end
+
+  it "applies an errata to a host with async flag" do
+    params = ["--errata-ids=#{errata_id}", "--host-id=#{host_id}", "--async"]
+
+    ex = api_expects(:host_errata, :apply, 'Host errata apply') do |par|
+      par['errata_ids'] == [errata_id] && par['host_id'] == host_id
+    end
+
+    ex.returns(response)
+
+    run_cmd(@cmd + params)
+  end
+end

data/test/functional/host/errata/info_test.rb

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

data/test/functional/host/errata/list_test.rb

@@ -0,0 +1,49 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require_relative '../../lifecycle_environment/lifecycle_environment_helpers'
+
+describe 'host errata listing' do
+  include LifecycleEnvironmentHelpers
+
+  before do
+    @cmd = %w(host errata list)
+  end
+
+  let(:org_id) { 1 }
+  let(:host_id) { 2 }
+  let(:lifecycle_env_id) { 3 }
+  let(:empty_response) do
+    {
+      "total" => 0,
+      "subtotal" => 0,
+      "page" => "1",
+      "per_page" => "1000",
+      "error" => nil,
+      "search" => nil,
+      "sort" => {
+        "by" => nil,
+        "order" => nil
+      },
+      "results" => []
+    }
+  end
+
+  it "lists the host errata belonging to a lifecycle-environment by name" do
+    params = ["--host-id=#{host_id}", "--organization-id=#{org_id}", '--lifecycle-environment=test']
+
+    expect_lifecycle_environment_search(org_id.to_s, 'test', lifecycle_env_id)
+
+    ex = api_expects(:host_errata, :index, 'host errata list').
+         with_params('host_id': host_id,
+                     'environment_id': lifecycle_env_id,
+                     'page': 1,
+                     'per_page': 1000)
+
+    ex.returns(empty_response)
+    expected_result = success_result("---|------------|------|-------|------------
+ID | ERRATUM ID | TYPE | TITLE | INSTALLABLE
+---|------------|------|-------|------------
+")
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/host/errata/recalculate_test.rb

@@ -0,0 +1,40 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+
+describe 'recalculate errata' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(host errata recalculate)
+  end
+
+  let(:host_id) { 1 }
+  let(:task_id) { '5' }
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'stopped'
+    }
+  end
+
+  it "recalculates errata for a host" do
+    params = ["--host-id=#{host_id}"]
+
+    api_expects(:host_errata, :applicability, 'Host errata recalculate')
+      .with_params('host_id' => host_id)
+      .returns(response)
+
+    expect_foreman_task(task_id)
+
+    run_cmd(@cmd + params)
+  end
+
+  it "applies an errata to a host with async flag" do
+    params = ["--host-id=#{host_id}", "--async"]
+
+    api_expects(:host_errata, :applicability, 'Host errata recalculate')
+      .with_params('host_id' => host_id)
+      .returns(response)
+
+    run_cmd(@cmd + params)
+  end
+end

data/test/functional/host/extensions/create_test.rb

@@ -0,0 +1,95 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_foreman/host'
+
+module HammerCLIForeman
+  describe Host do
+    # These tests are only for the extensions Katello adds to the host command
+    # See hammer-cli-foreman for the core host tests
+    describe CreateCommand do
+      let(:env_name) { "world" }
+      let(:env_id) { 100 }
+      let(:cv_id) { 220 }
+      let(:repo_name) { "life_of_kickstart" }
+      let(:repo_id) { 12 }
+      let(:content_source_name) { "life_of_cs" }
+      let(:content_source_id) { 82 }
+      let(:host_name) { "mercury" }
+      let(:domain_id) { 134 }
+      let(:operatingsystem_id) { 103 }
+      let(:architecture_id) { 28 }
+      let(:partition_table_id) { 243 }
+      let(:location_id) { 213 }
+      let(:organization_id) { 324 }
+
+      it 'allows kickstart repository name' do
+        api_expects(:lifecycle_environments, :index).
+          with_params('name' => env_name,
+                      'organization_id' => organization_id).
+          returns(index_response([{'id' => env_id}]))
+
+        api_expects(:repositories, :index).
+          with_params('name' => repo_name,
+                      'environment_id' => env_id,
+                      'content_view_id' => cv_id).
+          returns(index_response([{'id' => repo_id}]))
+
+        api_expects(:hosts, :create).
+          with_params('host' => {
+                        'name' => host_name,
+                        'location_id' => location_id,
+                        'organization_id' => organization_id,
+                        'operatingsystem_id' => operatingsystem_id,
+                        'architecture_id' => architecture_id,
+                        'ptable_id' => partition_table_id,
+                        'domain_id' => domain_id,
+                        'content_facet_attributes' => {
+                          'lifecycle_environment_id' => env_id,
+                          'content_view_id' => cv_id,
+                          'kickstart_repository_id' => repo_id
+                        }
+                      })
+
+        cmd = "host create --name=#{host_name} --content-view-id=#{cv_id}"\
+              " --lifecycle-environment=#{env_name}"\
+              " --kickstart-repository=#{repo_name}"\
+              " --operatingsystem-id=#{operatingsystem_id}"\
+              " --architecture-id=#{architecture_id}"\
+              " --organization-id=#{organization_id}"\
+              " --partition-table-id=#{partition_table_id}"\
+              " --location-id=#{location_id}"\
+              " --domain-id=#{domain_id}"
+        run_cmd(cmd.split)
+      end
+
+      it 'allows content source name' do
+        api_expects(:smart_proxies, :index).
+          with_params(:search => "name = \"#{content_source_name}\"").
+          returns(index_response([{'id' => content_source_id}]))
+
+        api_expects(:hosts, :create).
+          with_params('host' => {
+                        'name' => host_name,
+                        'location_id' => location_id,
+                        'organization_id' => organization_id,
+                        'operatingsystem_id' => operatingsystem_id,
+                        'architecture_id' => architecture_id,
+                        'ptable_id' => partition_table_id,
+                        'domain_id' => domain_id,
+                        'content_facet_attributes' => {
+                          'content_source_id' => content_source_id
+                        }
+                      })
+
+        cmd = "host create --name=#{host_name}"\
+              " --content-source=#{content_source_name}"\
+              " --operatingsystem-id=#{operatingsystem_id}"\
+              " --architecture-id=#{architecture_id}"\
+              " --organization-id=#{organization_id}"\
+              " --partition-table-id=#{partition_table_id}"\
+              " --location-id=#{location_id}"\
+              " --domain-id=#{domain_id}"
+        run_cmd(cmd.split)
+      end
+    end
+  end
+end