hammer_cli_katello 1.4.1 → 1.5.2

data/test/functional/acs/create_test.rb

@@ -0,0 +1,27 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'create content-credentials' do
+  before do
+    @cmd = %w(alternate-content-sources create)
+  end
+
+  let(:name) { 'pizza' }
+  let(:url) { 'http://proxy.example.com' }
+  let(:acs_type) { 'custom' }
+  let(:acs_content) { 'yum' }
+  let(:verify_ssl) { false }
+  let(:proxy_id) { 1 }
+  let(:subpaths) { ['test/repo1/'] }
+
+  it 'Creates an ACS' do
+    params = ["--name=#{name}", "--alternate-content-source-type=#{acs_type}",
+              "--base-url=#{url}", "--content-type=#{acs_content}", "--verify-ssl=#{verify_ssl}",
+              "--smart-proxy-ids=#{proxy_id}", "--subpaths=#{subpaths}"]
+
+    api_expects(:alternate_content_sources, :create, 'Create an ACS')
+
+    expected_result = success_result("Alternate Content Source created.\n")
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/acs/delete_test.rb

@@ -0,0 +1,13 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'delete an acs' do
+  let(:id) { 1 }
+
+  it 'by id' do
+    api_expects(:alternate_content_sources, :destroy, 'delete acs').
+      with_params('id' => id)
+
+    command = %W(alternate-content-sources delete --id #{id})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+end

data/test/functional/acs/info_test.rb

@@ -0,0 +1,55 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require 'hammer_cli_katello/associating_commands'
+
+describe 'get acs info' do
+  before do
+    @cmd = %w(alternate-content-sources info)
+  end
+
+  it 'shows acs info by id' do
+    params = ['--id=1']
+    ex = api_expects(:alternate_content_sources, :show, 'Get info')
+    ex.returns(
+      'id' => 1,
+      'name' => 'Pizza ACS',
+      'label' => 'Pizza ACS',
+      'base_url' => 'https://proxy.example.com',
+      'alternate_content_source_type' => 'custom',
+      'content_type' => 'yum',
+      'smart_proxies' => {
+        'id' => 1,
+        'name' => 'centos7.example.com',
+        'url' => 'https://centos7.example.com:9090',
+        'created_at' => '2022-05-09T17:40:21.007Z',
+        'updated_at' => '2022-05-09T17:40:21.007Z',
+        'expired_logs' => 0,
+        'download_policy' => 'on_demand'
+      },
+      'subpaths' => [
+        'test/repo1'
+      ]
+    )
+    result = run_cmd(@cmd + params)
+    # rubocop:disable Style/WordArray
+    expected_fields = [['ID', '1'],
+                       ['Name', 'Pizza ACS'],
+                       ['Label', 'Pizza ACS'],
+                       ['Base URL', 'https://proxy.example.com'],
+                       ['Alternate content source type', 'custom'],
+                       ['Content type', 'yum'],
+                       ['Smart proxies', ''],
+                       ['Id', '1'],
+                       ['Name', 'centos7.example.com'],
+                       ['URL', 'https://centos7.example.com:9090'],
+                       ['Created at', '2022-05-09T17:40:21.007Z'],
+                       ['Updated at', '2022-05-09T17:40:21.007Z'],
+                       ['Expired logs', '0'],
+                       ['Download policy', 'on_demand'],
+                       ['Subpaths', ''],
+                       ['', 'test/repo1']]
+
+    # rubocop:enable Style/WordArray
+    expected_results = expected_fields.map { |field| success_result(FieldMatcher.new(*field)) }
+    expected_results.each { |expected|  assert_cmd(expected, result) }
+  end
+end

data/test/functional/acs/list_test.rb

@@ -0,0 +1,62 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'listing acs' do
+  before do
+    @cmd = %w(alternate-content-sources list)
+  end
+
+  let(:empty_response) do
+    {
+      "total" => 0,
+      "subtotal" => 0,
+      "page" => "1",
+      "per_page" => "1000",
+      "error" => nil,
+      "search" => nil,
+      "sort" => {
+        "by" => nil,
+        "order" => nil
+      },
+      "results" => []
+    }
+  end
+
+  let(:acs_response) do
+    {
+      'id' => 1,
+      'name' => 'pizza',
+      'alternate_content_source_type' => 'custom'
+    }
+  end
+
+  it "lists acs and returns empty response" do
+    ex = api_expects(:alternate_content_sources, :index, 'acs list') do |par|
+      par['page'] == 1 && par['per_page'] == 1000
+    end
+
+    ex.returns(empty_response)
+
+    expected_result = success_result("---|------|-----
+ID | NAME | TYPE
+---|------|-----
+")
+
+    result = run_cmd(@cmd)
+    assert_cmd(expected_result, result)
+  end
+
+  it "lists acs and returns response" do
+    ex = api_expects(:alternate_content_sources, :index, 'acs list') do |par|
+      par['page'] == 1 && par['per_page'] == 1000
+    end
+
+    ex.returns(acs_response)
+
+    expected_result = success_result("1  | pizza | custom
+---|-------|-------
+")
+
+    result = run_cmd(@cmd)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/acs/update_test.rb

@@ -0,0 +1,23 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'listing acs' do
+  before do
+    @cmd = %w(alternate-content-sources update)
+  end
+
+  let(:id) { 1 }
+  let(:desc) { 'pizza' }
+
+  it 'update acs' do
+    params = ["--id=#{id}", "--description=#{desc}"]
+
+    ex = api_expects(:alternate_content_sources, :update, 'acs update') do |par|
+      par['id'] == 1 && par['description'] == 'pizza'
+    end
+
+    ex.returns({})
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+end

data/test/functional/content_export/complete/repository_test.rb

@@ -0,0 +1,226 @@
+require File.join(File.dirname(__FILE__), '../../../test_helper')
+require 'hammer_cli_katello/content_export_complete'
+require_relative '../../product/product_helpers'
+require_relative '../../repository/repository_helpers'
+require_relative '../content_export_helpers'
+
+describe 'content-export complete repository' do
+  include ForemanTaskHelpers
+  include ContentExportHelpers
+  include ProductHelpers
+  include RepositoryHelpers
+
+  before do
+    @cmd = %w(content-export complete repository)
+  end
+
+  let(:task_id) { '5' }
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'planned',
+      'output' => {
+        'export_history_id' => 2
+      }
+    }
+  end
+
+  let(:export_history) do
+    {
+      "id": 1000,
+      "path": "/tmp",
+      "metadata": {}
+    }
+  end
+
+  let(:default_repository_options) do
+    {"download_policy" => "immediate", "id" => repository_id}
+  end
+
+  let(:product_id) { '77' }
+  let(:repository_id) { '1001' }
+
+  let(:name) { 'repo' }
+
+  it "performs export with required options and async" do
+    params = [
+      "--id=#{repository_id}",
+      '--async'
+    ]
+    expects_repository(repository_id, default_repository_options)
+    ex = api_expects(:content_exports, :repository)
+    ex.returns(response)
+
+    result = run_cmd(@cmd + params)
+
+    assert_equal("Repository is being exported in task #{task_id}.\n"\
+      + "Once the task completes the export metadata must be generated with the "\
+      + "command:\n hammer content-export generate-metadata --task-id #{task_id}\n", result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "performs export with required options" do
+    params = [
+      "--id=#{repository_id}"
+    ]
+    expects_repository(repository_id, default_repository_options)
+    ex = api_expects(:content_exports, :repository)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportComplete::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Generated .*metadata.*json/, result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails on missing required params' do
+    params = [
+      '--boo-id=2'
+    ]
+
+    result = run_cmd(@cmd + params)
+    expected_error = "Could not export the repository:\n"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_equal(result.err[/#{expected_error}/], expected_error)
+  end
+
+  it 'fails on missing product name/id' do
+    params = ["--name=foo"]
+
+    result = run_cmd(@cmd + params)
+    expected_error = "At least one of options --product-id, --product is required"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'fails on missing repository' do
+    params = ["--product-id=2"]
+    result = run_cmd(@cmd + params)
+    expected_error = "--name is required."
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'fails on missing product missing org' do
+    params = ["--product=lol", "--name=#{name}"]
+    result = run_cmd(@cmd + params)
+    expected_error = "At least one of options --organization-id,"\
+                        " --organization, --organization-label is required"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'correctly resolves product_id and repository name' do
+    params = ["--product-id=#{product_id}",
+              "--name=#{name}",
+              "--async"]
+    expects_repository(repository_id, default_repository_options)
+
+    cvv_expect = api_expects(:repositories, :index) do |p|
+      assert_equal p['product_id'].to_s, product_id.to_s
+      assert_equal p["name"], name
+    end
+
+    cvv_expect.at_least_once.
+      returns(index_response([{'id' => repository_id}]))
+
+    ex = api_expects(:content_exports, :repository) do |p|
+      assert_equal p['id'], repository_id
+    end
+    ex.returns(response)
+    run_cmd(@cmd + params)
+  end
+
+  it 'correctly resolves chunk-size-gb' do
+    chunk_size = 1000
+    params = ["--id=#{repository_id}",
+              "--chunk-size-gb=#{chunk_size}",
+              "--organization-id=99"]
+
+    expects_repository(repository_id, default_repository_options)
+
+    ex = api_expects(:content_exports, :repository) do |p|
+      assert_equal p["chunk_size_gb"], chunk_size
+    end
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportComplete::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Generated .*metadata.*json/, result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails on invalid chunk-size-gb value' do
+    params = ["--id=#{repository_id}",
+              "--chunk-size-gb=0.5",
+              "--organization-id=99"]
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Error: Option '--chunk-size-gb': Numeric value is required/, result.err)
+    assert_equal(HammerCLI::EX_USAGE, result.exit_code)
+  end
+
+  it 'Errors out on lazy repositories' do
+    params = ["--id=#{repository_id}"]
+    expects_repository(repository_id, "download_policy" => "on_demand", "id" => repository_id)
+
+    ex = api_expects(:content_exports, :repository)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportComplete::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    HammerCLIKatello::ContentExportComplete::RepositoryCommand.
+      any_instance.
+      expects(:exit).with(HammerCLI::EX_SOFTWARE)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Unable to fully export this repository because/, result.out)
+    assert_match(/#{repository_id}/, result.out)
+  end
+
+  it 'should accept product and get the right repository' do
+    params = ["--product-id=#{product_id}",
+              "--name=#{name}"]
+    expect_repository_search(product_id.to_i, name, repository_id).at_least_once
+    repo_search = api_expects(:repositories, :index, 'Find a repository') do |p|
+      assert_equal p["product_id"].to_s, product_id.to_s
+      assert_equal p["name"], name
+    end
+    repo_search.returns(index_response([{'id' => repository_id}]))
+
+    expects_repository(repository_id, default_repository_options)
+    ex = api_expects(:content_exports, :repository)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportComplete::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+    result = run_cmd(@cmd + params)
+    assert_match(/Generated .*metadata.*json/, result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+end

data/test/functional/content_export/content_export_helpers.rb

@@ -2,6 +2,13 @@ require_relative '../search_helpers'
 
 module ContentExportHelpers
   include SearchHelpers
+  def expects_repository(repository_id, returns = {})
+    ex = api_expects(:repositories, :show, "Find Repo #{repository_id}") do |p|
+      assert_equal p[:id].to_s, repository_id.to_s
+    end
+    ex.returns(returns)
+  end
+
   def expects_repositories_in_library(organization_id, returns = [])
     expect_lenient_search(:repositories,
                           params: {

data/test/functional/content_export/incremental/repository_test.rb

@@ -0,0 +1,212 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_export'
+require_relative '../../product/product_helpers'
+require_relative '../../repository/repository_helpers'
+require_relative '../content_export_helpers'
+
+describe 'content-export incremental version' do
+  include ForemanTaskHelpers
+  include ContentExportHelpers
+  include ProductHelpers
+  include RepositoryHelpers
+
+  before do
+    @cmd = %w(content-export incremental repository)
+  end
+
+  let(:task_id) { '5' }
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'planned',
+      'output' => {
+        'export_history_id' => 2
+      }
+    }
+  end
+
+  let(:export_history_id) { 1000 }
+
+  let(:export_history) do
+    {
+      "id": export_history_id,
+      "path": "/tmp",
+      "metadata": {}
+    }
+  end
+
+  let(:default_repository_options) do
+    {"download_policy" => "immediate", "id" => repository_id}
+  end
+
+  let(:product_id) { '77' }
+  let(:repository_id) { 100 }
+
+  let(:name) { 'great' }
+
+  it "performs export with required options and async" do
+    params = [
+      "--id=#{repository_id}",
+      '--async'
+    ]
+    expects_repository(repository_id, default_repository_options)
+
+    ex = api_expects(:content_export_incrementals, :repository)
+    ex.returns(response)
+
+    result = run_cmd(@cmd + params)
+
+    assert_equal("Repository is being exported in task #{task_id}.\n"\
+      + "Once the task completes the export metadata must be generated with the "\
+      + "command:\n hammer content-export generate-metadata --task-id #{task_id}\n", result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "performs export with required options" do
+    params = [
+      "--id=#{repository_id}"
+    ]
+
+    expects_repository(repository_id, default_repository_options)
+    ex = api_expects(:content_export_incrementals, :repository)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Generated .*metadata.*json/, result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "performs export with history id" do
+    params = [
+      "--id=#{repository_id}",
+      "--from-history-id=#{export_history_id}"
+    ]
+    expects_repository(repository_id, default_repository_options)
+    api_expects(:content_export_incrementals, :repository)
+      .with_params('id' => repository_id,
+                   'from_history_id' => export_history_id)
+      .returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Generated .*metadata.*json/, result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails on missing required params' do
+    params = [
+      '--boo-id=2'
+    ]
+
+    result = run_cmd(@cmd + params)
+    expected_error = "Could not export the repository:\n"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_equal(result.err[/#{expected_error}/], expected_error)
+  end
+
+  it 'correctly resolves product_id and repository name' do
+    params = ["--product-id=#{product_id}",
+              "--name=#{name}",
+              "--async"]
+    expects_repository(repository_id, default_repository_options)
+
+    cvv_expect = api_expects(:repositories, :index) do |p|
+      assert_equal p['product_id'].to_s, product_id.to_s
+      assert_equal p["name"], name
+    end
+
+    cvv_expect.at_least_once.
+      returns(index_response([{'id' => repository_id}]))
+
+    ex = api_expects(:content_export_incrementals, :repository) do |p|
+      assert_equal p['id'], repository_id
+    end
+    ex.returns(response)
+    run_cmd(@cmd + params)
+  end
+
+  it 'fails on missing product name/id' do
+    params = ["--name=foo"]
+    result = run_cmd(@cmd + params)
+    expected_error = "At least one of options --product-id, --product is required"
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'fails on missing repository' do
+    params = ["--product-id=2"]
+    result = run_cmd(@cmd + params)
+    expected_error = "--name is required."
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'fails on missing product missing org' do
+    params = ["--product=lol", "--name=#{name}"]
+    result = run_cmd(@cmd + params)
+    expected_error = "At least one of options --organization-id,"\
+                      " --organization, --organization-label is required"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'correctly resolves product_id and repository name' do
+    params = ["--product-id=#{product_id}",
+              "--name=#{name}",
+              "--async"]
+    expects_repository(repository_id, default_repository_options)
+
+    cvv_expect = api_expects(:repositories, :index) do |p|
+      assert_equal p['product_id'].to_s, product_id.to_s
+      assert_equal p["name"], name
+    end
+
+    cvv_expect.at_least_once.
+      returns(index_response([{'id' => repository_id}]))
+
+    ex = api_expects(:content_export_incrementals, :repository) do |p|
+      assert_equal p['id'], repository_id
+    end
+    ex.returns(response)
+    run_cmd(@cmd + params)
+  end
+
+  it 'Errors out on lazy repositories' do
+    params = ["--id=#{repository_id}"]
+    expects_repository(repository_id, "download_policy" => "on_demand", "id" => repository_id)
+
+    ex = api_expects(:content_export_incrementals, :repository)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::RepositoryCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    HammerCLIKatello::ContentExportIncremental::RepositoryCommand.
+      any_instance.
+      expects(:exit).with(HammerCLI::EX_SOFTWARE)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Unable to fully export this repository because/, result.out)
+    assert_match(/#{repository_id}/, result.out)
+  end
+end

data/test/functional/module_stream/info_test.rb

@@ -12,10 +12,10 @@ module HammerCLIKatello
     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}]))
+        .returns(index_response([{'id' => "1"}]))
 
       api_expects(:module_streams, :show)
-        .with_params(repository_id: 1, id: 1)
+        .with_params(repository_id: 1, id: "1")
 
       run_cmd(%w(module-stream info --name duck --repository-id 1))
     end

data/test/functional/organization/cdn_configuration_test.rb

@@ -16,7 +16,7 @@ describe HammerCLIKatello::Organization::ConfigureCdnCommand do
     username = "foo"
     url = "https://goo.com"
     upstream_label = "GreatOrg"
-    type = 'upstream_server'
+    type = 'network_sync'
     params = ["--label=#{org_label}",
               "--url=#{url}",
               "--username=#{username}",

data/test/functional/organization/info_test.rb

@@ -7,6 +7,10 @@ describe HammerCLIKatello::Organization::InfoCommand 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}"]
@@ -14,7 +18,8 @@ describe HammerCLIKatello::Organization::InfoCommand do
 
     api_expects(:organizations, :show)
       .with_params('id' => org_id.to_s)
-      .returns("simple_content_access" => true, "id" => org_id)
+      .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)
@@ -26,7 +31,8 @@ describe HammerCLIKatello::Organization::InfoCommand do
     api_expects(:organizations, :index).returns(index_response([{'id' => org_id}]))
 
     cdn_configuration = {
-      "url" => "https://cdn.redhat.com"
+      "url" => "https://cdn.redhat.com",
+      "type" => "redhat_cdn"
     }
 
     api_expects(:organizations, :show)