hammer_cli_katello 0.1.3 → 0.2.0

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 | 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 | 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 | 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

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

@@ -0,0 +1,93 @@
+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 HammerCLIForeman
+  describe UpdateCommand do
+    include ContentViewHelpers
+    include OrganizationHelpers
+
+    before do
+      @cmd = %w(content-view filter update)
+    end
+
+    it 'accepts filter id' do
+      params = ['--id=1', '--new-name=valis']
+
+      api_expects(:content_view_filters, :update) do |par|
+        par['id'] == '1' && par['name'] == 'valis'
+      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', '--new-name=ubik']
+
+      expect_organization_search('pkd', 1)
+      expect_content_view_search(1, 'darkly', 1)
+      expect_content_view_search(1, 'darkly', 1) # redmine #15930
+
+      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, :update) do |par|
+        par['id'] == '1' && par['name'] == 'ubik'
+      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', '--new-name=ubik']
+
+      expect_organization_search('pkd', 1, field: 'label')
+      expect_content_view_search(1, 'darkly', 1)
+      expect_content_view_search(1, 'darkly', 1) # redmine #15930
+
+      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, :update) do |par|
+        par['id'] == '1' && par['name'] == 'ubik'
+      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', '--new-name=ubik']
+
+      expect_content_view_search('1', 'darkly', 1)
+      expect_content_view_search('1', 'darkly', 1) # redmine #15930
+
+      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, :update) do |par|
+        par['id'] == '1' && par['name'] == 'ubik'
+      end
+
+      run_cmd(@cmd + params)
+    end
+
+    it 'requires content view name or id if name is supplied' do
+      params = ["--name=high-castle"]
+      expected_result = usage_error_result(
+        @cmd,
+        'At least one of options --content-view-id, --content-view is required',
+        'Could not update the filter'
+      )
+      api_expects_no_call
+      result = run_cmd(@cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

data/test/functional/filter_rule/create_test.rb

@@ -16,84 +16,5 @@ module HammerCLIKatello
       end
       run_cmd(%w(content-view filter rule create --content-view-filter-id 1 --names rpm1,rpm2))
     end
-
-    it 'allows name resolution of filter with content-view-id' do
-      ex = api_expects(:content_view_filters, :index) do |p|
-        p['name'] == 'cvf1' && p['content_view_id'] == 3
-      end
-      ex.returns(index_response([{'id' => 1}]))
-
-      api_expects(:content_view_filter_rules, :create) do |p|
-        p['content_view_filter_id'] == 1 && p['name'] == %w(rule9)
-      end
-      run_cmd(%w(content-view filter rule create --content-view-filter cvf1 --content-view-id 3
-                 --name rule9))
-    end
-
-    describe 'organization' do
-      it 'ID can be specified to resolve content view name' do
-        ex = api_expects(:content_views, :index) do |p|
-          p['name'] == 'cv3' && p['organization_id'] == '6'
-        end
-        ex.returns(index_response([{'id' => 3}]))
-
-        ex = api_expects(:content_view_filters, :index) do |p|
-          p['name'] == 'cvf1' && p['content_view_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        api_expects(:content_view_filter_rules, :create) do |p|
-          p['content_view_filter_id'] == 1 && p['name'] == %w(rule9)
-        end
-        run_cmd(%w(content-view filter rule create --content-view-filter cvf1 --organization-id 6
-                   --content-view cv3 --name rule9))
-      end
-
-      it 'name can be specified to resolve content view name' do
-        ex = api_expects(:organizations, :index) do |p|
-          p[:search] == "name = \"org6\""
-        end
-        ex.returns(index_response([{'id' => 6}]))
-
-        ex = api_expects(:content_views, :index) do |p|
-          p['name'] == 'cv3' && p['organization_id'] == 6
-        end
-        ex.returns(index_response([{'id' => 3}]))
-
-        ex = api_expects(:content_view_filters, :index) do |p|
-          p['name'] == 'cvf1' && p['content_view_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        api_expects(:content_view_filter_rules, :create) do |p|
-          p['content_view_filter_id'] == 1 && p['name'] == %w(rule9)
-        end
-        run_cmd(%w(content-view filter rule create --content-view-filter cvf1 --organization org6
-                   --content-view cv3 --name rule9))
-      end
-
-      it 'label can be specified to resolve content view name' do
-        ex = api_expects(:organizations, :index) do |p|
-          p[:search] == "label = \"org6\""
-        end
-        ex.returns(index_response([{'id' => 6}]))
-
-        ex = api_expects(:content_views, :index) do |p|
-          p['name'] == 'cv3' && p['organization_id'] == 6
-        end
-        ex.returns(index_response([{'id' => 3}]))
-
-        ex = api_expects(:content_view_filters, :index) do |p|
-          p['name'] == 'cvf1' && p['content_view_id'] == 3
-        end
-        ex.returns(index_response([{'id' => 1}]))
-
-        api_expects(:content_view_filter_rules, :create) do |p|
-          p['content_view_filter_id'] == 1 && p['name'] == %w(rule9)
-        end
-        run_cmd(%w(content-view filter rule create --content-view-filter cvf1 --organization-label
-                   org6 --content-view cv3 --name rule9))
-      end
-    end
   end
 end

data/test/functional/host/extensions/data/host.json

@@ -92,7 +92,9 @@
       "content_view_version_id":1,
       "content_view_default?":true,
       "lifecycle_environment_library?":true,
-      "katello_agent_installed":true
+      "katello_agent_installed":true,
+      "applicable_package_count": 5,
+      "upgradable_package_count": 4
    },
    "subscription_facet_attributes":{
       "id":2,
@@ -307,4 +309,4 @@
       "ipmi_boot":true,
       "puppetrun_hosts":true
    }
-}
+}

data/test/functional/host/extensions/info_test.rb

@@ -20,7 +20,9 @@ describe 'host info' do
                        ['Content View', 'Default Organization View'],
                        ['Release Version', '7Server'],
                        ['Bug Fix', '0'],
-                       ['Name', 'my host collection']]
+                       ['Name', 'my host collection'],
+                       ['Applicable Packages', '5'],
+                       ['Upgradable Packages', '4']]
     expected_results = expected_fields.map { |field| success_result(FieldMatcher.new(*field)) }
     expected_results.each { |expected|  assert_cmd(expected, result) }
   end

data/test/functional/lifecycle_environment/create_test.rb

@@ -0,0 +1,14 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/lifecycle_environment'
+
+module HammerCLIKatello
+  describe LifecycleEnvironmentCommand::CreateCommand do
+    it 'allows minimal options' do
+      api_expects(:lifecycle_environments, :create) do |p|
+        p['name'] == 'le1' && p['prior_id'] == '3' && p['organization_id'] == 1
+      end
+
+      run_cmd(%w(lifecycle-environment create --name le1 --prior-id 3 --organization-id 1))
+    end
+  end
+end

data/test/functional/lifecycle_environment/list_test.rb

@@ -0,0 +1,38 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/lifecycle_environment'
+
+module HammerCLIKatello
+  describe LifecycleEnvironmentCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:lifecycle_environments, :index)
+
+      run_cmd(%w(lifecycle-environment list))
+    end
+
+    describe 'allows organization' do
+      it 'id' do
+        api_expects(:lifecycle_environments, :index) { |p| p['organization_id'] == 1 }
+
+        run_cmd(%w(lifecycle-environment list --organization-id 1))
+      end
+
+      it 'name' do
+        api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+          .at_least_once.returns(index_response([{'id' => 1}]))
+
+        api_expects(:lifecycle_environments, :index) { |p| p['organization_id'] == 1 }
+
+        run_cmd(%w(lifecycle-environment list --organization org1))
+      end
+
+      it 'label' do
+        api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+          .at_least_once.returns(index_response([{'id' => 1}]))
+
+        api_expects(:lifecycle_environments, :index) { |p| p['organization_id'] == 1 }
+
+        run_cmd(%w(lifecycle-environment list --organization-label org1))
+      end
+    end
+  end
+end

data/test/functional/lifecycle_environment/update_test.rb

@@ -0,0 +1,14 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/lifecycle_environment'
+
+module HammerCLIKatello
+  describe LifecycleEnvironmentCommand::UpdateCommand do
+    it 'allows minimal options' do
+      api_expects(:lifecycle_environments, :update) do |p|
+        p['new_name'] == 'le3' && p['id'] == 3
+      end
+
+      run_cmd(%w(lifecycle-environment update --id 3 --new-name le3))
+    end
+  end
+end

data/test/functional/organization/organization_helpers.rb

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

data/test/functional/package/list_test.rb

@@ -0,0 +1,48 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/package'
+
+module HammerCLIKatello
+  describe PackageCommand::ListCommand do
+    it 'allows minimal options' do
+      api_expects(:packages, :index)
+
+      run_cmd(%w(package list))
+    end
+
+    describe 'organization options' do
+      it 'can be provided by organization ID' do
+        api_expects(:packages, :index) do |params|
+          params['organization_id'] == 1
+        end
+
+        run_cmd(%w(package list --organization-id 1))
+      end
+
+      it 'can be provided by organization name' do
+        ex = api_expects(:organizations, :index) do |params|
+          params[:search] == "name = \"org1\""
+        end
+        ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+        api_expects(:packages, :index) do |params|
+          params['organization_id'] == 1
+        end
+
+        run_cmd(%w(package list --organization org1))
+      end
+
+      it 'can be provided by organization label' do
+        ex = api_expects(:organizations, :index) do |params|
+          params[:search] == "label = \"org1\""
+        end
+        ex.at_least_once.returns(index_response([{'id' => 1}]))
+
+        api_expects(:packages, :index) do |params|
+          params['organization_id'] == 1
+        end
+
+        run_cmd(%w(package list --organization-label org1))
+      end
+    end
+  end
+end

data/test/functional/repository/delete_test.rb

@@ -0,0 +1,101 @@
+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
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 3 && p['name'] == 'repo1'
+        end
+        ex.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
+        ex = api_expects(:products, :index) do |p|
+          p['organization_id'] == '6' && p['name'] == 'product3'
+        end
+        ex.returns(index_response([{'id' => 3}]))
+
+        ex = api_expects(:repositories, :index) do |p|
+          p['product_id'] == 3 && p['name'] == 'repo1' && p['organization_id'] == '6'
+        end
+        ex.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
+          ex = api_expects(:repositories, :index) do |p|
+            p['product_id'] == 3 && p['name'] == 'repo1' && p['organization_id'] == '6'
+          end
+          ex.returns(index_response([{'id' => 1}]))
+
+          api_expects(:repositories, :destroy) { |p| p['id'] == 1 }
+
+          run_cmd(%w(repository delete --name repo1 --product-id 3 --organization-id 6))
+        end
+
+        it 'can be specified by organization name' do
+          ex = api_expects(:organizations, :index) do |p|
+            p[:search] == "name = \"org6\""
+          end
+          ex.returns(index_response([{'id' => 6}]))
+
+          ex = api_expects(:products, :index) do |p|
+            p['organization_id'] == 6 && p['name'] == 'product3'
+          end
+          ex.returns(index_response([{'id' => 3}]))
+
+          ex = api_expects(:repositories, :index) do |p|
+            p['product_id'] == 3 && p['name'] == 'repo1'
+          end
+          ex.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
+          ex = api_expects(:organizations, :index) do |p|
+            p[:search] == "label = \"org6\""
+          end
+          ex.returns(index_response([{'id' => 6}]))
+
+          ex = api_expects(:products, :index) do |p|
+            p['organization_id'] == 6 && p['name'] == 'product3'
+          end
+          ex.returns(index_response([{'id' => 3}]))
+
+          ex = api_expects(:repositories, :index) do |p|
+            p['product_id'] == 3 && p['name'] == 'repo1'
+          end
+          ex.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