hammer_cli_katello 1.1.3 → 1.2.0

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

@@ -0,0 +1,57 @@
+require_relative '../../test_helper'
+require_relative '../../repository/repository_helpers'
+
+describe 'content-view filter create' do
+  include RepositoryHelpers
+  before do
+    @cmd = %w(content-view filter create)
+    @base_params = ["--name=#{filter_name}", "--content-view-id=#{content_view_id}", "--type=rpm"]
+  end
+
+  let(:org_id) { 1 }
+  let(:filter_name) { 'test_filter' }
+  let(:content_view_id) { 1 }
+  let(:repositories) do
+    [
+      {'name' => 'repo-1', 'id' => 1},
+      {'name' => 'repo-2', 'id' => 2},
+      {'name' => 'repo-3', 'id' => 3}
+    ]
+  end
+  let(:repo_ids) { repositories.map { |repo| repo['id'] } }
+  let(:repo_names) { repositories.map { |repo| repo['name'] } }
+
+  it 'creates a content-view filter with repository ids' do
+    ids = repo_ids.join(',')
+    params = %W(--repository-ids=#{ids})
+
+    api_expects(:content_view_filters, :create, 'Create content-view filter')
+      .with_params('name' => filter_name, 'repository_ids' => repo_ids.map(&:to_s))
+      .returns({})
+
+    expected_result = success_result("Filter created.\n")
+    result = run_cmd(@cmd + @base_params + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it 'creates a content-view filter with repository names' do
+    params = %W(--repositories=#{repo_names.join(',')} --product-id 3)
+
+    expect_generic_repositories_search({'names' => repo_names, 'product_id' => 3}, repositories)
+
+    api_expects(:content_view_filters, :create, "Create content-view filter")
+      .with_params('name' => filter_name, 'repository_ids' => repo_ids, 'type' => 'rpm')
+
+    expected_result = success_result("Filter created.\n")
+    result = run_cmd(@cmd + @base_params + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it 'should fail with no product specified' do
+    ids = repo_ids.join(',')
+    params = ["--repositories=#{ids}", "--name=#{filter_name}", \
+              "--content-view-id=#{content_view_id}", "--type=rpm"]
+    result = run_cmd(@cmd + params)
+    assert(result.err[/--product-id, --product is required/])
+  end
+end

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

@@ -0,0 +1,90 @@
+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::DeleteCommand' do
+    include ContentViewHelpers
+    include OrganizationHelpers
+
+    before do
+      @cmd = %w(content-view filter delete)
+    end
+
+    it 'accepts filter id' do
+      params = ['--id=1']
+
+      api_expects(:content_view_filters, :destroy) do |par|
+        par['id'] == 1
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'accepts filter name, content view name, and org name' do
+      params = ['--name=scanner', '--content-view=darkly', '--organization=pkd']
+
+      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, :destroy) do |par|
+        par['id'] == '1'
+      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']
+
+      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, :destroy) do |par|
+        par['id'] == '1'
+      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']
+
+      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, :destroy) do |par|
+        par['id'] == '1'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'requires organization name or id if content view name is supplied' do
+      params = ["--name=high-castle", "--content-view=grasshopper"]
+      expected_result = usage_error_result(
+        @cmd,
+        'At least one of options --organization-id, --organization, --organization-label ' \
+        'is required.',
+        'Could not delete the filter'
+      )
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

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

@@ -0,0 +1,89 @@
+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::InfoCommand' do
+    include ContentViewHelpers
+    include OrganizationHelpers
+
+    before do
+      @cmd = %w(content-view filter info)
+    end
+
+    it 'accepts filter id' do
+      params = ['--id=1']
+
+      api_expects(:content_view_filters, :show) do |par|
+        par['id'] == 1
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'accepts filter name, content view name, and org name' do
+      params = ['--name=scanner', '--content-view=darkly', '--organization=pkd']
+
+      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, :show) do |par|
+        par['id'] == '1'
+      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']
+
+      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, :show) do |par|
+        par['id'] == '1'
+      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']
+
+      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, :show) do |par|
+        par['id'] == '1'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'requires organization name or id if content view name is supplied' do
+      params = ["--name=high-castle", "--content-view=grasshopper"]
+      expected_result = usage_error_result(
+        @cmd,
+        'At least one of options --organization-id, --organization, --organization-label ' \
+        'is required.'
+      )
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

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

@@ -0,0 +1,98 @@
+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')
+
+describe 'listing content view filters' do
+  include ContentViewHelpers
+  include OrganizationHelpers
+
+  before do
+    @cmd = %w(content-view filter list)
+  end
+
+  let(:org_id) { 1 }
+  let(:org_label) { 'Ubik' }
+  let(:cv_name) { "Trystero" }
+  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 content view filters" do
+    params = ["--content-view-id=1"]
+
+    ex = api_expects(:content_view_filters, :index, 'Content view filters list') do |par|
+      par['content_view_id'] == 1 && par['page'] == 1 && par['per_page'] == 1000
+    end
+    ex.returns(empty_response)
+
+    expected_result = success_result("----------|------|-------------|------|----------
+FILTER ID | NAME | DESCRIPTION | TYPE | INCLUSION
+----------|------|-------------|------|----------
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "lists content view filters for a content view name and org id" do
+    params = ["--organization-id=#{org_id}", "--content-view=#{cv_name}"]
+
+    expect_content_view_search(org_id.to_s, cv_name, 1)
+
+    ex = api_expects(:content_view_filters, :index, 'Content view filters list') do |par|
+      par['content_view_id'] == 1 && par['page'] == 1 && par['per_page'] == 1000
+    end
+    ex.returns(empty_response)
+
+    expected_result = success_result("----------|------|-------------|------|----------
+FILTER ID | NAME | DESCRIPTION | TYPE | INCLUSION
+----------|------|-------------|------|----------
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "lists content view filters for a content view name and org label" do
+    params = ["--organization-label=#{org_label}", "--content-view=#{cv_name}"]
+
+    expect_organization_search(org_label, org_id, field: 'label')
+    expect_content_view_search(org_id, cv_name, 1)
+
+    ex = api_expects(:content_view_filters, :index, 'Content view filters list') do |par|
+      par['content_view_id'] == 1 && par['page'] == 1 && par['per_page'] == 1000
+    end
+    ex.returns(empty_response)
+
+    expected_result = success_result("----------|------|-------------|------|----------
+FILTER ID | NAME | DESCRIPTION | TYPE | INCLUSION
+----------|------|-------------|------|----------
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it 'requires org name, label, or id if content view name is supplied' do
+    params = ["--content-view=#{cv_name}"]
+    expected_result = usage_error_result(
+      @cmd,
+      'At least one of options --organization-id, --organization, --organization-label is required.'
+    )
+    api_expects_no_call
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

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