hammer_cli_katello 1.1.3 → 1.2.0

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

@@ -0,0 +1,268 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_export'
+require_relative '../../lifecycle_environment/lifecycle_environment_helpers'
+require_relative '../../content_view/content_view_helpers'
+require_relative '../content_export_helpers'
+
+describe 'content-export incremental version' do
+  include ForemanTaskHelpers
+  include ContentExportHelpers
+  include ContentViewHelpers
+  include LifecycleEnvironmentHelpers
+
+  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 = "--version, --lifecycle-environment-id, --lifecycle-environment 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
+
+  it 'should accept content view and lifecycle environment and get the right version' do
+    env = "foo"
+    org_id = '100'
+    env_id = '223'
+    params = ["--content-view-id=#{content_view_id}",
+              "--lifecycle-environment=#{env}",
+              "--organization-id=#{org_id}"]
+    expect_lifecycle_environment_search(org_id, env, env_id)
+    expect_content_view_version_search({'environment_id' => env_id,
+                                        'content_view_id' => content_view_id},
+                                       'id' => content_view_version_id).at_least_once
+
+    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
+end

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/add_content_view_version_test.rb

@@ -0,0 +1,88 @@
+require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::AddContentViewVersionCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
+    it 'allows minimal options' do
+      ex = api_expects(:content_views, :show) do |p|
+        p[:id] == '1'
+      end
+      ex.returns('id' => 1, 'component_ids' => [1, 3])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3 6)
+      end
+      run_cmd(%w(content-view add-version --id 1 --content-view-version-id 6))
+    end
+
+    it 'resolves content view version ID' do
+      ex = api_expects(:content_view_versions, :index) do |p|
+        p['content_view_id'] == 3 && p['version'] == '2.1'
+      end
+      ex.returns(index_response([{'id' => 6}]))
+      ex.returns('id' => 1, 'component_ids' => [1, 3])
+      ex = api_expects(:content_views, :show) do |p|
+        p[:id] == '1'
+      end
+      ex.returns('id' => 1, 'component_ids' => [1, 3])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3 6)
+      end
+      run_cmd(%w(content-view add-version --id 1 --content-view-id 3 --content-view-version 2.1))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view add-version --name cv1))
+        assert(result.err[/--organization-id, --organization, --organization-label is required/],
+               "Organization option requirements are validated")
+      end
+
+      it 'allows organization id' do
+        expect_content_view_search('1', 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization-id 1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization org1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization-label org1
+                   --content-view-version-id 3))
+      end
+    end
+  end
+end

data/test/functional/content_view/add_repository_test.rb

@@ -0,0 +1,21 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView do
+    describe 'AddRepositoryCommand' do
+      it 'allows adding a repository by ID' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\""
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv' && p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view add-repository --organization org1 --name cv --repository-id 1))
+      end
+    end
+  end
+end