hammer_cli_katello 0.24.0 → 0.24.1

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

@@ -0,0 +1,236 @@
+require File.join(File.dirname(__FILE__), '../../../test_helper')
+require_relative '../content_export_helpers'
+require 'hammer_cli_katello/content_export'
+
+describe 'content-export incremental version' do
+  include ForemanTaskHelpers
+  include ContentExportHelpers
+
+  before do
+    @cmd = %w(content-export incremental version)
+  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(:content_view_id) { '77' }
+  let(:content_view_version_id) { 100 }
+
+  let(:version) { '10.0' }
+  let(:destination_server) { "dream.example.com" }
+
+  it "performs export with required options and async" do
+    params = [
+      "--id=#{content_view_version_id}",
+      '--destination-server=foo',
+      '--async'
+    ]
+    expects_repositories_in_version(content_view_version_id)
+
+    ex = api_expects(:content_export_incrementals, :version)
+    ex.returns(response)
+
+    result = run_cmd(@cmd + params)
+
+    assert_equal("Content view version 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=#{content_view_version_id}",
+      '--destination-server=foo'
+    ]
+
+    expects_repositories_in_version(content_view_version_id)
+    ex = api_expects(:content_export_incrementals, :version)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::VersionCommand.
+      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=#{content_view_version_id}",
+      "--destination-server=#{destination_server}",
+      "--from-history-id=#{export_history_id}"
+    ]
+    expects_repositories_in_version(content_view_version_id)
+    api_expects(:content_export_incrementals, :version)
+      .with_params('id' => content_view_version_id,
+                   'destination_server' => destination_server,
+                   'from_history_id' => export_history_id)
+      .returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::VersionCommand.
+      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 content view version:\n"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_equal(result.err[/#{expected_error}/], expected_error)
+  end
+
+  it 'correctly resolves content-view-id and content view version number' do
+    params = ["--content-view-id=#{content_view_id}",
+              "--version=#{version}",
+              "--destination-server=#{destination_server}",
+              "--async"]
+    expects_repositories_in_version(content_view_version_id)
+    cvv_expect = api_expects(:content_view_versions, :index) do |p|
+      assert_equal p['content_view_id'].to_s, content_view_id.to_s
+      assert_equal p["version"], version
+    end
+
+    cvv_expect.at_least_once.
+      returns(index_response([{'id' => content_view_version_id}]))
+
+    ex = api_expects(:content_export_incrementals, :version) do |p|
+      assert_equal p['id'], content_view_version_id
+      assert_equal p["destination_server"], destination_server
+    end
+    ex.returns(response)
+
+    run_cmd(@cmd + params)
+  end
+
+  it 'fails on missing content-view name/id' do
+    params = []
+
+    result = run_cmd(@cmd + params)
+    expected_error = " At least one of options --id, --content-view, --content-view-id is required"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'fails on missing content-view version' do
+    params = ["--content-view-id=2"]
+    result = run_cmd(@cmd + params)
+    expected_error = "Option --version is required"
+
+    assert_equal(result.exit_code, HammerCLI::EX_USAGE)
+    assert_match(/#{expected_error}/, result.err)
+  end
+
+  it 'fails on missing content-view missing org' do
+    params = ["--content-view=lol", "--version=4.0"]
+    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 content-view-id and content view version number' do
+    params = ["--content-view-id=#{content_view_id}",
+              "--version=#{version}",
+              "--destination-server=#{destination_server}",
+              "--async"]
+
+    cvv_expect = api_expects(:content_view_versions, :index) do |p|
+      assert_equal p['content_view_id'].to_s, content_view_id.to_s
+      assert_equal p["version"], version
+    end
+
+    cvv_expect.at_least_once.
+      returns(index_response([{'id' => content_view_version_id}]))
+
+    api_expects(:content_export_incrementals, :version)
+      .with_params('id' => content_view_version_id)
+      .returns(response)
+
+    expects_repositories_in_version(content_view_version_id)
+    run_cmd(@cmd + params)
+  end
+
+  it 'warns of lazy repositories' do
+    params = ["--id=#{content_view_version_id}"]
+    expects_repositories_in_version(content_view_version_id, [{id: 200}])
+
+    api_expects(:content_export_incrementals, :version)
+      .with_params('id' => content_view_version_id)
+      .returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::VersionCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Unable to fully export this version because/, result.out)
+    assert_match(/200/, result.out)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'Errors out on lazy repositories if --fail-on-missing-content' do
+    params = ["--id=#{content_view_version_id}",
+              "--fail-on-missing-content"]
+    expects_repositories_in_version(content_view_version_id, [{id: 200}])
+
+    ex = api_expects(:content_export_incrementals, :version)
+    ex.returns(response)
+
+    expect_foreman_task(task_id).at_least_once
+
+    HammerCLIKatello::ContentExportIncremental::VersionCommand.
+      any_instance.
+      expects(:fetch_export_history).
+      returns(export_history)
+
+    HammerCLIKatello::ContentExportIncremental::VersionCommand.
+      any_instance.
+      expects(:exit).with(HammerCLI::EX_SOFTWARE)
+
+    result = run_cmd(@cmd + params)
+    assert_match(/Unable to fully export this version because/, result.out)
+    assert_match(/200/, result.out)
+  end
+end

data/test/functional/{content_view/version/export_histories_test.rb → content_export/list_test.rb}

@@ -1,7 +1,7 @@
 require File.join(File.dirname(__FILE__), '../../test_helper')
-require 'hammer_cli_katello/content_view_version'
+require 'hammer_cli_katello/content_export'
 
-describe 'content-view version export-histories' do
+describe 'content-export list' do
   let(:empty_response) do
     {
       "total" => 0,
@@ -19,7 +19,7 @@ describe 'content-view version export-histories' do
   end
 
   it 'allows minimal options' do
-    ex = api_expects(:content_view_versions, :export_histories)
+    ex = api_expects(:content_exports, :index)
 
     ex.returns(empty_response)
     # rubocop:disable LineLength
@@ -28,7 +28,7 @@ ID | DESTINATION SERVER | PATH | CONTENT VIEW VERSION | CONTENT VIEW VERSION ID
 ---|--------------------|------|----------------------|-------------------------|------------|-----------
 ')
     # rubocop:enable LineLength
-    result = run_cmd(%w(content-view version export-histories))
+    result = run_cmd(%w(content-export list))
     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/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(:content_view_id) { 3 }
+
+  it "performs import with required options and async" do
+    params = [
+      "--content-view-id=#{content_view_id}",
+      "--path=#{path}",
+      '--async'
+    ]
+    api_expects(:content_imports, :version)
+      .with_params('content_view_id' => content_view_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 = [
+      "--content-view-id=#{content_view_id}",
+      "--path=#{path}"
+    ]
+
+    api_expects(:content_imports, :version)
+      .with_params('content_view_id' => content_view_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 = [
+      "--content-view-id=#{content_view_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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_katello
 version: !ruby/object:Gem::Version
-  version: 0.24.0
+  version: 0.24.1
 platform: ruby
 authors:
 - Adam Price
@@ -35,7 +35,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-11-02 00:00:00.000000000 Z
+date: 2020-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman
@@ -240,6 +240,11 @@ files:
 - lib/hammer_cli_katello/commands.rb
 - lib/hammer_cli_katello/composite_content_view_name_resolvable.rb
 - lib/hammer_cli_katello/content_credential.rb
+- lib/hammer_cli_katello/content_export.rb
+- lib/hammer_cli_katello/content_export_complete.rb
+- lib/hammer_cli_katello/content_export_helper.rb
+- lib/hammer_cli_katello/content_export_incremental.rb
+- lib/hammer_cli_katello/content_import.rb
 - lib/hammer_cli_katello/content_override.rb
 - lib/hammer_cli_katello/content_view.rb
 - lib/hammer_cli_katello/content_view_component.rb
@@ -364,6 +369,16 @@ files:
 - test/functional/capsule/list_test.rb
 - test/functional/content_credentials/info_test.rb
 - test/functional/content_credentials/list_test.rb
+- test/functional/content_export/complete/library_test.rb
+- test/functional/content_export/complete/version_test.rb
+- test/functional/content_export/content_export_helpers.rb
+- test/functional/content_export/generate_metadata_test.rb
+- test/functional/content_export/incremental/library_test.rb
+- test/functional/content_export/incremental/version_test.rb
+- test/functional/content_export/list_test.rb
+- test/functional/content_import/library_test.rb
+- test/functional/content_import/metadata.json
+- test/functional/content_import/version_test.rb
 - test/functional/content_view/add_content_view_version_test.rb
 - test/functional/content_view/add_repository_test.rb
 - test/functional/content_view/component/add_test.rb
@@ -388,7 +403,6 @@ files:
 - test/functional/content_view/remove_test.rb
 - test/functional/content_view/update_test.rb
 - test/functional/content_view/version/default_export_test.rb
-- test/functional/content_view/version/export_histories_test.rb
 - test/functional/content_view/version/export_test.rb
 - test/functional/content_view/version/import_test.rb
 - test/functional/content_view/version/incremental_update_test.rb
@@ -521,8 +535,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6.2
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Katello commands for Hammer
@@ -572,6 +585,16 @@ test_files:
 - test/functional/capsule/list_test.rb
 - test/functional/content_credentials/info_test.rb
 - test/functional/content_credentials/list_test.rb
+- test/functional/content_export/complete/library_test.rb
+- test/functional/content_export/complete/version_test.rb
+- test/functional/content_export/content_export_helpers.rb
+- test/functional/content_export/generate_metadata_test.rb
+- test/functional/content_export/incremental/library_test.rb
+- test/functional/content_export/incremental/version_test.rb
+- test/functional/content_export/list_test.rb
+- test/functional/content_import/library_test.rb
+- test/functional/content_import/metadata.json
+- test/functional/content_import/version_test.rb
 - test/functional/content_view/add_content_view_version_test.rb
 - test/functional/content_view/add_repository_test.rb
 - test/functional/content_view/component/add_test.rb
@@ -596,7 +619,6 @@ test_files:
 - test/functional/content_view/remove_test.rb
 - test/functional/content_view/update_test.rb
 - test/functional/content_view/version/default_export_test.rb
-- test/functional/content_view/version/export_histories_test.rb
 - test/functional/content_view/version/export_test.rb
 - test/functional/content_view/version/import_test.rb
 - test/functional/content_view/version/incremental_update_test.rb