hammer_cli_katello 1.1.3 → 1.2.0

data/test/functional/product/create_test.rb

@@ -0,0 +1,54 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../organization/organization_helpers')
+require File.join(File.dirname(__FILE__), '../sync_plan/sync_plan_helpers')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+
+describe "create a product" do
+  include OrganizationHelpers
+  include SyncPlanHelpers
+  include ProductHelpers
+
+  it 'by organization ID and product name' do
+    api_expects(:products, :create, 'create a product') do |par|
+      par['organization_id'] == 1 && par['name'] = 'product1'
+    end
+
+    assert_equal(
+      0,
+      run_cmd(%w(product create --organization-id 1 --name product1)).exit_code
+    )
+  end
+
+  it 'by organization name and product name' do
+    expect_organization_search('org1', 1)
+    api_expects(:products, :create, 'create a product') do |par|
+      par['organization_id'] == 1 && par['name'] = 'product1'
+    end
+    assert_equal(
+      0,
+      run_cmd(%w(product create --organization org1 --name product1)).exit_code
+    )
+  end
+
+  it 'by organization name, product name, and sync plan' do
+    expect_organization_search('org1', 1)
+    expect_sync_plan_search(1, 'sync_plan1', 1)
+    api_expects(:products, :create, 'create a product with a sync plan') do |par|
+      par['organization_id'] == 1 &&
+        par['name'] == 'product1' &&
+        par['sync_plan_id'] == 1
+    end
+    command = %w(product create --organization org1 --name product1 --sync-plan sync_plan1)
+    assert_equal(
+      0,
+      run_cmd(command).exit_code
+    )
+  end
+
+  it 'fails if no organization is given' do
+    refute_equal(
+      0,
+      run_cmd(%w(product create --name product1)).exit_code
+    )
+  end
+end

data/test/functional/product/delete_test.rb

@@ -0,0 +1,41 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../organization/organization_helpers')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+
+describe 'delete a product' do
+  include OrganizationHelpers
+  include ProductHelpers
+
+  it 'by ID' do
+    api_expects(:products, :destroy, 'delete a product') do |par|
+      par['id'] == 1
+    end
+    assert_equal(
+      0,
+      run_cmd(%w(product delete --id 1)).exit_code
+    )
+  end
+
+  it 'by organization ID and product name' do
+    expect_product_search(1, 'product1', 1)
+    api_expects(:products, :destroy, 'delete a product') do |par|
+      par['id'] == 1
+    end
+    assert_equal(
+      0,
+      run_cmd(%w(product delete --organization-id 1 --name product1)).exit_code
+    )
+  end
+
+  it 'by organization name and product name' do
+    expect_organization_search('org1', 1)
+    expect_product_search(1, 'product1', 1)
+    api_expects(:products, :destroy, 'delete a product') do |par|
+      par['id'] == 1
+    end
+    assert_equal(
+      0,
+      run_cmd(%w(product delete --organization org1 --name product1)).exit_code
+    )
+  end
+end

data/test/functional/product/info_test.rb

@@ -0,0 +1,33 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../organization/organization_helpers')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+
+describe "get product info" do
+  include OrganizationHelpers
+  include ProductHelpers
+
+  it 'by ID' do
+    expect_product_show(:id => 1)
+    assert_equal(
+      0,
+      run_cmd(%w(product info --id 1)).exit_code
+    )
+  end
+
+  it 'by organization id and product name' do
+    expect_product_show(:name => 'product1', :org_id => 1, :id => 1)
+    assert_equal(
+      0,
+      run_cmd(%w(product info --organization-id 1 --name product1)).exit_code
+    )
+  end
+
+  it 'by organization name and product name' do
+    expect_organization_search('org1', 1)
+    expect_product_show(:name => 'product1', :org_id => 1, :id => 1)
+    assert_equal(
+      0,
+      run_cmd(%w(product info --organization org1 --name product1)).exit_code
+    )
+  end
+end

data/test/functional/product/list_test.rb

@@ -0,0 +1,58 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), 'product_helpers')
+
+describe "listing products" do
+  include ProductHelpers
+
+  let(:empty_response_table) do
+    <<eostring
+---|------|-------------|--------------|--------------|-----------
+ID | NAME | DESCRIPTION | ORGANIZATION | REPOSITORIES | SYNC STATE
+---|------|-------------|--------------|--------------|-----------
+eostring
+  end
+
+  it 'displays the table properly' do
+    api_expects(:products, :index, 'index products').returns(index_response([]))
+    assert_cmd(
+      success_result(empty_response_table),
+      run_cmd(%w(product list --organization-id 1))
+    )
+  end
+
+  it 'by organization ID' do
+    ex = api_expects(:products, :index, 'index products') do |par|
+      par['organization_id'] == 1
+    end
+    ex.returns(index_response([]))
+
+    assert_equal(
+      0,
+      run_cmd(%w(product list --organization-id 1)).exit_code
+    )
+  end
+
+  it 'by organization ID and subscription ID' do
+    ex = api_expects(:products, :index, 'organization product list') do |par|
+      par['organization_id'] == 1 && par['subscription_id'] == 1
+    end
+    ex.returns(index_response([]))
+
+    assert_equal(
+      0,
+      run_cmd(%w(product list --organization-id 1 --subscription-id 1)).exit_code
+    )
+  end
+
+  it 'by organization ID and sync plan ID' do
+    ex = api_expects(:products, :index, 'organization product list') do |par|
+      par['organization_id'] == 1 && par['sync_plan_id'] == 1
+    end
+    ex.returns(index_response([]))
+
+    assert_equal(
+      0,
+      run_cmd(%w(product list --organization-id 1 --sync-plan-id 1)).exit_code
+    )
+  end
+end

data/test/functional/product/product_helpers.rb

@@ -0,0 +1,24 @@
+module ProductHelpers
+  def expect_generic_product_search(params = {}, returns = {})
+    api_expects(:products, :index, 'Find the Product')
+      .with_params(params).returns(index_response([returns]))
+  end
+
+  def expect_product_search(org_id, name, id)
+    expect_generic_product_search({'name' => name, 'organization_id' => org_id}, 'id' => id)
+  end
+
+  def expect_product_show(options = {})
+    raise "#{__method__} must be called with :id" unless (options.keys & [:id]).size == 1
+
+    if options[:name]
+      raise "#{__method__} used with :name also requires :org_id" unless options[:org_id]
+      expect_product_search(options[:org_id], options[:name], options[:id])
+    end
+
+    ex = api_expects(:products, :show, 'Show a Product') do |par|
+      par['id'] == options[:id]
+    end
+    ex.returns(index_response([{'id' => options[:id]}]))
+  end
+end

data/test/functional/product/remove_sync_plan_test.rb

@@ -0,0 +1,35 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../organization/organization_helpers')
+require File.join(File.dirname(__FILE__), '../sync_plan/sync_plan_helpers')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+
+describe "remove a product's sync plan" do
+  include OrganizationHelpers
+  include SyncPlanHelpers
+  include ProductHelpers
+
+  it "by product ID" do
+    api_expects(:products, :update, "set sync plan") do |params|
+      params['id'] == 1 && params['sync_plan_id'].nil?
+    end
+
+    assert_equal(
+      0,
+      run_cmd(%w(product remove-sync-plan --id 1)).exit_code
+    )
+  end
+
+  it "by product name" do
+    expect_product_search(1, 'product1', 1)
+    api_expects(:products, :update, "set sync plan") do |params|
+      params['id'] == 1 && params['sync_plan_id'].nil?
+    end
+
+    assert_equal(
+      0,
+      run_cmd(%w(product remove-sync-plan
+                 --organization-id 1
+                 --name product1)).exit_code
+    )
+  end
+end

data/test/functional/product/set_sync_plan_test.rb

@@ -0,0 +1,49 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../organization/organization_helpers')
+require File.join(File.dirname(__FILE__), '../sync_plan/sync_plan_helpers')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+
+describe "set a product's sync plan" do
+  include OrganizationHelpers
+  include SyncPlanHelpers
+  include ProductHelpers
+
+  it "by product ID and sync-plan ID" do
+    api_expects(:products, :update, "set sync plan") do |params|
+      params['id'] == 1 && params['sync_plan_id'] == 1
+    end
+
+    assert_equal(
+      0,
+      run_cmd(%w(product set-sync-plan --id 1 --sync-plan-id 1)).exit_code
+    )
+  end
+
+  it "by product ID and sync-plan name (requires org ID)" do
+    expect_sync_plan_search(1, 'syncplan1', 1)
+    api_expects(:products, :update, "set sync plan") do |params|
+      params['id'] == 1 && params['sync_plan_id'] == 1
+    end
+
+    assert_equal(
+      0,
+      run_cmd(%w(product set-sync-plan --organization-id 1 --id 1 --sync-plan syncplan1)).exit_code
+    )
+  end
+
+  it "by product name and sync-plan name (both requiring org ID)" do
+    expect_product_search(1, 'product1', 1)
+    expect_sync_plan_search(1, 'syncplan1', 1)
+    api_expects(:products, :update, "set sync plan") do |params|
+      params['id'] == 1 && params['sync_plan_id'] == 1
+    end
+
+    assert_equal(
+      0,
+      run_cmd(%w(product set-sync-plan
+                 --organization-id 1
+                 --name product1
+                 --sync-plan syncplan1)).exit_code
+    )
+  end
+end

data/test/functional/product/update_proxy.rb

@@ -0,0 +1,48 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'update an http proxy on a product' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(product update-proxy)
+  end
+
+  let(:sync_response) do
+    {
+      'id' => 1,
+      'state' => 'planned',
+      'action' => 'Update http proxy'
+    }
+  end
+
+  it 'updates product proxy policy' do
+    params = [
+      '--ids=1',
+      '--http-proxy-policy=use_selected_http_proxy',
+      '--http-proxy-id=1'
+    ]
+
+    ex = api_expects(:products_bulk_actions, :update_http_proxy, 'update an http-proxy')
+         .with_params('ids' => '1', 'http_proxy_policy' => 'use_selected_http_proxy',
+                      'http_proxy_id' => '1')
+    ex.returns(sync_response)
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it 'fails with missing required params' do
+    params = ['--proxy-id=1']
+
+    ex = api_expects(:products_bulk_actions, :update_http_proxy, 'update an http-proxy')
+         .with_params('proxy_id' => '1')
+    ex.returns(
+      'proxy_id' => '1'
+    )
+
+    result = run_cmd(@cmd + params)
+    assert_equal(result.exit_code, 70)
+  end
+end

data/test/functional/product/update_test.rb

@@ -0,0 +1,46 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../organization/organization_helpers')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+
+describe 'update a product' do
+  include OrganizationHelpers
+  include ProductHelpers
+
+  it 'by product ID' do
+    api_expects(:products, :update, 'update a product') do |par|
+      par['id'] == 1 &&
+        par['description'] == 'new_description'
+    end
+    assert_equal(
+      0,
+      run_cmd(%w(product update --organization-id 1 --id 1 --description new_description)).exit_code
+    )
+  end
+
+  it 'by organization ID and product name' do
+    expect_product_search(1, 'product1', 1)
+    api_expects(:products, :update, 'update a product') do |par|
+      par['id'] == 1 &&
+        par['description'] == 'new_description'
+    end
+    command = %w(product update --organization-id 1 --name product1 --description new_description)
+    assert_equal(
+      0,
+      run_cmd(command).exit_code
+    )
+  end
+
+  it 'by organization name and product name' do
+    expect_organization_search('org1', 1)
+    expect_product_search(1, 'product1', 1)
+    api_expects(:products, :update, 'update a product') do |par|
+      par['id'] == 1 &&
+        par['description'] == 'new_description'
+    end
+    command = %w(product update --organization org1 --name product1 --description new_description)
+    assert_equal(
+      0,
+      run_cmd(command).exit_code
+    )
+  end
+end

data/test/functional/repository/create_test.rb

@@ -0,0 +1,21 @@
+require_relative '../test_helper'
+
+describe "create repository" do
+  let(:org_id) { 1 }
+  let(:product_id) { 2 }
+  let(:name) { "repo1" }
+  let(:content_type) { "yum" }
+
+  it 'with basic options' do
+    api_expects(:repositories, :create)
+      .with_params(
+        name: name,
+        product_id: product_id,
+        content_type: content_type)
+
+    command = %W(repository create --organization-id #{org_id} --product-id #{product_id}
+                 --content-type #{content_type} --name #{name})
+
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+end

data/test/functional/repository/delete_test.rb

@@ -0,0 +1,95 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/repository'
+
+module HammerCLIKatello
+  describe Repository::DeleteCommand do
+    it 'allows minimal parameters' do
+      api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+      run_cmd(%w(repository delete --id 1))
+    end
+
+    describe 'product options' do
+      it 'are required to resolve repository name' do
+        api_expects_no_call
+        result = run_cmd(%w(repository delete --name repo1))
+        assert_includes(result.err, 'At least one of options --product, --product-id is required')
+      end
+
+      it 'can be specified via product id' do
+        api_expects(:repositories, :index)
+          .with_params('product_id' => 3, 'name' => 'repo1')
+          .returns(index_response([{'id' => 1}]))
+
+        api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+
+        run_cmd(%w(repository delete --name repo1 --product-id 3))
+      end
+
+      it 'can be specified via product name' do
+        api_expects(:products, :index)
+          .with_params('organization_id' => '6', 'name' => 'product3')
+          .returns(index_response([{'id' => 3}]))
+
+        api_expects(:repositories, :index)
+          .with_params('product_id' => 3, 'name' => 'repo1')
+          .returns(index_response([{'id' => 1}]))
+
+        api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+
+        run_cmd(%w(repository delete --name repo1 --product product3 --organization-id 6))
+      end
+
+      describe 'organization options' do
+        it 'can be specified by organization id' do
+          api_expects(:products, :index)
+            .with_params('organization_id' => '6', 'name' => 'product3')
+            .returns(index_response([{'id' => 3}]))
+
+          api_expects(:repositories, :index)
+            .with_params('product_id' => 3, 'name' => 'repo1')
+            .returns(index_response([{'id' => 1}]))
+
+          api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+
+          run_cmd(%w(repository delete --name repo1 --product product3 --organization-id 6))
+        end
+
+        it 'can be specified by organization name' do
+          api_expects(:organizations, :index)
+            .with_params(:search => "name = \"org6\"")
+            .returns(index_response([{'id' => 6}]))
+
+          api_expects(:products, :index)
+            .with_params('organization_id' => 6, 'name' => 'product3')
+            .returns(index_response([{'id' => 3}]))
+
+          api_expects(:repositories, :index)
+            .with_params('product_id' => 3, 'name' => 'repo1')
+            .returns(index_response([{'id' => 1}]))
+
+          api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+
+          run_cmd(%w(repository delete --name repo1 --product product3 --organization org6))
+        end
+
+        it 'can be specified by organization label' do
+          api_expects(:organizations, :index)
+            .with_params(:search => "label = \"org6\"")
+            .returns(index_response([{'id' => 6}]))
+
+          api_expects(:products, :index)
+            .with_params('organization_id' => 6, 'name' => 'product3')
+            .returns(index_response([{'id' => 3}]))
+
+          api_expects(:repositories, :index)
+            .with_params('product_id' => 3, 'name' => 'repo1')
+            .returns(index_response([{'id' => 1}]))
+
+          api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+
+          run_cmd(%w(repository delete --name repo1 --product product3 --organization-label org6))
+        end
+      end
+    end
+  end
+end

data/test/functional/repository/info_test.rb

@@ -0,0 +1,156 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+require File.join(File.dirname(__FILE__), './repository_helpers')
+require 'hammer_cli_katello/associating_commands'
+
+describe "get repository info" do
+  include ProductHelpers
+  include RepositoryHelpers
+
+  before do
+    @cmd = %w(repository info)
+  end
+
+  it "shows repository info by id" do
+    params = ['--id=1']
+    ex = api_expects(:repositories, :show, "Get info")
+    ex.returns(
+      'id' => 1,
+      'name' => 'Test Repo',
+      'label' => 'Test_Repo',
+      'description' => 'hammertime',
+      'organization' => {
+        'name' => 'Default Organization',
+        'label' => 'Default_Organization',
+        'id' => 1
+      },
+      'created_at' => '2020-08-05 15:35:36 UTC',
+      'updated_at' => '2020-08-05 15:35:36 UTC',
+      'content_type' => 'yum',
+      'product' => {
+        'id' => 79,
+        'name' => 'test'
+      },
+      'download_policy' => 'immediate',
+      'unprotected' => true,
+      'last_sync_words' => '3 minutes',
+      'mirror_on_sync' => true,
+      'relative_path' => 'Default_Organization/Library/Test_Repo',
+      'content_counts' => {
+        'rpm' => 1,
+        'srpm' => 0,
+        'package_group' => 0,
+        'erratum' => 1,
+        'module_stream' => 0
+      }
+    )
+    result = run_cmd(@cmd + params)
+    # rubocop:disable Style/WordArray
+    expected_fields = [['Id', '1'],
+                       ['Name', 'Test Repo'],
+                       ['Label', 'Test_Repo'],
+                       ['Description', 'hammertime'],
+                       ['Organization', 'Default Organization'],
+                       ['Red Hat Repository', 'no'],
+                       ['Content Type', 'yum'],
+                       ['Mirror on Sync', 'yes'],
+                       ['Publish Via HTTP', 'yes'],
+                       ['Relative Path', 'Default_Organization/Library/Test_Repo'],
+                       ['Download Policy', 'immediate'],
+                       ['HTTP Proxy', ''],
+                       ['Product', ''],
+                       ['Id', '79'],
+                       ['Name', 'test'],
+                       ['GPG Key', ''],
+                       ['Sync', ''],
+                       ['Status', 'Not Synced'],
+                       ['Last Sync Date', '3 minutes'],
+                       ['Created', '2020/08/05 15:35:36'],
+                       ['Updated', '2020/08/05 15:35:36'],
+                       ['Content Counts', ''],
+                       ['Packages', '1'],
+                       ['Source RPMS', '0'],
+                       ['Package Groups', '0'],
+                       ['Errata', '1'],
+                       ['Module Streams', '0']]
+    # rubocop:enable Style/WordArray
+    expected_results = expected_fields.map { |field| success_result(FieldMatcher.new(*field)) }
+    expected_results.each { |expected|  assert_cmd(expected, result) }
+  end
+
+  it "Shows information about a repository with organization-id and product name" do
+    org_id = 1
+    product_id = 2
+    repo_id = 3
+    params = ["--name=Test_Repo", "--product=Test_Product", "--organization-id=1"]
+
+    expect_product_search(org_id, 'Test_Product', product_id)
+
+    expect_repository_search(product_id, 'Test_Repo', repo_id)
+
+    ex = api_expects(:repositories, :show, "Get info")
+
+    ex.returns(
+      'id' => 1,
+      'name' => 'Test Repo',
+      'label' => 'Test_Repo',
+      'description' => 'hammertime',
+      'organization' => {
+        'name' => 'Default Organization',
+        'label' => 'Default_Organization',
+        'id' => 1
+      },
+      'created_at' => '2020-08-05 15:35:36 UTC',
+      'updated_at' => '2020-08-05 15:35:36 UTC',
+      'content_type' => 'yum',
+      'product' => {
+        'id' => 79,
+        'name' => 'Test_Product'
+      },
+      'download_policy' => 'immediate',
+      'unprotected' => true,
+      'last_sync_words' => '3 minutes',
+      'mirror_on_sync' => true,
+      'relative_path' => 'Default_Organization/Library/Test_Repo',
+      'content_counts' => {
+        'rpm' => 1,
+        'srpm' => 0,
+        'package_group' => 0,
+        'erratum' => 1,
+        'module_stream' => 0
+      }
+    )
+    result = run_cmd(@cmd + params)
+    # rubocop:disable Style/WordArray
+    expected_fields = [['Id', '1'],
+                       ['Name', 'Test Repo'],
+                       ['Label', 'Test_Repo'],
+                       ['Description', 'hammertime'],
+                       ['Organization', 'Default Organization'],
+                       ['Red Hat Repository', 'no'],
+                       ['Content Type', 'yum'],
+                       ['Mirror on Sync', 'yes'],
+                       ['Publish Via HTTP', 'yes'],
+                       ['Relative Path', 'Default_Organization/Library/Test_Repo'],
+                       ['Download Policy', 'immediate'],
+                       ['HTTP Proxy', ''],
+                       ['Product', ''],
+                       ['Id', '79'],
+                       ['Name', 'Test_Product'],
+                       ['GPG Key', ''],
+                       ['Sync', ''],
+                       ['Status', 'Not Synced'],
+                       ['Last Sync Date', '3 minutes'],
+                       ['Created', '2020/08/05 15:35:36'],
+                       ['Updated', '2020/08/05 15:35:36'],
+                       ['Content Counts', ''],
+                       ['Packages', '1'],
+                       ['Source RPMS', '0'],
+                       ['Package Groups', '0'],
+                       ['Errata', '1'],
+                       ['Module Streams', '0']]
+    # rubocop:enable Style/WordArray
+    expected_results = expected_fields.map { |field| success_result(FieldMatcher.new(*field)) }
+    expected_results.each { |expected|  assert_cmd(expected, result) }
+  end
+end