hammer_cli_katello 1.5.0 → 1.5.1

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

@@ -0,0 +1,134 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+# rubocop:disable Metrics/ModuleLength
+module HammerCLIKatello
+  describe ContentViewComponent::UpdateCommand do
+    before do
+      @cmd = %w(content-view component update)
+      @component_version = OpenStruct.new(:id => 666, :version => "1.0")
+      @component = OpenStruct.new(:id => 6, :name => "component",
+                                  :content_view_version => @component_version)
+      @content_view_component = OpenStruct.new(:id => 1444,
+                                               :content_view => @component,
+                                               :content_view_version => @component_version)
+
+      @composite = OpenStruct.new(:id => 2, :name => "composite",
+                                  :content_view_components => [@content_view_component])
+      @organization = OpenStruct.new(:id => 1, :name => "great", :label => "label")
+    end
+
+    def setup_org_expectations
+      org_expect = api_expects(:organizations, :index) do |par|
+        par[:search] == "name = \"#{@organization.name}\""
+      end
+      org_expect.at_least_once.returns(index_response([{'id' => @organization.id.to_s}]))
+    end
+
+    def setup_content_view_expectations(content_view)
+      cv_expect = api_expects(:content_views, :index) do |p|
+        p['name'] == content_view.name && p["organization_id"].to_s == @organization.id.to_s
+      end
+      cv_expect.at_least_once.returns(index_response([{"id" => content_view.id}]))
+    end
+
+    def setup_content_view_components_expectations
+      cvc_expect = api_expects(:content_view_components, :index) do |p|
+        p[:composite_content_view_id].to_s == @composite.id.to_s
+      end
+      ret = {
+        "id" => @content_view_component.id,
+        "content_view" => {"id" => @content_view_component.content_view.id},
+        "content_view_version" => {"id" => @content_view_component.content_view_version.id}
+      }
+
+      cvc_expect.at_least_once.returns(index_response([ret]))
+    end
+
+    describe 'allows updating a component with' do
+      it 'id and component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--id=#{@content_view_component.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--latest"]
+
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_id'].to_s == @component.id.to_s &&
+            !p.key?('content_view_version_id')
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'id and component-content-view-version-id and latest=false' do
+        latest = false
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--id=#{@content_view_component.id}",
+                  "--component-content-view-version-id=#{@component.content_view_version.id}"]
+
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_version_id'].to_s == @component.content_view_version.id.to_s &&
+            !p.key?('content_view_id')
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--latest"]
+        setup_content_view_components_expectations
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_id'].to_s == @component.id.to_s &&
+            !p.key?('content_view_version_id')
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'by names and orgs' do
+        latest = false
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-view=#{@component.name}",
+                  "--component-content-view-version=#{@component.content_view_version.version}",
+                  "--organization=#{@organization.name}"]
+        # 1) Should fetch the org using org name specified in --organization
+        # 2) Should fetch composite content view using name in --composite-content-view
+        # 3) Should fetch component content view using name in --component-content-view
+        # 4) Should fetch the component_content_view_version_id
+        #    using cv specified in --component-content-view-id and version specified in
+        #    --component-content-view-version
+        # 5) Should set the component[:content_view_id] && component[:content_view_version_id]
+        #    correctly
+
+        setup_org_expectations
+        setup_content_view_expectations(@component)
+        setup_content_view_expectations(@composite)
+        setup_content_view_components_expectations
+        cvv_expect = api_expects(:content_view_versions, :index) do |p|
+          p['content_view_id'].to_s == @component.id.to_s &&
+            p["version"].to_s == @component.content_view_version.version.to_s
+        end
+        cvv_expect.at_least_once.
+          returns(index_response([{'id' => @component.content_view_version.id}]))
+
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_id'].to_s == @component.id.to_s &&
+            p['content_view_version_id'].to_s == @component.content_view_version.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+  end
+end

data/test/functional/content_view/content_view_helpers.rb

@@ -0,0 +1,20 @@
+require_relative '../search_helpers'
+
+module ContentViewHelpers
+  include SearchHelpers
+
+  def expect_content_view_search(org_id, name, id)
+    expect_generic_content_view_search(params: {'name' => name, 'organization_id' => org_id},
+                                       returns: {'id' => id})
+  end
+
+  def expect_generic_content_view_search(args)
+    expect_generic_search(:content_views, args)
+  end
+
+  def expect_content_view_version_search(params, returns)
+    expect_lenient_search(:content_view_versions,
+                          params: params,
+                          returns: returns)
+  end
+end

data/test/functional/content_view/copy_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::CopyCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
+    it 'allows minimal options' do
+      api_expects(:content_views, :copy) do |p|
+        p['id'] == 2 && p['name'] == 'cv2-dup'
+      end
+      run_cmd(%w(content-view copy --id 2 --new-name cv2-dup))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view copy --new-name cv2-dup --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, :copy) do |p|
+          p['id'] == 2 && p['name'] == 'cv2-dup'
+        end
+        run_cmd(%w(content-view copy --name cv2 --new-name cv2-dup --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, :copy) do |p|
+          p['id'] == 2 && p['name'] == 'cv2-dup'
+        end
+        run_cmd(%w(content-view copy --name cv2 --new-name cv2-dup --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, :copy) do |p|
+          p['id'] == 2 && p['name'] == 'cv2-dup'
+        end
+        run_cmd(%w(content-view copy --name cv2 --new-name cv2-dup --organization-label org1))
+      end
+    end
+  end
+end

data/test/functional/content_view/create_test.rb

@@ -0,0 +1,34 @@
+require_relative '../test_helper'
+
+describe 'content-view create' do
+  before do
+    @cmd = %w(content-view create)
+    @base_params = ["--organization-id=#{org_id}", "--name=#{name}"]
+  end
+  let(:error_heading) { "Could not create the content view" }
+  let(:name) { 'test-cv' }
+  let(:org_id) { 1 }
+  let(:repositories) do
+    [
+      {'id' => '1'},
+      {'id' => '2'},
+      {'id' => '3'}
+    ]
+  end
+
+  it 'creates the content view with repositories specified by ids' do
+    wanted = repositories.take(2)
+    ids = wanted.map { |repo| repo['id'] }
+    params = %W(--repository-ids=#{ids.join(',')})
+
+    api_expects(:content_views, :create, 'Create content view') do |par|
+      par['name'] == name &&
+        par['repository_ids'] == ids &&
+        par['organization_id'] == org_id
+    end
+
+    expected_result = success_result("Content view created.\n")
+    result = run_cmd(@cmd + @base_params + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/content_view/delete_test.rb

@@ -0,0 +1,62 @@
+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::DeleteCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+    include ForemanTaskHelpers
+
+    it 'allows minimal options' do
+      ex = api_expects(:content_views, :destroy) do |p|
+        p['id'] == 2
+      end
+      ex.returns(id: '8')
+      expect_foreman_task('8')
+      run_cmd(%w(content-view delete --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 delete --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)
+        ex = api_expects(:content_views, :destroy) do |p|
+          p['id'] == 2
+        end
+        ex.returns(id: '8')
+        expect_foreman_task('8')
+        run_cmd(%w(content-view delete --name cv2 --organization-id 1))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :destroy) do |p|
+          p['id'] == 2
+        end
+        ex.returns(id: '8')
+        expect_foreman_task('8')
+        run_cmd(%w(content-view delete --name cv2 --organization org1))
+      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, :destroy) do |p|
+          p['id'] == 2
+        end
+        ex.returns(id: '8')
+        expect_foreman_task('8')
+        run_cmd(%w(content-view delete --name cv2 --organization-label org1))
+      end
+    end
+  end
+end

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