hammer_cli_katello 0.24.0 → 1.1.0

data/test/functional/content_export/list_test.rb

@@ -0,0 +1,64 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_katello/content_export'
+
+describe 'content-export list' do
+  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 'allows minimal options' do
+    ex = api_expects(:content_exports, :index)
+
+    ex.returns(empty_response)
+    # rubocop:disable LineLength
+    expected_result = success_result('---|--------------------|------|------|----------------------|-------------------------|------------|-----------
+ID | DESTINATION SERVER | PATH | TYPE | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID | CREATED AT | UPDATED AT
+---|--------------------|------|------|----------------------|-------------------------|------------|-----------
+')
+    # rubocop:enable LineLength
+    result = run_cmd(%w(content-export list))
+    assert_cmd(expected_result, result)
+  end
+
+  it 'works with content-view-id only' do
+    api_expects(:content_view_versions, :index).returns(empty_response)
+
+    ex = api_expects(:content_exports, :index)
+
+    ex.returns(empty_response)
+    # rubocop:disable LineLength
+    expected_result = success_result('---|--------------------|------|------|----------------------|-------------------------|------------|-----------
+ID | DESTINATION SERVER | PATH | TYPE | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID | CREATED AT | UPDATED AT
+---|--------------------|------|------|----------------------|-------------------------|------------|-----------
+')
+    # rubocop:enable LineLength
+    result = run_cmd(%w(content-export list --content-view-id=1))
+    assert_cmd(expected_result, result)
+  end
+
+  it 'works with content-view-version-id only' do
+    ex = api_expects(:content_exports, :index)
+
+    ex.returns(empty_response)
+    # rubocop:disable LineLength
+    expected_result = success_result('---|--------------------|------|------|----------------------|-------------------------|------------|-----------
+ID | DESTINATION SERVER | PATH | TYPE | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID | CREATED AT | UPDATED AT
+---|--------------------|------|------|----------------------|-------------------------|------------|-----------
+')
+    # rubocop:enable LineLength
+    result = run_cmd(%w(content-export list --content-view-version-id=1))
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/content_import/library_test.rb

@@ -0,0 +1,85 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_katello/content_import'
+
+describe 'content-import library' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-import library)
+  end
+
+  let(:task_id) { '5' }
+
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'planned'
+    }
+  end
+
+  let(:path) do
+    File.dirname(__FILE__)
+  end
+
+  let(:metadata) do
+    JSON.parse(File.read("#{path}/metadata.json"))
+  end
+
+  let(:organization_id) { 3 }
+
+  it "performs import with required options and async" do
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{path}",
+      '--async'
+    ]
+    api_expects(:content_imports, :library)
+      .with_params('organization_id' => organization_id, 'path' => path, 'metadata' => metadata)
+      .returns(response)
+
+    result = run_cmd(@cmd + params)
+
+    assert_equal("Archive is being imported in task #{task_id}.\n", result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "performs import with required options" do
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{path}"
+    ]
+
+    api_expects(:content_imports, :library)
+      .with_params('organization_id' => organization_id, 'path' => path, 'metadata' => metadata)
+      .returns(response)
+
+    expect_foreman_task(task_id)
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails on missing required params' do
+    params = [
+      '--id=2'
+    ]
+
+    result = run_cmd(@cmd + params)
+    expected_error = "Could not import the archive."
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_equal(result.err[/#{expected_error}/], expected_error)
+  end
+
+  it 'fails on missing metadata.json' do
+    bad_path = "/nosuchdir"
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{bad_path}"
+    ]
+    result = run_cmd(@cmd + params)
+    expected_error = "Unable to find '#{bad_path}/metadata.json'."
+
+    assert_match(/#{expected_error}/, result.err)
+  end
+end

data/test/functional/content_import/list_test.rb

@@ -0,0 +1,65 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_katello/content_import'
+
+describe 'content-import list' do
+  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 'allows minimal options' do
+    ex = api_expects(:content_imports, :index)
+
+    ex.returns(empty_response)
+
+    # rubocop:disable LineLength
+    expected_result = success_result('---|------|------|----------------------|-------------------------|------------|-----------
+ID | PATH | TYPE | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID | CREATED AT | UPDATED AT
+---|------|------|----------------------|-------------------------|------------|-----------
+')
+    # rubocop:enable LineLength
+    result = run_cmd(%w(content-import list))
+    assert_cmd(expected_result, result)
+  end
+
+  it 'works with content-view-id only' do
+    api_expects(:content_view_versions, :index).returns(empty_response)
+
+    ex = api_expects(:content_imports, :index)
+
+    ex.returns(empty_response)
+    # rubocop:disable LineLength
+    expected_result = success_result('---|------|------|----------------------|-------------------------|------------|-----------
+ID | PATH | TYPE | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID | CREATED AT | UPDATED AT
+---|------|------|----------------------|-------------------------|------------|-----------
+')
+    # rubocop:enable LineLength
+    result = run_cmd(%w(content-import list --content-view-id=1))
+    assert_cmd(expected_result, result)
+  end
+
+  it 'works with content-view-version-id only' do
+    ex = api_expects(:content_imports, :index)
+
+    ex.returns(empty_response)
+    # rubocop:disable LineLength
+    expected_result = success_result('---|------|------|----------------------|-------------------------|------------|-----------
+ID | PATH | TYPE | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID | CREATED AT | UPDATED AT
+---|------|------|----------------------|-------------------------|------------|-----------
+')
+    # rubocop:enable LineLength
+    result = run_cmd(%w(content-import list --content-view-version-id=1))
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/content_import/metadata.json

@@ -0,0 +1 @@
+{"organization":"export-9697","repository_mapping":{"katello-15062":{"repository":"katello","product":"prod","redhat":false},"misc-28137":{"repository":"misc","product":"prod","redhat":false},"candlepin-37918":{"repository":"candlepin","product":"prod","redhat":false}},"content_view":"view","content_view_version":{"major":1,"minor":0},"incremental":false,"toc":"export-24d91ced-3d11-4fa2-b5ea-19a41f7b97a5-20201118_1523-toc.json"}

data/test/functional/content_import/version_test.rb

@@ -0,0 +1,85 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_katello/content_import'
+
+describe 'content-import version' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-import version)
+  end
+
+  let(:task_id) { '5' }
+
+  let(:response) do
+    {
+      'id' => task_id,
+      'state' => 'planned'
+    }
+  end
+
+  let(:path) do
+    File.dirname(__FILE__)
+  end
+
+  let(:metadata) do
+    JSON.parse(File.read("#{path}/metadata.json"))
+  end
+
+  let(:organization_id) { 3 }
+
+  it "performs import with required options and async" do
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{path}",
+      '--async'
+    ]
+    api_expects(:content_imports, :version)
+      .with_params('organization_id' => organization_id, 'path' => path, 'metadata' => metadata)
+      .returns(response)
+
+    result = run_cmd(@cmd + params)
+
+    assert_equal("Archive is being imported in task #{task_id}.\n", result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "performs import with required options" do
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{path}"
+    ]
+
+    api_expects(:content_imports, :version)
+      .with_params('organization_id' => organization_id, 'path' => path, 'metadata' => metadata)
+      .returns(response)
+
+    expect_foreman_task(task_id)
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails on missing required params' do
+    params = [
+      '--id=2'
+    ]
+
+    result = run_cmd(@cmd + params)
+    expected_error = "Could not import the archive."
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_equal(result.err[/#{expected_error}/], expected_error)
+  end
+
+  it 'fails on missing metadata.json' do
+    bad_path = "/nosuchdir"
+    params = [
+      "--organization-id=#{organization_id}",
+      "--path=#{bad_path}"
+    ]
+    result = run_cmd(@cmd + params)
+    expected_error = "Unable to find '#{bad_path}/metadata.json'."
+
+    assert_match(/#{expected_error}/, result.err)
+  end
+end

data/test/functional/content_view/content_view_helpers.rb

@@ -13,6 +13,8 @@ module ContentViewHelpers
   end
 
   def expect_content_view_version_search(params, returns)
-    expect_generic_search(:content_view_versions, params: params, returns: returns)
+    expect_lenient_search(:content_view_versions,
+                          params: params,
+                          returns: returns)
   end
 end

data/test/functional/content_view/list_test.rb

@@ -1,8 +1,6 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
 require File.join(File.dirname(__FILE__), '../lifecycle_environment/lifecycle_environment_helpers')
 
-require 'hammer_cli_katello/content_view_puppet_module'
-
 describe 'listing content-views' do
   include LifecycleEnvironmentHelpers
 

data/test/functional/content_view/publish_test.rb

@@ -1,5 +1,4 @@
 require_relative '../test_helper'
-require 'hammer_cli_katello/content_view_puppet_module'
 require 'hammer_cli_katello/content_view'
 
 module HammerCLIKatello
@@ -22,24 +21,4 @@ module HammerCLIKatello
       run_cmd(%w(content-view publish --major 5 --minor 1 --organization org1 --name cv --async))
     end
   end
-
-  describe ContentViewPuppetModule do
-    it 'allows adding a puppet module' do
-      ex = api_expects(:organizations, :index) do |p|
-        p[:search] == "name = \"org1\""
-      end
-      ex.returns(index_response([{'id' => 1}]))
-
-      ex = api_expects(:content_views, :index) do |p|
-        p['name'] == 'cv' && p['organization_id'] == 1
-      end
-      ex.returns(index_response([{'id' => 3}]))
-
-      api_expects(:content_views, :publish) do |p|
-        p['id'] == 3
-      end
-
-      run_cmd(%w(content-view publish --organization org1 --name cv --async))
-    end
-  end
 end

data/test/functional/content_view/version/promote_test.rb

@@ -1,8 +1,7 @@
 require_relative '../../test_helper'
-require 'hammer_cli_katello/content_view_puppet_module'
 
 module HammerCLIKatello
-  describe ContentViewPuppetModule do
+  describe ContentViewVersion::PromoteCommand do
     it 'allows promoting a content view' do
       ex = api_expects(:organizations, :index) do |p|
         p[:search] == "name = \"org1\""

data/test/functional/content_view/version/republish_repositories_test.rb

@@ -1,8 +1,7 @@
 require_relative '../../test_helper'
-require 'hammer_cli_katello/content_view_puppet_module'
 
 module HammerCLIKatello
-  describe ContentViewPuppetModule do
+  describe ContentViewVersion::RepublishRepositoriesCommand do
     include ForemanTaskHelpers
     include OrganizationHelpers
 

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

@@ -1,7 +1,5 @@
 require File.join(File.dirname(__FILE__), '../../test_helper')
 
-require 'hammer_cli_katello/content_view_puppet_module'
-
 describe 'apply an errata' do
   include ForemanTaskHelpers
 

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

@@ -1,7 +1,5 @@
 require File.join(File.dirname(__FILE__), '../../test_helper')
 
-require 'hammer_cli_katello/content_view_puppet_module'
-
 describe 'recalculate errata' do
   include ForemanTaskHelpers
 

data/test/functional/lifecycle_environment/lifecycle_environment_helpers.rb

@@ -4,7 +4,7 @@ module LifecycleEnvironmentHelpers
   include SearchHelpers
 
   def expect_lifecycle_environment_search(org_id, name, id)
-    expect_generic_search(:lifecycle_environments,
+    expect_lenient_search(:lifecycle_environments,
                           params: {'name' => name, 'organization_id' => org_id},
                           returns: {'id' => id})
   end

data/test/functional/organization/info_test.rb

@@ -0,0 +1,22 @@
+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
+
+  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)
+    result = run_cmd(@cmd + params)
+    expected = success_result(FieldMatcher.new('Simple Content Access', 'Enabled'))
+    assert_cmd(expected, result)
+  end
+end

data/test/functional/ping_test.rb

@@ -1,22 +1,59 @@
 require_relative 'test_helper'
 
 describe 'ping' do
+  let(:standard_response_services) do
+    {
+      'katello_agent' =>
+        {'status' => 'ok', 'message' => '0 Processed, 0 Failed', 'duration_ms' => '34'},
+      'foreman_tasks' =>
+        {'status' => 'ok', 'duration_ms' => '34'},
+      'candlepin' => {'status' => 'ok', 'duration_ms' => '34'},
+      'candlepin_events' =>
+        {'status' => 'ok', 'message' => '0 Processed, 0 Failed', 'duration_ms' => '34'},
+      'candlepin_auth' =>
+        {'status' => 'ok', 'duration_ms' => '34'},
+      'katello_events' =>
+        {'status' => 'ok', 'message' => '0 Processed, 0 Failed', 'duration_ms' => '34'},
+      'pulp3' =>
+        {'status' => 'ok', 'duration_ms' => '34'}
+    }
+  end
+  let(:standard_response) do
+    {
+      'status' => 'ok',
+      'services' => standard_response_services
+    }
+  end
+  let(:standard_response_keys) do
+    %w(katello_agent foreman_tasks candlepin candlepin_events
+       candlepin_auth katello_events pulp3).sort
+  end
+  let(:hammer_ping) { %w(ping katello) }
+
   it 'does not require authentication' do
-    api_expects(:ping, :index).returns(
+    api_expects(:ping, :index).returns(standard_response)
+
+    run_cmd(hammer_ping)
+  end
+
+  it "includes all keys" do
+    api_expects(:ping, :index).returns(standard_response)
+
+    result = JSON.parse(run_cmd(%w(--output=json ping katello))&.out)&.first&.keys&.sort
+    expected = standard_response_keys
+
+    assert_equal result, expected
+  end
+
+  it "skips katello_agent if not included in API response" do
+    response_without_katello_agent = {
       'status' => 'ok',
-      'services' => {
-        'foreman_tasks' => {'status' => 'ok', 'duration_ms' => '34'},
-        'foreman_auth' => {'status' => 'ok', 'duration_ms' => '34'},
-        'candlepin' => {'status' => 'ok', 'duration_ms' => '34'},
-        'candlepin_events' => {'status' => 'ok', 'message' => '0 messages', 'duration_ms' => '34'},
-        'candlepin_auth' => {'status' => 'ok', 'duration_ms' => '34'},
-        'katello_events' => {'status' => 'ok', 'message' => '0 messages', 'duration_ms' => '34'},
-        'pulp' => {'status' => 'ok', 'duration_ms' => '34'},
-        'pulp_auth' => {'status' => 'ok', 'duration_ms' => '34'},
-        'pulp3' => {'status' => 'ok', 'duration_ms' => '34'}
-      }
-    )
-
-    run_cmd(%w(ping katello))
+      'services' => standard_response_services.select { |k, _v| k != 'katello_agent' }
+    }
+    api_expects(:ping, :index).returns(response_without_katello_agent)
+    result = JSON.parse(run_cmd(%w(--output=json ping katello))&.out)&.first&.keys&.sort
+    expected = standard_response_keys.select { |k| k != 'katello_agent' }
+
+    assert_equal result, expected
   end
 end