hammer_cli_katello 1.1.3 → 1.2.0

data/test/functional/host_collection/create_test.rb

@@ -0,0 +1,54 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::CreateCommand do
+    it 'requires organization options' do
+      result = run_cmd(%w(host-collection create --name hc1))
+      expected_error = "Could not find organization"
+      assert_equal(result.exit_code, HammerCLI::EX_SOFTWARE)
+      assert_equal(result.err[/#{expected_error}/], expected_error)
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :create) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :create) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization org1))
+    end
+
+    it 'allows unlimited-hosts flag with no arguments' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :create) do |par|
+        par['unlimited_hosts'] == true
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization org1 --unlimited-hosts))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :create) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection create --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/delete_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::DeleteCommand do
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :destroy)
+      run_cmd(%w(host-collection delete --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection delete --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :destroy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection delete --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :destroy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection delete --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :destroy) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection delete --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/hosts_test.rb

@@ -0,0 +1,39 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::HostsCommand do
+    it 'allows host collection id' do
+      api_expects(:hosts, :index) do |par|
+        par['search'] == "host_collection_id=1"
+      end
+
+      run_cmd(%w(host-collection hosts --id 1))
+    end
+
+    it 'allows host collection name' do
+      ex = api_expects(:host_collections, :index)
+      ex.returns(index_response([{'id' => 2}]))
+
+      api_expects(:hosts, :index) do |par|
+        par['search'] == "host_collection_id=2"
+      end
+
+      run_cmd(%w(host-collection hosts --name collection --organization-id 1))
+    end
+
+    it 'requires organization with host collection name' do
+      result = run_cmd(%w(host-collection hosts --name collection))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'requires host collection name or id' do
+      result = run_cmd(%w(host-collection hosts))
+      expected_error = "Error: At least one of options --id, --name is required"
+      assert_equal(HammerCLI::EX_USAGE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+  end
+end

data/test/functional/host_collection/info_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::InfoCommand do
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :show)
+      run_cmd(%w(host-collection info --id 1))
+    end
+
+    it 'requires organization options if id is not specified' do
+      result = run_cmd(%w(host-collection info --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :show) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection info --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :show) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection info --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :show) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection info --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/list_test.rb

@@ -0,0 +1,63 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::ListCommand do
+    it 'does not require organization options' do
+      api_expects(:host_collections, :index)
+
+      run_cmd(%w(host-collection list))
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection list --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection list --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) do |par|
+        par['organization_id'].to_i == 1
+      end
+
+      run_cmd(%w(host-collection list --organization-label org1))
+    end
+
+    it 'allows host id' do
+      api_expects(:host_collections, :index) do |par|
+        par['host_id'] == 1
+      end
+
+      run_cmd(%w(host-collection list --host-id 1 --organization-id 1))
+    end
+
+    it 'allows host name' do
+      ex = api_expects(:hosts, :index) do |par|
+        par[:search] == "name = \"host1\""
+      end
+      ex.returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) do |par|
+        par['host_id'] == 1
+      end
+
+      run_cmd(%w(host-collection list --host host1 --organization-id 1))
+    end
+  end
+end

data/test/functional/host_collection/remove_host_test.rb

@@ -0,0 +1,90 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::RemoveHostCommand do
+    describe 'handles individual host errors' do
+      it 'for successful results' do
+        api_expects(:host_collections, :remove_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => ["Successfully removed 2 Host(s)."],
+                                         "error" => []})
+        result = run_cmd(%w(host-collection remove-host --id 1 --host-ids 2,3))
+        assert_match("The host(s) has been removed", result.out)
+      end
+
+      it 'for mixed results' do
+        api_expects(:host_collections, :remove_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => ["Successfully removed 1 Host(s)."],
+                                         "error" => ["Host with ID 3 not found."]})
+        result = run_cmd(%w(host-collection remove-host --id 1 --host-ids 2,3))
+        assert_match("Could not remove host(s)", result.out)
+        assert_match("Host with ID 3 not found.", result.out)
+      end
+
+      it 'for errored results' do
+        api_expects(:host_collections, :remove_hosts)
+          .with_params('id' => 1, 'host_ids' => %w(2 3))
+          .returns("displayMessages" => {"success" => [], "error" => [
+            "Host with ID 1 does not exist in the host collection.",
+            "Host with ID 3 not found."]})
+        result = run_cmd(%w(host-collection remove-host --id 1 --host-ids 2,3))
+        assert_match("Could not remove host(s)", result.out)
+        assert_match("Host with ID 1 does not exist in the host collection.", result.out)
+        assert_match("Host with ID 3 not found.", result.out)
+      end
+    end
+
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :remove_hosts)
+      run_cmd(%w(host-collection remove-host --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection remove-host --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :remove_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection remove-host --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :remove_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection remove-host --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :remove_hosts) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection remove-host --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/host_collection/update_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/host_collection'
+
+module HammerCLIKatello
+  describe HostCollection::UpdateCommand do
+    it 'does not require organization options if id is specified' do
+      api_expects(:host_collections, :update)
+      run_cmd(%w(host-collection update --id 1))
+    end
+
+    it 'requires organization options if name is specified' do
+      result = run_cmd(%w(host-collection update --name hc1))
+      expected_error = "Missing options to search organization"
+      assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+      assert_equal(expected_error, result.err[/#{expected_error}/])
+    end
+
+    it 'allows organization id' do
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :update) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection update --name hc1 --organization-id 1))
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :update) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection update --name hc1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:host_collections, :index) { |par| par['organization_id'].to_i == 1 }
+        .returns(index_response([{'id' => 2}]))
+
+      api_expects(:host_collections, :update) do |par|
+        par['id'].to_i == 2
+      end
+
+      run_cmd(%w(host-collection update --name hc1 --organization-label org1))
+    end
+  end
+end

data/test/functional/hostgroup/create_test.rb

@@ -0,0 +1,112 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/hostgroup'
+
+module HammerCLIForeman
+  describe Hostgroup do
+    # These tests are only for the extensions Katello adds to the hostgroup command
+    # See hammer-cli-foreman for the core hostgroup tests
+    describe CreateCommand do
+      it 'allows content source id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['content_source_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-source-id 1))
+      end
+
+      it 'allows content source name' do
+        content_source_name = "life_of_cs"
+        content_source_id = 111
+        hg_name = "mercury"
+
+        api_expects(:smart_proxies, :index).
+          with_params(:search => "name = \"#{content_source_name}\"").
+          returns(index_response([{'id' => content_source_id}]))
+
+        api_expects(:hostgroups, :create).
+          with_params('hostgroup' => {'name' => hg_name,
+                                      'content_source_id' => content_source_id
+                                     })
+
+        cmd = "hostgroup create --name #{hg_name} --content-source #{content_source_name}"
+        run_cmd(cmd.split)
+      end
+
+      it 'allows content view id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-view-id 1))
+      end
+
+      it 'allows content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p[:search] = "name = \"cv1\"" && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['content_view_id'] == 1 && p['hostgroup']['name'] == 'hg1'
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-view cv1 --query-organization-id 1))
+      end
+
+      it 'requires organization options to resolve content view name' do
+        api_expects_no_call
+        result = run_cmd(%w(hostgroup create --name hg1 --content-view cv1))
+        assert_match(/--query-organization/, result.err)
+      end
+
+      it 'allows lifecycle environment id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['lifecycle_environment_id'] == 1 &&
+            p['hostgroup']['organization_ids'] == %w(1 2)
+        end
+        run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment-id 1 --organization-ids 1,2))
+      end
+
+      it 'allows lifecycle environment name' do
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p['name'] == 'le1' && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['lifecycle_environment_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment le1
+                   --query-organization-id 1 --organization-ids 1,2))
+      end
+
+      it 'requires organization options to resolve lifecycle environment name' do
+        api_expects_no_call
+        result = run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment le1))
+        assert_match(/--query-organization/, result.err)
+      end
+
+      it 'allows kickstart repository name' do
+        env_id = 100
+        cv_id = 200
+        repo_name = "life_of_kickstart"
+        repo_id = 111
+        hg_name = "mercury"
+
+        api_expects(:repositories, :index).
+          with_params('name' => repo_name,
+                      'environment_id' => env_id,
+                      'content_view_id' => cv_id).
+          returns(index_response([{'id' => repo_id}]))
+
+        expected_params = {'hostgroup' => {
+          'name' => hg_name,
+          'lifecycle_environment_id' => env_id,
+          'content_view_id' => cv_id,
+          'kickstart_repository_id' => repo_id
+        }
+                          }
+        api_expects(:hostgroups, :create).with_params(expected_params)
+
+        cmd = "hostgroup create --name #{hg_name} --lifecycle-environment-id #{env_id}"\
+              " --content-view-id #{cv_id} --kickstart-repository #{repo_name}"
+        run_cmd(cmd.split)
+      end
+    end
+  end
+end

data/test/functional/hostgroup/data/hostgroup.json

@@ -0,0 +1,50 @@
+{
+  "content_source_id":1,
+  "content_source_name":"foreman.example.com",
+  "content_view_id":1,
+  "content_view_name":"Default Organization View",
+  "lifecycle_environment_id":1,
+  "lifecycle_environment_name":"Library",
+  "kickstart_repository_id":10000,
+  "kickstart_repository_name":"Rhel 7",
+  "subnet_id":null,
+  "subnet_name":null,
+  "operatingsystem_id":null,
+  "operatingsystem_name":null,
+  "domain_id":null,
+  "domain_name":null,
+  "environment_id":1,
+  "environment_name":"Production Env",
+  "compute_profile_id":null,
+  "compute_profile_name":null,
+  "ancestry":null,
+  "parent_id":null,
+  "parent_name":null,
+  "puppet_proxy_id":null,
+  "puppet_ca_proxy_id":null,
+  "ptable_id":null,
+  "ptable_name":null,
+  "medium_id":null,
+  "medium_name":null,
+  "architecture_id":null,
+  "architecture_name":null,
+  "realm_id":null,
+  "realm_name":null,
+  "created_at":"2016-08-09 04:18:32 UTC",
+  "updated_at":"2016-08-09 04:18:32 UTC",
+  "id":1,
+  "parameters":[],
+  "template_combinations":[],
+  "puppetclasses":[],
+  "config_groups":[],
+  "all_puppetclasses":[],
+  "locations":[],
+  "organizations":[
+  {
+    "id":1,
+    "name":"Redhat",
+    "title":"Redhat",
+    "description":null
+  }
+  ]
+}

data/test/functional/hostgroup/info_test.rb

@@ -0,0 +1,33 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/hostgroup'
+
+module HammerCLIForeman
+  describe Hostgroup do
+    describe InfoCommand do
+      before do
+        @cmd = %w(hostgroup info)
+      end
+
+      let(:hostgroup_id) { 1 }
+
+      it "Shows information about a hostgroup" do
+        params = ["--id=#{hostgroup_id}"]
+
+        ex = api_expects(:hostgroups, :show, "Get info") do |par|
+          par["id"] == hostgroup_id.to_s
+        end
+        json_file = File.join(File.dirname(__FILE__), 'data', 'hostgroup.json')
+        ex.returns(JSON.parse(File.read(json_file)))
+
+        result = run_cmd(@cmd + params)
+        # rubocop:disable Style/WordArray
+        expected_fields = [['Name', 'Library'],
+                           ['Name', 'Default Organization View'],
+                           ['Name', 'foreman.example.com'],
+                           ['Name', 'Rhel 7']]
+        expected_results = expected_fields.map { |field| success_result(FieldMatcher.new(*field)) }
+        expected_results.each { |expected|  assert_cmd(expected, result) }
+      end
+    end
+  end
+end