hammer_cli_katello 1.5.0 → 1.5.1

data/test/functional/content_view/filter/update_test.rb

@@ -0,0 +1,91 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../content_view_helpers')
+require File.join(File.dirname(__FILE__), '../../organization/organization_helpers')
+
+module HammerCLIKatello
+  describe 'Filter::UpdateCommand' do
+    include ContentViewHelpers
+    include OrganizationHelpers
+
+    before do
+      @cmd = %w(content-view filter update)
+    end
+
+    it 'accepts filter id' do
+      params = ['--id=1', '--new-name=valis']
+
+      api_expects(:content_view_filters, :update) do |par|
+        par['id'] == 1 && par['name'] == 'valis'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'accepts filter name, content view name, description , and org name' do
+      params = ['--name=scanner', '--content-view=darkly', '--organization=pkd', '--new-name=ubik',
+                '--description=zoo']
+
+      expect_organization_search('pkd', 1)
+      expect_content_view_search(1, 'darkly', 1)
+
+      ex = api_expects(:content_view_filters, :index, 'Content view filters list') do |par|
+        par['content_view_id'] == 1 && par['name'] == 'scanner'
+      end
+      ex.returns(index_response([{'id' => '1'}]))
+
+      api_expects(:content_view_filters, :update) do |par|
+        par['id'] == '1' && par['name'] == 'ubik' && par['description'] == 'zoo'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'accepts filter name, content view name, and org label' do
+      params = ['--name=scanner', '--content-view=darkly',
+                '--organization-label=pkd', '--new-name=ubik']
+
+      expect_organization_search('pkd', 1, field: 'label')
+      expect_content_view_search(1, 'darkly', 1)
+
+      ex = api_expects(:content_view_filters, :index, 'Content view filters list') do |par|
+        par['content_view_id'] == 1 && par['name'] == 'scanner'
+      end
+      ex.returns(index_response([{'id' => '1'}]))
+
+      api_expects(:content_view_filters, :update) do |par|
+        par['id'] == '1' && par['name'] == 'ubik'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'accepts filter name, content view name, and org id' do
+      params = ['--name=scanner', '--content-view=darkly', '--organization-id=1', '--new-name=ubik']
+
+      expect_content_view_search('1', 'darkly', 1)
+
+      ex = api_expects(:content_view_filters, :index, 'Content view filters list') do |par|
+        par['content_view_id'] == 1 && par['name'] == 'scanner'
+      end
+      ex.returns(index_response([{'id' => '1'}]))
+
+      api_expects(:content_view_filters, :update) do |par|
+        par['id'] == '1' && par['name'] == 'ubik'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'requires content view name or id if name is supplied' do
+      params = ["--name=high-castle"]
+      expected_result = usage_error_result(
+        @cmd,
+        'At least one of options --content-view-id, --content-view is required.',
+        'Could not update the filter'
+      )
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

data/test/functional/content_view/list_test.rb

@@ -0,0 +1,68 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../lifecycle_environment/lifecycle_environment_helpers')
+
+describe 'listing content-views' do
+  include LifecycleEnvironmentHelpers
+
+  before do
+    @cmd = %w(content-view list)
+  end
+
+  let(:org_id) { 1 }
+  let(:lifecycle_env_id) { 1 }
+  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 "lists an organizations content-views" do
+    params = ["--organization-id=#{org_id}"]
+
+    ex = api_expects(:content_views, :index, 'Organizations content-views list') do |par|
+      par['organization_id'] == org_id && par['page'] == 1 &&
+        par['per_page'] == 1000
+    end
+
+    ex.returns(empty_response)
+    expected_result = success_result(
+      "----------------|------|-------|-----------|----------------|---------------
+CONTENT VIEW ID | NAME | LABEL | COMPOSITE | LAST PUBLISHED | REPOSITORY IDS
+----------------|------|-------|-----------|----------------|---------------
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "lists the content-views belonging to a lifecycle-environment by name" do
+    params = ["--organization-id=#{org_id}", '--lifecycle-environment=test']
+
+    expect_lifecycle_environment_search(org_id, 'test', lifecycle_env_id)
+
+    ex = api_expects(:content_views, :index, 'lifecycles content-views list') do |par|
+      par['organization_id'] == org_id && par['page'] == 1 &&
+        par['per_page'] == 1000
+    end
+
+    ex.returns(empty_response)
+    expected_result = success_result(
+      "----------------|------|-------|-----------|----------------|---------------
+CONTENT VIEW ID | NAME | LABEL | COMPOSITE | LAST PUBLISHED | REPOSITORY IDS
+----------------|------|-------|-----------|----------------|---------------
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/content_view/publish_test.rb

@@ -0,0 +1,24 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView do
+    it 'allows major & minor' 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 && p['major'] == 5 && p['minor'] == 1
+      end
+
+      run_cmd(%w(content-view publish --major 5 --minor 1 --organization org1 --name cv --async))
+    end
+  end
+end

data/test/functional/content_view/purge_test.rb

@@ -0,0 +1,97 @@
+require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require 'hammer_cli_katello/content_view'
+require 'hammer_cli_katello/content_view_purge'
+
+module HammerCLIKatello
+  describe ContentViewPurgeCommand do
+    include ForemanTaskHelpers
+
+    let(:versions) do
+      {'results' => [
+        {"id" => 23,
+         "version" => "1.0",
+         "content_view" => {"id" => 7, "name" => "test cv"},
+         "composite_content_views" => [{"id" => 14, "name" => "comp_view"}],
+         "composite_content_view_versions" => [],
+         "environments" => [{"id" => 1, "name" => "Library"}]},
+        {"id" => 34,
+         "version" => "2.0",
+         "content_view" => {"id" => 7, "name" => "test cv"},
+         "composite_content_views" => [],
+         "composite_content_view_versions" => [],
+         "environments" => []},
+        {"id" => 45,
+         "version" => "2.10",
+         "content_view" => {"id" => 7, "name" => "test cv"},
+         "composite_content_views" => [],
+         "composite_content_view_versions" => [],
+         "environments" => [{"id" => 1, "name" => "Library"}]},
+        {"id" => 56,
+         "version" => "3.0",
+         "content_view" => {"id" => 7, "name" => "test cv"},
+         "composite_content_views" => [{"id" => 14, "name" => "comp_view"}],
+         "composite_content_view_versions" => [],
+         "environments" => []},
+        {"id" => 67,
+         "version" => "7.0",
+         "content_view" => {"id" => 7, "name" => "test cv"},
+         "composite_content_views" => [],
+         "composite_content_view_versions" => [],
+         "environments" => []},
+        {"id" => 78,
+         "version" => "2.0",
+         "content_view" => {"id" => 7, "name" => "test cv"},
+         "composite_content_views" => [],
+         "composite_content_view_versions" => [{"id" => 87, "version" => "1.0"}],
+         "environments" => []}
+      ]}
+    end
+
+    it 'fails gracefully if count <= 0' do
+      api_expects_no_call
+
+      r = run_cmd(%w(content-view purge --id 2 --count -1))
+      assert(r.err.include?('Invalid value for --count option'), 'Incorrect error message')
+    end
+
+    it 'fails gracefully when there are no versions to delete' do
+      ex = api_expects(:content_view_versions, :index)
+      ex = ex.with_params("content_view_id" => '2')
+      ex.returns(versions)
+
+      r = run_cmd(%w(content-view purge --id 2 --count 3))
+      assert(r.err.include?('No versions to delete.'), 'Incorrect error message')
+    end
+
+    it 'only deletes unassociated versions' do
+      ex = api_expects(:content_view_versions, :index)
+      ex = ex.with_params("content_view_id" => '2')
+      ex.returns(versions)
+
+      [34, 67].each do |version_id|
+        ex = api_expects(:content_view_versions, :destroy)
+        ex = ex.with_params("id" => version_id)
+        ex.returns('id' => '3', 'state' => 'stopped')
+        expect_foreman_task('3')
+      end
+
+      run_cmd(%w(content-view purge --id 2 --count 0))
+    end
+
+    it 'allows for async purge of versions' do
+      ex = api_expects(:content_view_versions, :index)
+      ex = ex.with_params("content_view_id" => '2')
+      ex.returns(versions)
+
+      [34, 67].each do |version_id|
+        ex = api_expects(:content_view_versions, :destroy)
+        ex = ex.with_params("id" => version_id)
+        ex.returns('id' => '3')
+      end
+
+      run_cmd(%w(content-view purge --id 2 --count 0 --async))
+    end
+  end
+end

data/test/functional/content_view/remove_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::RemoveContentViewVersionCommand 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, 6])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3)
+      end
+      run_cmd(%w(content-view remove-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, 6])
+      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)
+      end
+      run_cmd(%w(content-view remove-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 remove-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, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-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, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-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, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-version --name cv2 --organization-label org1
+                   --content-view-version-id 3))
+      end
+    end
+  end
+end

data/test/functional/content_view/remove_test.rb

@@ -0,0 +1,79 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::RemoveCommand do
+    include ForemanTaskHelpers
+
+    describe 'content view version options' do
+      it 'allows removing versions by ID' do
+        ex = api_expects(:content_views, :remove) do |p|
+          p['id'] == 1 && p['content_view_version_ids'] == %w(6 7 8)
+        end
+        ex.returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --content-view-version-ids 6,7,8))
+      end
+
+      it 'allows removing versions by version number' do
+        versions = %w(6.0 7.0 8.0)
+        ids = [6, 7, 8]
+
+        api_expects(:content_view_versions, :index).with_params(
+          'search' => %(version = "#{versions.join('" or version = "')}"),
+          'content_view_id' => 1
+        ).returns(index_response(ids.map { |v| { 'id' => v } }))
+
+        api_expects(:content_views, :remove).with_params(
+          'id' => 1, 'content_view_version_ids' => ids
+        ).returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --content-view-versions 6.0,7.0,8.0))
+      end
+    end
+
+    describe 'environment options' do
+      it 'allows removing versions by ID' do
+        ex = api_expects(:content_views, :remove) do |p|
+          p['id'] == 1 && p['environment_ids'] == %w(6 7 8)
+        end
+        ex.returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --environment-ids 6,7,8))
+      end
+
+      it 'requires organization options when removing environments by name' do
+        api_expects_no_call
+
+        run_cmd(%w(content-view remove --id 1 --environments env6,env7,env8))
+      end
+
+      it 'allows removing environments by name' do
+        environment_ids = [6, 7, 8]
+        all_environment_ids = [3, 4, 5, 6, 7, 8]
+
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p['organization_id'] == '1'
+        end
+        ex.returns(index_response(all_environment_ids.map do |id|
+          {'id' => id, 'name' => "env#{id}"}
+        end))
+
+        ex = api_expects(:content_views, :remove) do |p|
+          p['id'] == 1 && p['environment_ids'] == environment_ids
+        end
+        ex.returns(id: '9')
+
+        expect_foreman_task('9')
+
+        run_cmd(%w(content-view remove --id 1 --environments env6,env7,env8 --organization-id 1))
+      end
+    end
+  end
+end

data/test/functional/content_view/update_test.rb

@@ -0,0 +1,53 @@
+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::UpdateCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
+    it 'allows minimal options' do
+      api_expects(:content_views, :update) do |p|
+        p['id'] == 2
+      end
+      run_cmd(%w(content-view update --id 2))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view update --name cv2))
+        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)
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2
+        end
+        run_cmd(%w(content-view update --name cv2 --organization-id 1))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2
+        end
+        run_cmd(%w(content-view update --name cv2 --organization org1))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2
+        end
+        run_cmd(%w(content-view update --name cv2 --organization-label org1))
+      end
+    end
+  end
+end

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

@@ -0,0 +1,108 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require File.join(File.dirname(__FILE__), '../../organization/organization_helpers')
+require File.join(File.dirname(__FILE__),
+                  '../../lifecycle_environment/lifecycle_environment_helpers')
+
+describe 'content-view version incremental-update' do
+  include OrganizationHelpers
+  include LifecycleEnvironmentHelpers
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-view version incremental-update)
+  end
+
+  it "performs incremental update with no updates" do
+    params = ['--errata-ids=FOO2012', '--lifecycle-environment-ids=1,2,3',
+              '--content-view-version-id=5']
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts'].nil? &&
+        par[:content_view_version_environments][0][:environment_ids] == %w(1 2 3) &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with update all hosts" do
+    params = ['--update-all-hosts=true', '--errata-ids=FOO2012',
+              '--lifecycle-environment-ids=1,2,3', '--content-view-version-id=5']
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts']['included'][:search] == '' &&
+        par[:content_view_version_environments][0][:environment_ids] == %w(1 2 3) &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with no environment" do
+    params = ['--update-all-hosts=true', '--errata-ids=FOO2012', '--content-view-version-id=5']
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts']['included'][:search] == '' &&
+        par[:content_view_version_environments][0][:environment_ids].nil? &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with names" do
+    params = ['--update-all-hosts=true', '--errata-ids=FOO2012',
+              '--lifecycle-environments=trump,cruz,bernie',
+              '--content-view-version-id=5', '--organization=USA']
+
+    expect_organization_search('USA', 5)
+    expect_lifecycle_environments_request(5, [{'name' => 'trump', 'id' => 1},
+                                              {'name' => 'cruz', 'id' => 2},
+                                              {'name' => 'bernie', 'id' => 3}])
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par['update_hosts']['included'][:search] == '' &&
+        par[:content_view_version_environments][0][:environment_ids] == [1, 2, 3] &&
+        par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par['add_content']['errata_ids'] == ['FOO2012']
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+
+  it "performs incremental update with packages" do
+    params = ['--content-view-version-id=5', '--packages=bla', '--lifecycle-environment-ids=1']
+
+    expect_generic_search(
+      :packages, params: {search: "name = \"bla\""}, returns: {'id' => 15}
+    )
+
+    ex = api_expects(:content_view_versions, :incremental_update, 'Incremental Update') do |par|
+      par[:content_view_version_environments][0][:content_view_version_id] == 5 &&
+        par["add_content"]["package_ids"] == [15]
+    end
+    ex.returns('id' => '3', 'state' => 'stopped')
+    expect_foreman_task('3')
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 0)
+  end
+end

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

@@ -0,0 +1,44 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_version'
+
+module HammerCLIKatello
+  describe ContentViewVersion::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:content_view_versions, :index)
+
+      run_cmd(%w(content-view version list))
+    end
+
+    describe 'allows organization' do
+      it 'id' do
+        api_expects(:content_view_versions, :index) do |p|
+          p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view version 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(:content_view_versions, :index) do |p|
+          p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view version 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(:content_view_versions, :index) do |p|
+          p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view version list --organization-label org1))
+      end
+    end
+  end
+end