hammer_cli_katello 0.0.26 → 0.1.0

data/test/data/Readme.md

@@ -0,0 +1,5 @@
+# Data for tests
+
+Follow the [hammer-cli-foreman
+instructions](https://github.com/theforeman/hammer-cli-foreman/blob/master/test/data/README.md) with the Katello plugin installed in
+Foreman.

data/test/functional/activaton_key/add_host_collection_test.rb

@@ -0,0 +1,62 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/associating_commands'
+
+module HammerCLIKatello
+  module AssociatingCommands
+    module HostCollection
+      describe AddHostCollectionCommand do
+        it 'allows minimal options' do
+          ex = api_expects(:activation_keys, :show) do |p|
+            p[:id] == '1'
+          end
+          ex.returns('id' => 1, host_collections: [{}])
+
+          api_expects(:activation_keys, :update) do |p|
+            p['id'] == '1' && p['organization_id'] == 1 && p['host_collection_ids'] == ['3']
+          end
+
+          run_cmd(%w(activation-key add-host-collection --organization-id 1
+                     --host-collection-id 3 --id 1))
+        end
+
+        it 'allows resolving host collection name' do
+          ex = api_expects(:host_collections, :index) do |p|
+            p['name'] == 'hc3' && p['organization_id'] == 1
+          end
+          ex.returns(index_response([{'id' => 3}]))
+
+          ex = api_expects(:activation_keys, :show) do |p|
+            p[:id] == '1'
+          end
+          ex.returns(id: 1, host_collection_ids: [])
+
+          api_expects(:activation_keys, :update) do |p|
+            p['id'] == '1' && p['organization_id'] == 1 && p['host_collection_ids'] == ['3']
+          end
+
+          run_cmd(%w(activation-key add-host-collection --organization-id 1
+                     --host-collection hc3 --id 1))
+        end
+
+        it 'allows resolving activation key name' do
+          ex = api_expects(:activation_keys, :index) do |p|
+            p['name'] == 'ak1' && p['organization_id'] == 1
+          end
+          ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+          ex = api_expects(:activation_keys, :show) do |p|
+            p[:id] == 1
+          end
+          ex.returns(id: 1, host_collections: [])
+
+          api_expects(:activation_keys, :update) do |p|
+            p['id'] == 1 && p['organization_id'] == 1 && p['host_collection_ids'] == ['3']
+          end
+
+          run_cmd(%w(activation-key add-host-collection --organization-id 1
+                     --host-collection-id 3 --name ak1))
+        end
+      end
+    end
+  end
+end

data/test/functional/activaton_key/create_test.rb

@@ -0,0 +1,33 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/activation_key'
+
+module HammerCLIKatello
+  describe ActivationKeyCommand do
+    describe CreateCommand do
+      it 'allows promoting a content view' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\""
+        end
+        ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p['organization_id'] == 1 && p['name'] == 'test'
+        end
+        ex.returns(index_response([{'id' => 9}]))
+
+        ex = api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv' && p['organization_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        api_expects(:activation_keys, :create) do |p|
+          p['name'] == 'ak' && p['environment_id'] == 9 && p['organization_id'] == 1 &&
+            p['content_view_id'] == 3
+        end
+
+        run_cmd(%w(activation-key create --organization org1 --name ak --content-view cv
+                   --lifecycle-environment test --unlimited-hosts))
+      end
+    end
+  end
+end

data/test/functional/activaton_key/list_test.rb

@@ -1,8 +1,6 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
 require File.join(File.dirname(__FILE__), '../lifecycle_environment/lifecycle_environment_helpers')
 
-require 'hammer_cli_katello/content_view_puppet_module'
-
 describe 'listing activation-keys' do
   include LifecycleEnvironmentHelpers
 
@@ -12,19 +10,20 @@ describe 'listing activation-keys' do
 
   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" => []
-  }
+  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 activation-keys" do
@@ -49,7 +48,7 @@ ID | NAME | HOST LIMIT | LIFECYCLE ENVIRONMENT | CONTENT VIEW
   it "lists the activation-keys 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).at_least_once
+    expect_lifecycle_environment_search(org_id, 'test', lifecycle_env_id)
 
     ex = api_expects(:activation_keys, :index, 'lifecycle activation-key list') do |par|
       par['organization_id'] == org_id && par['page'] == 1 &&

data/test/functional/activaton_key/product_content_test.rb

@@ -0,0 +1,23 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+
+describe 'listing available product content' do
+  let(:activation_key_id) { 1 }
+  let(:empty_response_table) do
+    <<eostring
+---|------|------|-----|---------|-------|----------|---------
+ID | NAME | TYPE | URL | GPG KEY | LABEL | ENABLED? | OVERRIDE
+---|------|------|-----|---------|-------|----------|---------
+eostring
+  end
+
+  it "lists content available for an activation key" do
+    ex = api_expects(:activation_keys, :product_content) do |p|
+      p['id'] = activation_key_id
+    end
+    ex.returns(index_response([]))
+    assert_cmd(
+      success_result(empty_response_table),
+      run_cmd(%w(activation-key product-content --id 1))
+    )
+  end
+end

data/test/functional/activaton_key/remove_host_collection_test.rb

@@ -0,0 +1,62 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/associating_commands'
+
+module HammerCLIKatello
+  module AssociatingCommands
+    module HostCollection
+      describe RemoveHostCollectionCommand do
+        it 'allows minimal options' do
+          ex = api_expects(:activation_keys, :show) do |p|
+            p[:id] == '1'
+          end
+          ex.returns('id' => 1, host_collections: [{'id' => 3}])
+
+          api_expects(:activation_keys, :update) do |p|
+            p['id'] == '1' && p['organization_id'] == 1 && p['host_collection_ids'] == []
+          end
+
+          run_cmd(%w(activation-key remove-host-collection --organization-id 1
+                     --host-collection-id 3 --id 1))
+        end
+
+        it 'allows resolving host collection name' do
+          ex = api_expects(:host_collections, :index) do |p|
+            p['name'] == 'hc3' && p['organization_id'] == 1
+          end
+          ex.returns(index_response([{id: 1}]))
+
+          ex = api_expects(:activation_keys, :show) do |p|
+            p[:id] == '1'
+          end
+          ex.returns(id: 1, host_collections: [{'id' => 3}])
+
+          api_expects(:activation_keys, :update) do |p|
+            p['id'] == '1' && p['organization_id'] == 1 && p['host_collection_ids'] == []
+          end
+
+          run_cmd(%w(activation-key remove-host-collection --organization-id 1
+                     --host-collection hc3 --id 1))
+        end
+
+        it 'allows resolving activation key name' do
+          ex = api_expects(:activation_keys, :index) do |p|
+            p['name'] == 'ak1' && p['organization_id'] == 1
+          end
+          ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+          ex = api_expects(:activation_keys, :show) do |p|
+            p[:id] == 1
+          end
+          ex.returns(id: 1, host_collections: [{'id' => 3}])
+
+          api_expects(:activation_keys, :update) do |p|
+            p['id'] == 1 && p['organization_id'] == 1 && p['host_collection_ids'] == []
+          end
+
+          run_cmd(%w(activation-key remove-host-collection --organization-id 1
+                     --host-collection-id 3 --name ak1))
+        end
+      end
+    end
+  end
+end

data/test/functional/activaton_key/subscriptions_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/activation_key'
+
+module HammerCLIKatello
+  describe ActivationKeyCommand::SubscriptionsCommand do
+    it 'requires organization options' do
+      expected_error = "Could not find organization"
+      result = run_cmd(%w(activation-key subscriptions --id 1))
+      assert_equal(result.err[/#{expected_error}/], expected_error)
+    end
+
+    it 'allows organization name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:subscriptions, :index) { |par| par['organization_id'].to_i == 1 }
+
+      run_cmd(%w(activation-key subscriptions --id 1 --organization org1))
+    end
+
+    it 'allows organization label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .returns(index_response([{'id' => 1}]))
+
+      api_expects(:subscriptions, :index) { |par| par['organization_id'].to_i == 1 }
+
+      run_cmd(%w(activation-key subscriptions --id 1 --organization-label org1))
+    end
+
+    it 'lists subscriptions available for the activation key' do
+      api_expects(:subscriptions, :index) do |p|
+        p['available_for'] == 'activation_key'
+      end
+      run_cmd(%w(activation-key subscriptions --organization-id 1 --id 1
+                 --available-for activation_key))
+    end
+
+    it 'lists subscriptions used by the activation key' do
+      api_expects(:subscriptions, :index) { |p| p['activation_key_id'] == 1 }
+      run_cmd(%w(activation-key subscriptions --organization-id 1 --id 1))
+    end
+
+    it 'allows activation_key_name' do
+      api_expects(:activation_keys, :index) { |p| p['name'] == 'ak1' }
+        .returns(index_response([{'id' => 1}]))
+      api_expects(:subscriptions, :index) { |p| p['activation_key_id'] == 1 }
+      run_cmd(%w(activation-key subscriptions --organization-id 1 --name ak1))
+    end
+
+    it 'allows subscriptions to be added to an activation key' do
+      api_expects(:activation_keys, :add_subscriptions) do |p|
+        p['id'] == '1' && p['subscription_id'] == '3'
+      end
+      run_cmd(%w(activation-key add-subscription --id 1 --subscription-id 3))
+    end
+  end
+end

data/test/functional/activaton_key/update_test.rb

@@ -0,0 +1,26 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/activation_key'
+
+module HammerCLIKatello
+  describe ActivationKeyCommand do
+    describe UpdateCommand do
+      it 'allows promoting a content view' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\""
+        end
+        ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+        ex = api_expects(:activation_keys, :index) do |p|
+          p['name'] == 'ak' && p['organization_id'] == 1
+        end
+        ex.returns(index_response([{'id' => 6}]))
+
+        api_expects(:activation_keys, :update) do |p|
+          p['id'] == 6 && p['organization_id'] == 1
+        end
+
+        run_cmd(%w(activation-key update --organization org1 --name ak --auto-attach false))
+      end
+    end
+  end
+end

data/test/functional/capsule/content/cancel_synchronization_test.rb

@@ -8,7 +8,7 @@ describe 'capsule content cancel-synchronization' do
     @cmd = ['capsule', 'content', 'cancel-synchronization']
   end
 
-  MESSAGE = "There's no running synchronization for this capsule."
+  MESSAGE = "There's no running synchronization for this capsule.".freeze
 
   it "triggers the cancel" do
     expected_result = success_result("#{MESSAGE}\n")

data/test/functional/capsule/content/capsule_content_helpers.rb

@@ -1,10 +1,8 @@
 module CapsuleContentHelpers
-
   def expect_capsule_search(name, id)
     ex = api_expects(:smart_proxies, :index, 'Find the proxy') do |par|
       par[:search] == "name = \"#{name}\""
     end
     ex.returns(index_response([{'id' => id}]))
   end
-
 end

data/test/functional/content_view/add_content_view_version_test.rb

@@ -0,0 +1,34 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::AddContentViewVersionCommand do
+    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])
+      api_expects(:content_views, :update) do |p|
+        p['id'] == '1' && p['component_ids'] == %w(1 3 6)
+      end
+      run_cmd(%w(content-view add-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])
+      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 6)
+      end
+      run_cmd(%w(content-view add-version --id 1 --content-view-version-content-view-id 3
+                 --content-view-version 2.1))
+    end
+  end
+end

data/test/functional/content_view/add_repository_test.rb

@@ -0,0 +1,20 @@
+require_relative '../test_helper'
+
+module HammerCLIKatello
+  describe ContentView do
+    describe 'AddRepositoryCommand' do
+      it 'allows adding a repository by ID' do
+        ex = api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\""
+        end
+        ex.returns(index_response([{'id' => 1}]))
+
+        api_expects(:content_views, :index) do |p|
+          p['name'] == 'cv' && p['organization_id'] == 1
+        end
+
+        run_cmd(%w(content-view add-repository --organization org1 --name cv --repository-id 1))
+      end
+    end
+  end
+end

data/test/functional/content_view/content_view_helpers.rb

@@ -1,10 +1,8 @@
 module ContentViewHelpers
-
   def expect_content_view_search(org_id, name, id)
     ex = api_expects(:content_views, :index, 'Find the content view') do |par|
       par['name'] == name && par['organization_id'] == org_id
     end
     ex.returns(index_response([{'id' => id}]))
   end
-
 end

data/test/functional/content_view/create_test.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), '../test_helper')
+require_relative '../test_helper'
 
 # Workaround for issue #14289
 require 'hammer_cli_katello/content_view_puppet_module'
@@ -46,7 +46,7 @@ describe 'content-view create' do
 
     api_repositories = api_expects(:repositories, :index,
                                    'Find repositories belonging to product') do |par|
-      par['product_id'] == product['id'] && par['organization_id'] == org_id
+      par['product_id'] == product['id']
     end
     api_repositories.returns(repositories)
 
@@ -74,7 +74,7 @@ describe 'content-view create' do
 
     api_repositories = api_expects(:repositories, :index,
                                    'Find repositories belonging to product') do |par|
-      par['product_id'] == product['id'] && par['organization_id'] == org_id
+      par['product_id'] == product['id']
     end
     api_repositories.returns(repositories)
 

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

@@ -0,0 +1,63 @@
+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 = ["--organization-id=#{org_id}", "--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})
+
+    ex = api_expects(:content_view_filters, :create, 'Create content-view filter') do |par|
+      par['name'] == filter_name && par['repository_ids'] == repo_ids
+      true
+    end
+
+    ex.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(',')})
+
+    expect_repositories_search(org_id.to_s, repo_names, repo_ids)
+
+    api_expects(:content_view_filters, :create, "Create content-view filter") do |par|
+      par['name'] == filter_name && par['repository_ids'] == repo_ids &&
+        par['type'] == 'rpm'
+    end
+
+    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 organization 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[/--organization-id, --organization, --organization-label is required/])
+  end
+end

data/test/functional/content_view/list_test.rb

@@ -12,19 +12,20 @@ describe 'listing content-views' do
 
   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" => []
-  }
+  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
@@ -36,9 +37,10 @@ describe 'listing content-views' do
     end
 
     ex.returns(empty_response)
-    expected_result = success_result("----------------|------|-------|-----------|---------------
-CONTENT VIEW ID | NAME | LABEL | COMPOSITE | REPOSITORY IDS
-----------------|------|-------|-----------|---------------
+    expected_result = success_result(
+      "----------------|------|-------|-----------|----------------|---------------
+CONTENT VIEW ID | NAME | LABEL | COMPOSITE | LAST PUBLISHED | REPOSITORY IDS
+----------------|------|-------|-----------|----------------|---------------
 ")
 
     result = run_cmd(@cmd + params)
@@ -48,7 +50,7 @@ CONTENT VIEW ID | NAME | LABEL | COMPOSITE | REPOSITORY IDS
   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).at_least_once
+    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 &&
@@ -56,9 +58,10 @@ CONTENT VIEW ID | NAME | LABEL | COMPOSITE | REPOSITORY IDS
     end
 
     ex.returns(empty_response)
-    expected_result = success_result("----------------|------|-------|-----------|---------------
-CONTENT VIEW ID | NAME | LABEL | COMPOSITE | REPOSITORY IDS
-----------------|------|-------|-----------|---------------
+    expected_result = success_result(
+      "----------------|------|-------|-----------|----------------|---------------
+CONTENT VIEW ID | NAME | LABEL | COMPOSITE | LAST PUBLISHED | REPOSITORY IDS
+----------------|------|-------|-----------|----------------|---------------
 ")
 
     result = run_cmd(@cmd + params)

data/test/functional/content_view/publish_test.rb

@@ -0,0 +1,23 @@
+require_relative '../test_helper'
+
+module HammerCLIKatello
+  describe ContentViewPuppetModule do
+    it 'allows adding a puppet module' 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
+      end
+
+      run_cmd(%w(content-view publish --organization org1 --name cv --async))
+    end
+  end
+end

data/test/functional/content_view/puppet_module/add_test.rb

@@ -0,0 +1,23 @@
+require_relative '../../test_helper'
+
+module HammerCLIKatello
+  describe ContentViewPuppetModule do
+    it 'allows adding a puppet module' 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_view_puppet_modules, :create) do |p|
+        p['content_view_id'] == 3
+      end
+
+      run_cmd(%w(content-view puppet-module add --organization org1 --content-view cv --id 1))
+    end
+  end
+end

data/test/functional/content_view/remove_content_view_version_test.rb

@@ -0,0 +1,34 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::RemoveContentViewVersionCommand do
+    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-version-content-view-id 3
+                 --content-view-version 2.1))
+    end
+  end
+end