hammer_cli_katello 1.5.0 → 1.5.1

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

data/test/functional/hostgroup/update_test.rb

@@ -0,0 +1,120 @@
+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 UpdateCommand do
+      it 'allows content source id' do
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['content_source_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --content-source-id 1))
+      end
+
+      it 'allows content source name' do
+        content_source_name = "life_of_cs"
+        content_source_id = 111
+        hg_id = 441
+
+        api_expects(:smart_proxies, :index).
+          with_params(:search => "name = \"#{content_source_name}\"").
+          returns(index_response([{'id' => content_source_id}]))
+
+        api_expects(:hostgroups, :update).
+          with_params('id' => hg_id.to_s,
+                      'hostgroup' => {'content_source_id' => content_source_id
+                                     })
+
+        cmd = "hostgroup update --id #{hg_id} --content-source #{content_source_name}"
+        run_cmd(cmd.split)
+      end
+
+      it 'allows content view id' do
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --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, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --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 update --id 1 --content-view cv1))
+        assert_match(/--query-organization/, result.err)
+      end
+
+      it 'allows lifecycle environment id' do
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['lifecycle_environment_id'] == 1 &&
+            p['hostgroup']['organization_ids'] == %w(1 2)
+        end
+        run_cmd(%w(hostgroup update --id 1 --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, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['lifecycle_environment_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --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 update --name hg1 --lifecycle-environment le1))
+        assert_match(/--query-organization/, result.err)
+      end
+
+      it 'requires organization options to resolve lifecycle environment name' do
+        api_expects_no_call
+        result = run_cmd(%w(hostgroup update --id 1 --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"
+        hg_id = 108
+
+        api_expects(:hostgroups, :index).
+          with_params(:search => "name = \"#{hg_name}\"").
+          returns(index_response([{'id' => hg_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(:hostgroups, :update).
+          with_params('id' => hg_id,
+                      'hostgroup' => {'lifecycle_environment_id' => env_id,
+                                      'content_view_id' => cv_id,
+                                      'kickstart_repository_id' => repo_id
+                                     })
+
+        cmd = "hostgroup update --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/lifecycle_environment/create_test.rb

@@ -0,0 +1,14 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/lifecycle_environment'
+
+module HammerCLIKatello
+  describe LifecycleEnvironmentCommand::CreateCommand do
+    it 'allows minimal options' do
+      api_expects(:lifecycle_environments, :create) do |p|
+        p['name'] == 'le1' && p['prior_id'] == 3 && p['organization_id'] == 1
+      end
+
+      run_cmd(%w(lifecycle-environment create --name le1 --prior-id 3 --organization-id 1))
+    end
+  end
+end

data/test/functional/lifecycle_environment/lifecycle_environment_helpers.rb

@@ -0,0 +1,18 @@
+require_relative '../search_helpers'
+
+module LifecycleEnvironmentHelpers
+  include SearchHelpers
+
+  def expect_lifecycle_environment_search(org_id, name, id)
+    expect_lenient_search(:lifecycle_environments,
+                          params: {'name' => name, 'organization_id' => org_id},
+                          returns: {'id' => id})
+  end
+
+  def expect_lifecycle_environments_request(org_id, results)
+    ex = api_expects(:lifecycle_environments, :index, 'List the lifecycle environments') do |par|
+      par['organization_id'] == org_id
+    end
+    ex.returns(index_response(results))
+  end
+end

data/test/functional/lifecycle_environment/list_test.rb

@@ -0,0 +1,38 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/lifecycle_environment'
+
+module HammerCLIKatello
+  describe LifecycleEnvironmentCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:lifecycle_environments, :index)
+
+      run_cmd(%w(lifecycle-environment list))
+    end
+
+    describe 'allows organization' do
+      it 'id' do
+        api_expects(:lifecycle_environments, :index) { |p| p['organization_id'] == 1 }
+
+        run_cmd(%w(lifecycle-environment 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(:lifecycle_environments, :index) { |p| p['organization_id'] == 1 }
+
+        run_cmd(%w(lifecycle-environment 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(:lifecycle_environments, :index) { |p| p['organization_id'] == 1 }
+
+        run_cmd(%w(lifecycle-environment list --organization-label org1))
+      end
+    end
+  end
+end

data/test/functional/lifecycle_environment/update_test.rb

@@ -0,0 +1,14 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/lifecycle_environment'
+
+module HammerCLIKatello
+  describe LifecycleEnvironmentCommand::UpdateCommand do
+    it 'allows minimal options' do
+      api_expects(:lifecycle_environments, :update) do |p|
+        p['new_name'] == 'le3' && p['id'] == 3
+      end
+
+      run_cmd(%w(lifecycle-environment update --id 3 --new-name le3))
+    end
+  end
+end

data/test/functional/local_helper_test.rb

@@ -0,0 +1,30 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'local helper' do
+  before do
+    class SuperLocalTestHelper
+      def parse_subcommand
+        true
+      end
+    end
+
+    class LocalTestHelper < SuperLocalTestHelper
+      include HammerCLIKatello::LocalHelper
+    end
+
+    @local_helper = LocalTestHelper.new
+  end
+
+  it "does not fail if /usr/share/foreman exists" do
+    File.expects(:exist?).with('/usr/share/foreman').returns(true)
+    assert @local_helper.parse_subcommand
+  end
+
+  it "fails if /usr/share/foreman does not exist" do
+    File.expects(:exist?).with('/usr/share/foreman').returns(false)
+
+    assert_raises RuntimeError do
+      @local_helper.parse_subcommand
+    end
+  end
+end

data/test/functional/module_stream/info_test.rb

@@ -0,0 +1,58 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/module_stream'
+
+module HammerCLIKatello
+  describe ModuleStreamCommand::InfoCommand do
+    it 'allows ID' do
+      api_expects(:module_streams, :show).with_params('id' => '1')
+
+      run_cmd(%w(module-stream info --id 1))
+    end
+
+    it 'resolves ID from name and repo id' do
+      api_expects(:module_streams, :index)
+        .with_params(search: "name = \"duck\"", name: "duck", repository_id: "1")
+        .returns(index_response([{'id' => "1"}]))
+
+      api_expects(:module_streams, :show)
+        .with_params(repository_id: 1, id: "1")
+
+      run_cmd(%w(module-stream info --name duck --repository-id 1))
+    end
+
+    it 'resolves ID from name, repo name, and product id' do
+      api_expects(:module_streams, :index)
+        .with_params(search: "name = \"walrus\"")
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:repositories, :index)
+        .with_params(name: 'zoo', product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :show)
+        .with_params(repository_id: 1, id: 1)
+
+      run_cmd(%w(module-stream info --name walrus --repository zoo --product-id 1))
+    end
+
+    it 'resolves ID from name, repo name, product name, and org id' do
+      api_expects(:products, :index)
+        .with_params(name: "habitat", organization_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :index)
+        .with_params(search: "name = \"walrus\"")
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:repositories, :index)
+        .with_params(name: 'zoo', product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :show)
+        .with_params(repository_id: 1, id: 1)
+
+      run_cmd(%w(module-stream info --name walrus --repository zoo
+                 --product habitat --organization-id 1))
+    end
+  end
+end

data/test/functional/module_stream/list_test.rb

@@ -0,0 +1,53 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/module_stream'
+
+module HammerCLIKatello
+  describe ModuleStreamCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:module_streams, :index)
+
+      run_cmd(%w(module-stream list))
+    end
+
+    it 'can be provided by repository ID' do
+      api_expects(:module_streams, :index).with_params(repository_id: 1)
+
+      run_cmd(%w(module-stream list --repository-id 1))
+    end
+
+    it 'product params needed with repository name' do
+      cmd = run_cmd(%w(module-stream list --repository Repo))
+      error_msg = "At least one of options --product, --product-id is required."
+      api_expects_no_call
+      assert_match error_msg, cmd.err
+    end
+
+    it 'can be provided by repository name and product id' do
+      api_expects(:repositories, :index)
+        .with_params(name: "Repo", product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :index).with_params(repository_id: 1)
+
+      run_cmd(%w(module-stream list --repository Repo --product-id 1))
+    end
+
+    it 'can be provided by repository id and organization id' do
+      api_expects(:module_streams, :index)
+        .with_params(repository_id: 1, organization_id: 1)
+
+      run_cmd(%w(module-stream list --repository-id 1 --organization-id 1))
+    end
+
+    it 'can be provided by repository name, product id, and organization id' do
+      api_expects(:repositories, :index)
+        .with_params(name: "Repo", product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :index)
+        .with_params(repository_id: 1, organization_id: 1)
+
+      run_cmd(%w(module-stream list --repository Repo --product-id 1 --organization-id 1))
+    end
+  end
+end

data/test/functional/organization/cdn_configuration_test.rb

@@ -0,0 +1,45 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/organization'
+require 'hammer_cli_katello/associating_commands'
+
+describe HammerCLIKatello::Organization::ConfigureCdnCommand do
+  include OrganizationHelpers
+  before do
+    @cmd = %w(organization configure-cdn)
+  end
+
+  it 'by updates with the right params' do
+    org_id = '100'
+    org_label = 'org1'
+    ssl_cred_id = '1'
+    password = "pass"
+    username = "foo"
+    url = "https://goo.com"
+    upstream_label = "GreatOrg"
+    type = 'network_sync'
+    params = ["--label=#{org_label}",
+              "--url=#{url}",
+              "--username=#{username}",
+              "--upstream-organization-label=#{upstream_label}",
+              "--password=#{password}",
+              "--ssl-ca-credential-id=#{ssl_cred_id}",
+              "--type=#{type}"
+            ]
+    expect_organization_search(org_label, org_id, field: 'label')
+
+    api_expects(:organizations, :cdn_configuration) do |par|
+      par['id'].to_s == org_id &&
+        par['username'] == username &&
+        par['url'] == url &&
+        par['password'] == password &&
+        par['ssl_ca_credential_id'].to_s == ssl_cred_id &&
+        par['upstream_organization_label'] == upstream_label &&
+        par['type'] == type
+    end
+
+    assert_equal(
+      0,
+      run_cmd(@cmd + params).exit_code
+    )
+  end
+end

data/test/functional/organization/delete_test.rb

@@ -0,0 +1,26 @@
+require_relative '../test_helper'
+
+module HammerCLIKatello
+  describe 'Organization::DeleteCommand' do
+    it 'it requires the organization ID and must be resolved from name' do
+      api_expects(:organizations, :index).with_params(
+        :search => "name = \"my_org\"", :per_page => 1000, :page => 1
+      )
+      run_cmd(%w(organization delete --name my_org))
+    end
+
+    it 'it requires the organization ID and must be resolved from name' do
+      api_expects(:organizations, :index).with_params(
+        :search => "label = \"my_org\"", :per_page => 1000, :page => 1
+      )
+      run_cmd(%w(organization delete --label my_org))
+    end
+
+    it 'it requires the organization ID and must be resolved from name' do
+      api_expects(:organizations, :index).with_params(
+        :search => "title = \"my_org\"", :per_page => 1000, :page => 1
+      )
+      run_cmd(%w(organization delete --title my_org))
+    end
+  end
+end

data/test/functional/organization/info_test.rb

@@ -0,0 +1,45 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/organization'
+require 'hammer_cli_katello/associating_commands'
+
+describe HammerCLIKatello::Organization::InfoCommand do
+  before do
+    @cmd = %w(organization info)
+  end
+
+  let(:cdn_configuration) do
+    { "type" => 'network_sync' }
+  end
+
+  it "includes simple content access attributes" do
+    org_id = 2
+    params = ["--id=#{org_id}"]
+    api_expects(:organizations, :index).returns(index_response([{'id' => org_id}]))
+
+    api_expects(:organizations, :show)
+      .with_params('id' => org_id.to_s)
+      .returns("simple_content_access" => true, "id" => org_id,
+               "cdn_configuration" => cdn_configuration)
+    result = run_cmd(@cmd + params)
+    expected = success_result(FieldMatcher.new('Simple Content Access', 'Enabled'))
+    assert_cmd(expected, result)
+  end
+
+  it "includes cdn configuration attributes" do
+    org_id = 2
+    params = ["--id=#{org_id}"]
+    api_expects(:organizations, :index).returns(index_response([{'id' => org_id}]))
+
+    cdn_configuration = {
+      "url" => "https://cdn.redhat.com",
+      "type" => "redhat_cdn"
+    }
+
+    api_expects(:organizations, :show)
+      .with_params('id' => org_id.to_s)
+      .returns("cdn_configuration" => cdn_configuration, "id" => org_id)
+    result = run_cmd(@cmd + params)
+    expected = success_result(FieldMatcher.new('URL', cdn_configuration["url"]))
+    assert_cmd(expected, result)
+  end
+end

data/test/functional/organization/organization_helpers.rb

@@ -0,0 +1,10 @@
+require_relative '../search_helpers'
+
+module OrganizationHelpers
+  include SearchHelpers
+
+  def expect_organization_search(name_or_label, id, field: 'name')
+    expect_generic_search(:organizations, params: {search: "#{field} = \"#{name_or_label}\""},
+                                          returns: {'id' => id}).at_least_once
+  end
+end