hammer_cli_katello 0.0.26 → 0.1.0

data/test/functional/hostgroup/create_test.rb

@@ -0,0 +1,55 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/hostgroup'
+
+module HammerCLIForeman
+  describe Hostgroup do
+    # These tests are only for the extensions Katello adds to the hostgroup command
+    # See hammer-cli-foreman for the core hostgroup tests
+    describe CreateCommand do
+      it 'allows content source id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['content_source_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-source-id 1))
+      end
+
+      it 'allows content view id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-view-id 1))
+      end
+
+      it 'allows content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p[:search] = "name = \"cv1\"" && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['content_view_id'] == 1 && p['hostgroup']['name'] == 'hg1'
+        end
+        run_cmd(%w(hostgroup create --name hg1 --content-view cv1 --query-organization-id 1))
+      end
+
+      it 'allows lifecycle environment id' do
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['lifecycle_environment_id'] == 1 &&
+            p['hostgroup']['organization_ids'] == %w(1 2)
+        end
+        run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment-id 1 --organization-ids 1,2))
+      end
+
+      it 'allows lifecycle environment name' do
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p[:name] = 'le1' && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :create) do |p|
+          p['hostgroup']['name'] == 'hg1' && p['hostgroup']['lifecycle_environment_id'] == 1
+        end
+        run_cmd(%w(hostgroup create --name hg1 --lifecycle-environment le1
+                   --query-organization-id 1 --organization-ids 1,2))
+      end
+    end
+  end
+end

data/test/functional/hostgroup/data/hostgroup.json

@@ -0,0 +1,48 @@
+{
+	"content_source_id":null,
+	"content_source_name":null,
+	"content_view_id":1,
+	"content_view_name":"Default Organization View",
+	"lifecycle_environment_id":1,
+	"lifecycle_environment_name":"Library",
+	"subnet_id":null,
+	"subnet_name":null,
+	"operatingsystem_id":null,
+	"operatingsystem_name":null,
+	"domain_id":null,
+	"domain_name":null,
+	"environment_id":1,
+	"environment_name":"Production Env",
+	"compute_profile_id":null,
+	"compute_profile_name":null,
+	"ancestry":null,
+	"parent_id":null,
+	"parent_name":null,
+	"puppet_proxy_id":null,
+	"puppet_ca_proxy_id":null,
+	"ptable_id":null,
+	"ptable_name":null,
+	"medium_id":null,
+	"medium_name":null,
+	"architecture_id":null,
+	"architecture_name":null,
+	"realm_id":null,
+	"realm_name":null,
+	"created_at":"2016-08-09 04:18:32 UTC",
+	"updated_at":"2016-08-09 04:18:32 UTC",
+	"id":1,
+	"parameters":[],
+	"template_combinations":[],
+	"puppetclasses":[],
+	"config_groups":[],
+	"all_puppetclasses":[],
+	"locations":[],
+	"organizations":[
+	{
+		"id":1,
+		"name":"Redhat",
+		"title":"Redhat",
+		"description":null
+	}
+	]
+}

data/test/functional/hostgroup/info_test.rb

@@ -0,0 +1,30 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/hostgroup'
+
+module HammerCLIForeman
+  describe Hostgroup do
+    describe InfoCommand do
+      before do
+        @cmd = %w(hostgroup info)
+      end
+
+      let(:hostgroup_id) { 1 }
+
+      it "Shows information about a hostgroup" do
+        params = ["--id=#{hostgroup_id}"]
+
+        ex = api_expects(:hostgroups, :show, "Get info") do |par|
+          par["id"] == hostgroup_id.to_s
+        end
+        json_file = File.join(File.dirname(__FILE__), 'data', 'hostgroup.json')
+        ex.returns(JSON.parse(File.read(json_file)))
+
+        result = run_cmd(@cmd + params)
+        expected_fields = [['Lifecycle Environment', 'Library'],
+                           ['Content View', 'Default Organization View']]
+        expected_results = expected_fields.map { |field| success_result(FieldMatcher.new(*field)) }
+        expected_results.each { |expected|  assert_cmd(expected, result) }
+      end
+    end
+  end
+end

data/test/functional/hostgroup/update_test.rb

@@ -0,0 +1,55 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/hostgroup'
+
+module HammerCLIForeman
+  describe Hostgroup do
+    # These tests are only for the extensions Katello adds to the hostgroup command
+    # See hammer-cli-foreman for the core hostgroup tests
+    describe UpdateCommand do
+      it 'allows content source id' do
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['content_source_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --content-source-id 1))
+      end
+
+      it 'allows content view id' do
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --content-view-id 1))
+      end
+
+      it 'allows content view name' do
+        ex = api_expects(:content_views, :index) do |p|
+          p[:search] = "name = \"cv1\"" && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['content_view_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --content-view cv1 --query-organization-id 1))
+      end
+
+      it 'allows lifecycle environment id' do
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['lifecycle_environment_id'] == 1 &&
+            p['hostgroup']['organization_ids'] == %w(1 2)
+        end
+        run_cmd(%w(hostgroup update --id 1 --lifecycle-environment-id 1 --organization-ids 1,2))
+      end
+
+      it 'allows lifecycle environment name' do
+        ex = api_expects(:lifecycle_environments, :index) do |p|
+          p[:name] = 'le1' && p['organization_id'] == '1'
+        end
+        ex.returns(index_response([{'id' => 1}]))
+        api_expects(:hostgroups, :update) do |p|
+          p['id'] == '1' && p['hostgroup']['lifecycle_environment_id'] == 1
+        end
+        run_cmd(%w(hostgroup update --id 1 --lifecycle-environment le1
+                   --query-organization-id 1 --organization-ids 1,2))
+      end
+    end
+  end
+end

data/test/functional/lifecycle_environment/lifecycle_environment_helpers.rb

@@ -1,5 +1,4 @@
 module LifecycleEnvironmentHelpers
-
   def expect_lifecycle_environment_search(org_id, name, id)
     ex = api_expects(:lifecycle_environments, :index, 'Find the lifecycle environment') do |par|
       par['name'] == name && par['organization_id'] == org_id
@@ -13,5 +12,4 @@ module LifecycleEnvironmentHelpers
     end
     ex.returns(index_response(results))
   end
-
 end

data/test/functional/organization/organization_helpers.rb

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

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

@@ -5,4 +5,18 @@ module ProductHelpers
     end
     ex.returns(index_response([{'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_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/info_test.rb

@@ -24,7 +24,7 @@ describe "get repository info" do
     ex.returns({})
 
     result = run_cmd(@cmd + params)
-    assert(result.exit_code, 0)
+    assert_equal(result.exit_code, 0)
   end
 
   it "Shows information about a repository with organization-id and product name" do
@@ -32,7 +32,7 @@ describe "get repository info" do
 
     expect_product_search(org_id, 'test_product', product_id)
 
-    expect_repository_search(org_id, product_id, 'test_repo', repo_id)
+    expect_repository_search(product_id, 'test_repo', repo_id)
 
     ex2 = api_expects(:repositories, :show, "Get info") do |par|
       par["id"] == repo_id
@@ -41,6 +41,6 @@ describe "get repository info" do
     ex2.returns({})
 
     result = run_cmd(@cmd + params)
-    assert(result.exit_code, 0)
+    assert_equal(result.exit_code, 0)
   end
 end

data/test/functional/repository/list_test.rb

@@ -12,19 +12,20 @@ describe 'listing repositories' 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 repositories" do

data/test/functional/repository/remove_content_test.rb

@@ -0,0 +1,18 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/repository'
+
+module HammerCLIKatello
+  describe Repository::RemoveContentCommand do
+    it 'allows minimal parameters' do
+      api_expects(:repositories, :remove_content) { |p| p['id'] == '1' && p['ids'] == %w(1) }
+      run_cmd(%w(repository remove-content --ids 1 --id 1))
+    end
+
+    it 'resolves repository id' do
+      api_expects(:repositories, :index) { |p| p['name'] == 'repo1' }
+        .returns(index_response([{'id' => '1'}]))
+      api_expects(:repositories, :remove_content) { |p| p['id'] == '1' && p['ids'] == %w(1) }
+      run_cmd(%w(repository remove-content --ids 1 --name repo1))
+    end
+  end
+end

data/test/functional/repository/repository_helpers.rb

@@ -1,9 +1,15 @@
 module RepositoryHelpers
-  def expect_repository_search(org_id, product_id, name, id)
+  def expect_repository_search(product_id, name, id)
     ex = api_expects(:repositories, :index, 'Find a repository') do |par|
-      par['name'] == name && par['organization_id'] == org_id &&
-        par['product_id'] == product_id
+      par['name'] == name && par['product_id'] == product_id
     end
     ex.returns(index_response([{'id' => id}]))
   end
+
+  def expect_repositories_search(org_id, names, ids)
+    ex = api_expects(:repositories, :index, 'Find repositories') do |par|
+      par['names'] == names && par['organization_id'] == org_id
+    end
+    ex.returns(index_response(ids.zip(names).map { |id, name| { 'id' => id, 'name' => name } }))
+  end
 end