hammer_cli_katello 0.3.0 → 0.10.0

data/test/functional/content_view/remove_test.rb

@@ -0,0 +1,78 @@
+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
+        %w(6.0 7.0 8.0).each do |version|
+          ex = api_expects(:content_view_versions, :index) do |p|
+            p['version'] == version && p['content_view_id'] == 1
+          end
+          ex.returns(index_response('id' => version.to_i))
+        end
+        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-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/republish_repositories_test.rb

@@ -0,0 +1,35 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_puppet_module'
+
+module HammerCLIKatello
+  describe ContentViewPuppetModule 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/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"
+      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/host/subscription/content_override_test.rb

@@ -0,0 +1,95 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../host_helpers')
+require 'hammer_cli_katello/host_subscription'
+
+describe 'host subscription content-override' do
+  include HostHelpers
+
+  before do
+    @cmd = %w(host subscription content-override)
+  end
+
+  it "attaches a content label" do
+    label = "foo"
+    value = 'default'
+    id = '20'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--value=#{value}"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['value'] == value &&
+        par['content_overrides'][0]['name'] == "enabled"
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "attaches a content label with name" do
+    label = "foo"
+    value = '1'
+    id = '20'
+    name = 'protected'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--value=#{value}", "--name=#{name}"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['value'] == value &&
+        par['content_overrides'][0]['name'] == name
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "removes override" do
+    label = "foo"
+    id = '20'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--remove"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['remove'] == true &&
+        par['content_overrides'][0]['name'] == "enabled"
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "removes override with name" do
+    label = "foo"
+    id = '20'
+    name = 'protected'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--name=#{name}", "--remove"]
+    ex = api_expects(:host_subscriptions, :content_override, "content override") do |par|
+      par['host_id'].to_s == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['remove'] == true &&
+        par['content_overrides'][0]['name'] == name
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "validation fails on no override value or remove" do
+    api_expects_no_call
+    label = "foo"
+    id = '20'
+    name = 'protected'
+    params = ["--host-id=#{id}", "--content-label=#{label}", "--name=#{name}"]
+    result = run_cmd(@cmd + params)
+
+    assert(result.err[/At least one of options --remove, --value is required/],
+               "Remove or Value must be provided")
+  end
+end

data/test/functional/host/subscription/product_content_test.rb

@@ -0,0 +1,27 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../host_helpers')
+require 'hammer_cli_katello/host_subscription'
+
+describe 'listing available product content' do
+  include HostHelpers
+
+  let(:host_id) { 1 }
+  let(:empty_response_table) do
+    <<eostring
+---|------|------|-----|---------|-------|----------|---------
+ID | NAME | TYPE | URL | GPG KEY | LABEL | ENABLED? | OVERRIDE
+---|------|------|-----|---------|-------|----------|---------
+eostring
+  end
+
+  it "lists content available for a host" do
+    ex = api_expects(:host_subscriptions, :product_content) do |p|
+      p['id'] = host_id
+    end
+    ex.returns(index_response([]))
+    assert_cmd(
+      success_result(empty_response_table),
+      run_cmd(%w(host subscription product-content --host-id 1))
+    )
+  end
+end

data/test/functional/host_collection/content_api_expectations.rb

@@ -0,0 +1,35 @@
+module HammerCLIKatello
+  module ContentAPIExpectations
+    ORGANIZATION_ID = 1
+    HOST_COLLECTION_ID = 3
+    HOST_COLLECTION_NAME = 'Test'.freeze
+
+    def host_collection
+      {
+        'id' => HOST_COLLECTION_ID,
+        'name' => HOST_COLLECTION_NAME
+      }
+    end
+
+    def api_expects_content_action(action, content_type, expected_params)
+      expected_params[:organization_id] ||= ORGANIZATION_ID
+      expected_params[:host_collection_id] ||= HOST_COLLECTION_ID
+
+      api_expects(:hosts_bulk_actions, action) do |p|
+        search_query = "host_collection_id=\"#{expected_params[:host_collection_id]}\""
+        p['organization_id'] == expected_params[:organization_id] &&
+          p['included'] == { search: search_query } &&
+          p['content'] == expected_params[:content] &&
+          p['content_type'] == content_type
+      end
+    end
+
+    def api_expects_collection_search
+      expectation = api_expects(:host_collections, :index) do |p|
+        p["name"] == HOST_COLLECTION_NAME &&
+          p["organization_id"] == ORGANIZATION_ID
+      end
+      expectation.returns(index_response([host_collection]))
+    end
+  end
+end

data/test/functional/host_collection/content_install_test.rb

@@ -1,37 +1,62 @@
 require_relative '../test_helper'
+require_relative './content_api_expectations'
 require 'hammer_cli_katello/host_collection'
 
 module HammerCLIKatello
   describe HostCollection::InstallContentBaseCommand do
-    it 'installs packages to hosts in a host collection' do
-      api_expects(:hosts_bulk_actions, :install_content) do |p|
-        p['organization_id'] == 1 &&
-          p['included'] == { search: "host_collection_id=\"3\"" } &&
-          p['content'] == ['wget']
-      end
+    include HammerCLIKatello::ContentAPIExpectations
+
+    def api_expects_content_install(content_type, expected_params)
+      api_expects_content_action(:install_content, content_type, expected_params)
+    end
 
+    it 'installs packages to hosts in a host collection' do
+      api_expects_content_install('package', :content => ['wget'])
       run_cmd(%w(host-collection package install --id 3 --packages wget --organization-id 1))
     end
 
+    it 'installs packages to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('package', :content => ['wget'])
+      run_cmd(%w(host-collection package install --name Test --packages wget --organization-id 1))
+    end
+
     it 'installs package groups to hosts in a host collection' do
-      api_expects(:hosts_bulk_actions, :install_content) do |p|
-        p['organization_id'] == 1 &&
-          p['included'] == { search: "host_collection_id=\"3\"" } &&
-          p['content'] == ['birds']
-      end
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --id 3
+                 --package-groups birds --organization-id 1))
+    end
 
+    it 'installs package groups to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --name Test
+                 --package-groups birds --organization-id 1))
+    end
+
+    it 'installs package groups to hosts in a host collection' do
+      api_expects_content_install('package_group', :content => ['birds'])
       run_cmd(%w(host-collection package-group install --id 3
                  --package-groups birds --organization-id 1))
     end
 
-    it 'installs erratum to hosts in a host collection' do
-      api_expects(:hosts_bulk_actions, :install_content) do |p|
-        p['organization_id'] == 1 &&
-          p['included'] == { search: "host_collection_id=\"3\"" } &&
-          p['content'] == ['Bird_Erratum']
-      end
+    it 'installs package groups to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('package_group', :content => ['birds'])
+      run_cmd(%w(host-collection package-group install --name Test
+                 --package-groups birds --organization-id 1))
+    end
 
+    it 'installs erratum to hosts in a host collection' do
+      api_expects_content_install('errata', :content => ['Bird_Erratum'])
       run_cmd(%w(host-collection erratum install --id 3 --errata Bird_Erratum --organization-id 1))
     end
+
+    it 'installs erratum to hosts in a host collection specified by name' do
+      api_expects_collection_search
+      api_expects_content_install('errata', :content => ['Bird_Erratum'])
+      run_cmd(%w(host-collection erratum install --name Test --errata Bird_Erratum
+                 --organization-id 1))
+    end
   end
 end