hammer_cli_katello 0.14.1 → 0.15.0

data/test/functional/apipie_helper_test.rb

@@ -0,0 +1,26 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'apipie helper' do
+  before do
+    class ApipieTestHelper
+      include HammerCLIKatello::ApipieHelper
+    end
+
+    @apipie_helper = ApipieTestHelper.new
+  end
+
+  it "calls show method apipie resource" do
+    api_expects(:repositories, :show).with_params('id' => '1').returns({})
+    assert @apipie_helper.show(:repositories, 'id' => '1')
+  end
+
+  it "calls index method apipie resource" do
+    api_expects(:repositories, :index).with_params('name' => 'foo').returns('results' => [])
+    assert @apipie_helper.index(:repositories, 'name' => 'foo')
+  end
+
+  it "call method for apipie resources works" do
+    api_expects(:repositories, :index).with_params('name' => 'foo').returns('results' => [])
+    assert @apipie_helper.call(:index, :repositories, 'name' => 'foo')
+  end
+end

data/test/functional/content_view/publish_test.rb

@@ -2,6 +2,26 @@ require_relative '../test_helper'
 require 'hammer_cli_katello/content_view_puppet_module'
 
 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
+
   describe ContentViewPuppetModule do
     it 'allows adding a puppet module' do
       ex = api_expects(:organizations, :index) do |p|

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

@@ -0,0 +1,87 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+
+describe 'content-view version export' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-view version export)
+  end
+
+  it "performs export" do
+    params = [
+      '--id=5',
+      '--export-dir=/tmp/exports'
+    ]
+
+    ex = api_expects(:content_view_versions, :show)
+    ex.returns(
+      'id' => '5',
+      'repositories' => [{'id' => '2'}],
+      'major' => 1,
+      'minor' => 0,
+      'content_view' => {'name' => 'cv'}
+    )
+
+    ex = api_expects(:repositories, :show).with_params('id' => '2')
+    ex.returns(
+      'id' => '2',
+      'label' => 'Test_Repo',
+      'content_type' => 'yum',
+      'backend_identifier' => 'Default_Organization-Library-Test_Repo',
+      'relative_path' => 'Default_Organization/Library/Test_Repo',
+      'library_instance_id' => '1'
+    )
+
+    api_expects(:repositories, :show).with_params('id' => '1').returns(
+      'id' => '1',
+      'download_policy' => 'immediate'
+    )
+    api_expects(:packages, :index).returns('results' => [])
+    api_expects(:errata, :index).returns('results' => [])
+
+    File.expects(:exist?).with('/usr/share/foreman').returns(true)
+    Dir.expects(:chdir).with("/var/lib/pulp/published/yum/https/repos/").returns(true)
+    Dir.expects(:mkdir).with('/tmp/exports/export-5').returns(0)
+    Dir.expects(:chdir).with('/tmp/exports').returns(0)
+    Dir.expects(:chdir).with('/tmp/exports/export-5').returns(0)
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "fails export if any repository is set to on_demand" do
+    params = [
+      '--id=5',
+      '--export-dir=/tmp/exports'
+    ]
+
+    ex = api_expects(:content_view_versions, :show)
+    ex.returns(
+      'id' => '5',
+      'repositories' => [{'id' => '2'}],
+      'major' => 1,
+      'minor' => 0,
+      'content_view' => {'name' => 'cv'}
+    )
+
+    ex = api_expects(:repositories, :show).with_params('id' => '2')
+    ex.returns(
+      'id' => '2',
+      'label' => 'Test_Repo',
+      'content_type' => 'yum',
+      'backend_identifier' => 'Default_Organization-Library-Test_Repo',
+      'relative_path' => 'Default_Organization/Library/Test_Repo',
+      'library_instance_id' => '1'
+    )
+
+    api_expects(:repositories, :show).with_params('id' => '1').returns(
+      'id' => '1',
+      'download_policy' => 'on_demand'
+    )
+
+    File.expects(:exist?).with('/usr/share/foreman').returns(true)
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+  end
+end

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

@@ -0,0 +1,111 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+
+describe 'content-view version import' do
+  include ForemanTaskHelpers
+
+  before do
+    @cmd = %w(content-view version import)
+  end
+
+  it "performs import" do
+    params = [
+      '--export-tar=/tmp/exports/export-2.tar',
+      '--organization-id=1'
+    ]
+
+    File.expects(:exist?).with('/usr/share/foreman').returns(true)
+
+    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
+    Dir.expects(:chdir).with('/tmp/exports').returns(0)
+    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
+    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
+      JSON.dump(
+        'name' => 'Foo View',
+        'major' => '5',
+        'minor' => '0',
+        'repositories' => [{
+          'label' => 'foo',
+          'rpm_filenames' => ['foo-1.0-1.el7']
+        }]
+      )
+    )
+
+    ex = api_expects(:content_views, :index)
+    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
+    ex.returns(
+      'results' => [{
+        'id' => '5',
+        'repositories' => [{'id' => '2', 'label' => 'foo'}],
+        'content_view' => {'name' => 'cv'}
+      }]
+    )
+
+    ex = api_expects(:repositories, :index)
+    ex = ex.with_params('organization_id' => '1', 'library' => true)
+    ex.returns(
+      'results' => [{
+        'id' => '2',
+        'label' => 'foo'
+      }]
+    )
+
+    ex = api_expects(:repositories, :sync)
+    ex = ex.with_params('id' => '2', 'source_url' => "file:///tmp/exports/export-2/")
+    ex.returns('id' => '2', 'state' => 'planned')
+
+    expect_foreman_task('3')
+
+    ex = api_expects(:content_views, :publish)
+    ex = ex.with_params(
+      'id' => '5',
+      'major' => '5',
+      'minor' => '0',
+      'repos_units' => [{
+        'label' => 'foo',
+        'rpm_filenames' => ['foo-1.0-1.el7']
+      }]
+    )
+    ex.returns('id' => '2', 'state' => 'planned')
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_OK, result.exit_code)
+  end
+
+  it "fails import if any repository does not exist" do
+    params = [
+      '--export-tar=/tmp/exports/export-2.tar',
+      '--organization-id=1'
+    ]
+
+    File.expects(:exist?).with('/usr/share/foreman').returns(true)
+
+    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
+    Dir.expects(:chdir).with('/tmp/exports').returns(0)
+    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
+    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
+      JSON.dump(
+        'name' => 'Foo View',
+        'repositories' => ['label' => 'foo']
+      )
+    )
+
+    ex = api_expects(:content_views, :index)
+    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
+    ex.returns(
+      'results' => [{
+        'id' => '5',
+        'repositories' => [{'id' => '2', 'label' => 'foo'}],
+        'content_view' => {'name' => 'cv'}
+      }]
+    )
+
+    ex = api_expects(:repositories, :index)
+    ex = ex.with_params('organization_id' => '1', 'library' => true)
+    ex.returns([])
+
+    result = run_cmd(@cmd + params)
+    assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
+  end
+end

data/test/functional/local_helper_test.rb

@@ -0,0 +1,30 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'local helper' do
+  before do
+    class SuperLocalTestHelper
+      def parse_subcommand
+        true
+      end
+    end
+
+    class LocalTestHelper < SuperLocalTestHelper
+      include HammerCLIKatello::LocalHelper
+    end
+
+    @local_helper = LocalTestHelper.new
+  end
+
+  it "does not fail if /usr/share/foreman exists" do
+    File.expects(:exist?).with('/usr/share/foreman').returns(true)
+    assert @local_helper.parse_subcommand
+  end
+
+  it "fails if /usr/share/foreman does not exist" do
+    File.expects(:exist?).with('/usr/share/foreman').returns(false)
+
+    assert_raises RuntimeError do
+      @local_helper.parse_subcommand
+    end
+  end
+end

data/test/functional/module_stream/info_test.rb

@@ -0,0 +1,58 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/module_stream'
+
+module HammerCLIKatello
+  describe ModuleStreamCommand::InfoCommand do
+    it 'allows ID' do
+      api_expects(:module_streams, :show).with_params('id' => '1')
+
+      run_cmd(%w(module-stream info --id 1))
+    end
+
+    it 'resolves ID from name and repo id' do
+      api_expects(:module_streams, :index)
+        .with_params(search: "name = \"duck\"", name: "duck", repository_id: "1")
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :show)
+        .with_params(repository_id: "1", id: 1)
+
+      run_cmd(%w(module-stream info --name duck --repository-id 1))
+    end
+
+    it 'resolves ID from name, repo name, and product id' do
+      api_expects(:module_streams, :index)
+        .with_params(search: "name = \"walrus\"")
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:repositories, :index)
+        .with_params(name: 'zoo', product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :show)
+        .with_params(repository_id: 1, id: 1)
+
+      run_cmd(%w(module-stream info --name walrus --repository zoo --product-id 1))
+    end
+
+    it 'resolves ID from name, repo name, product name, and org id' do
+      api_expects(:products, :index)
+        .with_params(name: "habitat", organization_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :index)
+        .with_params(search: "name = \"walrus\"")
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:repositories, :index)
+        .with_params(name: 'zoo', product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :show)
+        .with_params(repository_id: 1, id: 1)
+
+      run_cmd(%w(module-stream info --name walrus --repository zoo
+                 --product habitat --organization-id 1))
+    end
+  end
+end

data/test/functional/module_stream/list_test.rb

@@ -0,0 +1,53 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/module_stream'
+
+module HammerCLIKatello
+  describe ModuleStreamCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:module_streams, :index)
+
+      run_cmd(%w(module-stream list))
+    end
+
+    it 'can be provided by repository ID' do
+      api_expects(:module_streams, :index).with_params(repository_id: 1)
+
+      run_cmd(%w(module-stream list --repository-id 1))
+    end
+
+    it 'product params needed with repository name' do
+      cmd = run_cmd(%w(module-stream list --repository Repo))
+      error_msg = "At least one of options --product, --product-id is required."
+      api_expects_no_call
+      assert_match error_msg, cmd.err
+    end
+
+    it 'can be provided by repository name and product id' do
+      api_expects(:repositories, :index)
+        .with_params(name: "Repo", product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :index).with_params(repository_id: 1)
+
+      run_cmd(%w(module-stream list --repository Repo --product-id 1))
+    end
+
+    it 'can be provided by repository id and organization id' do
+      api_expects(:module_streams, :index)
+        .with_params(repository_id: 1, organization_id: 1)
+
+      run_cmd(%w(module-stream list --repository-id 1 --organization-id 1))
+    end
+
+    it 'can be provided by repository name, product id, and organization id' do
+      api_expects(:repositories, :index)
+        .with_params(name: "Repo", product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:module_streams, :index)
+        .with_params(repository_id: 1, organization_id: 1)
+
+      run_cmd(%w(module-stream list --repository Repo --product-id 1 --organization-id 1))
+    end
+  end
+end

data/test/functional/package_group/list_test.rb

@@ -17,17 +17,22 @@ module HammerCLIKatello
       run_cmd(%w(package-group list --repository-id 1))
     end
 
-    it 'can be provided by repository name' do
-      ex = api_expects(:repositories, :index) do |params|
-        params['name'] = 'Repo'
-      end
-      ex.returns(index_response([{'id' => 1}]))
+    it 'needs product options with repository name' do
+      cmd = run_cmd(%w(package-group list --repository Repo))
+      api_expects_no_call
+      error_msg = "At least one of options --product, --product-id is required."
+      assert_match error_msg, cmd.err
+    end
 
-      api_expects(:package_groups, :index) do |params|
-        params['repository_id'] == 1
-      end
+    it 'can be provided product id and repository name' do
+      api_expects(:repositories, :index)
+        .with_params(name: "Repo", product_id: 1)
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:package_groups, :index)
+        .with_params(repository_id: 1)
 
-      run_cmd(%w(package-group list --repository Repo))
+      run_cmd(%w(package-group list --repository Repo --product-id 1))
     end
   end
 end

data/test/functional/sync_plan/create_test.rb

@@ -0,0 +1,60 @@
+require_relative '../test_helper'
+
+describe "create sync plan" do
+  let(:org_id) { 1 }
+  let(:name) { "sync_plan1" }
+  let(:hourly) { "hourly" }
+  let(:custom) { "custom cron" }
+  let(:cron) { "10 * * * *" }
+  let(:date) { "2018-09-08T00:00:00+00:00" }
+
+  it 'with organization ID,name,interval,date and enabled' do
+    api_expects(:sync_plans, :create, 'create a sync plan').
+      with_params('organization_id' => org_id,
+                  'name' => name,
+                  'interval' => hourly,
+                  'sync_date' => date,
+                  'enabled' => true)
+    command = %W(sync-plan create --organization-id #{org_id} --name #{name}
+                 --interval #{hourly} --enabled 1 --sync-date #{date})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'with custom cron' do
+    api_expects(:sync_plans, :create, 'create a sync plan').
+      with_params('organization_id' => org_id,
+                  'name' => name,
+                  'interval' => custom,
+                  'cron_expression' => cron,
+                  'sync_date' => date,
+                  'enabled' => true)
+    # end
+    command = %W(sync-plan create --organization-id #{org_id} --name #{name} --interval #{custom}
+                 --cron-expression #{cron} --enabled 1 --sync-date #{date})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'fails without organization-id' do
+    command = %w(sync-plan create --name #{name}
+                 --interval #{hourly} --enabled 1 --sync-date #{date})
+    refute_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'fails without name' do
+    command = %w(sync-plan create --organization-id #{org_id}
+                 --interval #{hourly} --enabled 1 --sync-date #{date})
+    refute_equal(0, run_cmd(command).err)
+  end
+
+  it 'fails without interval' do
+    command = %w(sync-plan create --organization-id #{org_id} --name #{name}
+                 --enabled 1 --sync-date #{date})
+    refute_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'fails without enabled' do
+    command = %w(sync-plan create --organization-id #{org_id} --name #{name}
+                 --interval #{hourly} --sync-date #{date})
+    refute_equal(0, run_cmd(command).exit_code)
+  end
+end

data/test/functional/sync_plan/delete_test.rb

@@ -0,0 +1,46 @@
+require_relative '../test_helper'
+require_relative 'sync_plan_helpers'
+require_relative '../organization/organization_helpers'
+
+describe 'delete a sync plan' do
+  include OrganizationHelpers
+  include SyncPlanHelpers
+
+  let(:org_id) { 1 }
+  let(:id) { 1 }
+  let(:name) { "sync_plan1" }
+  let(:org_name) { "org1" }
+
+  it 'by organization ID and sync plan id' do
+    api_expects(:sync_plans, :destroy, 'delete a sync plan').
+      with_params('organization_id' => org_id,
+                  'id' => id)
+    command = %W(sync-plan delete --organization-id #{org_id} --id #{id})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'by organization ID and sync plan name' do
+    expect_sync_plan_search(1, 'sync_plan1', 1)
+    api_expects(:sync_plans, :destroy, 'delete a sync plan').
+      with_params('id' => id)
+    command = %W(sync-plan delete --organization-id #{org_id} --name #{name})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'by organization name and sync plan name' do
+    expect_organization_search('org1', 1)
+    expect_sync_plan_search(1, 'sync_plan1', 1)
+    api_expects(:sync_plans, :destroy, 'delete a sync plan').
+      with_params('id' => id)
+    command = %W(sync-plan delete --organization #{org_name} --name #{name})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'by organization name and sync plan id' do
+    expect_organization_search(org_name, org_id)
+    api_expects(:sync_plans, :destroy, 'delete a sync plan').
+      with_params('id' => id)
+    command = %W(sync-plan delete --organization #{org_name} --id #{id})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+end

data/test/functional/sync_plan/update_test.rb

@@ -0,0 +1,44 @@
+require_relative '../test_helper'
+require_relative 'sync_plan_helpers'
+require_relative '../organization/organization_helpers'
+
+describe 'update a sync plan' do
+  include OrganizationHelpers
+  include SyncPlanHelpers
+
+  let(:org_id) { 1 }
+  let(:id) { 1 }
+  let(:name) { "sync_plan1" }
+  let(:org_name) { "org1" }
+  let(:desc) { "New Description" }
+
+  it 'with organization id and sync plan ID' do
+    api_expects(:sync_plans, :update, 'update a sync plan').
+      with_params('description' => desc,
+                  'id' => id)
+    command = %W(sync-plan update --organization-id #{org_id} --id #{id}
+                 --description #{desc})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'with organization ID and sync plan name' do
+    expect_sync_plan_search(org_id, name, id)
+    api_expects(:sync_plans, :update, 'update a sync plan').
+      with_params('description' => desc,
+                  'id' => id)
+    command = %W(sync-plan update --organization-id #{org_id} --name #{name}
+                 --description #{desc})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+
+  it 'with organization name and sync plan name' do
+    expect_organization_search(org_name, org_id)
+    expect_sync_plan_search(org_id, name, id)
+    api_expects(:sync_plans, :update, 'update a sync plan').
+      with_params('description' => desc,
+                  'id' => id)
+    command = %W(sync-plan update --organization #{org_name} --name #{name}
+                 --description #{desc})
+    assert_equal(0, run_cmd(command).exit_code)
+  end
+end

data/test/test_helper.rb

@@ -17,7 +17,7 @@ require 'minitest/spec'
 require 'mocha/setup'
 require 'hammer_cli'
 
-KATELLO_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '3.8')
+KATELLO_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '3.9')
 
 if HammerCLI.context[:api_connection]
   HammerCLI.context[:api_connection].create('foreman') do

data/test/unit/search_options_creators_test.rb

@@ -118,9 +118,12 @@ describe HammerCLIKatello::SearchOptionsCreators do
   end # describe 'without the katello api'
 
   describe '#create_search_options_with_katello_api' do
+    let(:searchable1) { HammerCLIForeman::Searchable.new('one', '') }
+    let(:searchable2) { HammerCLIForeman::Searchable.new('two', '') }
+
     it 'translates all searchable fields from options' do
       search_options_creators.stubs(:searchables).
-        returns([search_options_creators.method(:one), search_options_creators.method(:two)])
+        returns([searchable1, searchable2])
 
       search_options_creators.create_search_options_with_katello_api(
         {'option_one' => 1, 'option_two' => 2}, resource