hammer_cli_katello 0.3.0 → 0.10.0

data/test/functional/activation_key/content_override_test.rb

@@ -0,0 +1,91 @@
+require File.join(File.dirname(__FILE__), '../../test_helper')
+require 'hammer_cli_katello/activation_key'
+
+describe 'activation-key content-override' do
+  before do
+    @cmd = %w(activation-key content-override)
+  end
+  it "attaches a content label" do
+    label = "foo"
+    value = 'default'
+    id = '20'
+    params = ["--id=#{id}", "--content-label=#{label}", "--value=#{value}"]
+    ex = api_expects(:activation_keys, :content_override) do |par|
+      par['id'] == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['value'] == value &&
+        par['content_overrides'][0]['name'] == "enabled"
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "attaches a content label with name" do
+    label = "foo"
+    value = '1'
+    id = '20'
+    name = 'protected'
+    params = ["--id=#{id}", "--content-label=#{label}", "--value=#{value}", "--name=#{name}"]
+    ex = api_expects(:activation_keys, :content_override) do |par|
+      par['id'] == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['value'] == value &&
+        par['content_overrides'][0]['name'] == name
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "removes override" do
+    label = "foo"
+    id = '20'
+    params = ["--id=#{id}", "--content-label=#{label}", "--remove"]
+    ex = api_expects(:activation_keys, :content_override) do |par|
+      par['id'] == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['remove'] == true &&
+        par['content_overrides'][0]['name'] == "enabled"
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "removes override with name" do
+    label = "foo"
+    id = '20'
+    name = 'protected'
+    params = ["--id=#{id}", "--content-label=#{label}", "--name=#{name}", "--remove"]
+    ex = api_expects(:activation_keys, :content_override) do |par|
+      par['id'] == id && par["content_overrides"][0]['content_label'] == label &&
+        par['content_overrides'][0]['remove'] == true &&
+        par['content_overrides'][0]['name'] == name
+    end
+    ex.returns({})
+
+    expected_result = success_result("Updated content override\n")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "validation fails on no override value or remove" do
+    api_expects_no_call
+    label = "foo"
+    id = '20'
+    name = 'protected'
+    params = ["--id=#{id}", "--content-label=#{label}", "--name=#{name}"]
+    result = run_cmd(@cmd + params)
+
+    assert(result.err[/At least one of options --remove, --value is required/],
+               "Remove or Value must be provided")
+  end
+end

data/test/functional/content_view/add_content_view_version_test.rb

@@ -1,8 +1,13 @@
 require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
 require 'hammer_cli_katello/content_view'
 
 module HammerCLIKatello
   describe ContentView::AddContentViewVersionCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
     it 'allows minimal options' do
       ex = api_expects(:content_views, :show) do |p|
         p[:id] == '1'
@@ -16,7 +21,7 @@ module HammerCLIKatello
 
     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'
+        p['content_view_id'] == 3 && p['version'] == '2.1'
       end
       ex.returns(index_response([{'id' => 6}]))
       ex.returns('id' => 1, 'component_ids' => [1, 3])
@@ -27,8 +32,57 @@ module HammerCLIKatello
       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))
+      run_cmd(%w(content-view add-version --id 1 --content-view-id 3 --content-view-version 2.1))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view add-version --name cv1))
+        assert(result.err[/--organization-id, --organization, --organization-label is required/],
+               "Organization option requirements are validated")
+      end
+
+      it 'allows organization id' do
+        expect_content_view_search('1', 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization-id 1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization org1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2 3)
+        end
+        run_cmd(%w(content-view add-version --name cv2 --organization-label org1
+                   --content-view-version-id 3))
+      end
     end
   end
 end

data/test/functional/content_view/add_repository_test.rb

@@ -1,4 +1,5 @@
 require_relative '../test_helper'
+require 'hammer_cli_katello/content_view'
 
 module HammerCLIKatello
   describe ContentView do

data/test/functional/content_view/copy_test.rb

@@ -0,0 +1,53 @@
+require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::CopyCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
+    it 'allows minimal options' do
+      api_expects(:content_views, :copy) do |p|
+        p['id'] == '2' && p['name'] == 'cv2-dup'
+      end
+      run_cmd(%w(content-view copy --id 2 --new-name cv2-dup))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view copy --new-name cv2-dup --name cv2))
+        assert(result.err[/--organization-id, --organization, --organization-label is required/],
+               "Organization option requirements are validated")
+      end
+
+      it 'allows organization id' do
+        expect_content_view_search('1', 'cv2', 2)
+        api_expects(:content_views, :copy) do |p|
+          p['id'] == 2 && p['name'] == 'cv2-dup'
+        end
+        run_cmd(%w(content-view copy --name cv2 --new-name cv2-dup --organization-id 1))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        api_expects(:content_views, :copy) do |p|
+          p['id'] == 2 && p['name'] == 'cv2-dup'
+        end
+        run_cmd(%w(content-view copy --name cv2 --new-name cv2-dup --organization org1))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        api_expects(:content_views, :copy) do |p|
+          p['id'] == 2 && p['name'] == 'cv2-dup'
+        end
+        run_cmd(%w(content-view copy --name cv2 --new-name cv2-dup --organization-label org1))
+      end
+    end
+  end
+end

data/test/functional/content_view/delete_test.rb

@@ -0,0 +1,62 @@
+require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require 'hammer_cli_katello/content_view'
+
+module HammerCLIKatello
+  describe ContentView::DeleteCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+    include ForemanTaskHelpers
+
+    it 'allows minimal options' do
+      ex = api_expects(:content_views, :destroy) do |p|
+        p['id'] == 2
+      end
+      ex.returns(id: '8')
+      expect_foreman_task('8')
+      run_cmd(%w(content-view delete --id 2))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view delete --name cv2))
+        assert(result.err[/--organization-id, --organization, --organization-label is required/],
+               "Organization option requirements are validated")
+      end
+
+      it 'allows organization id' do
+        expect_content_view_search('1', 'cv2', 2)
+        ex = api_expects(:content_views, :destroy) do |p|
+          p['id'] == 2
+        end
+        ex.returns(id: '8')
+        expect_foreman_task('8')
+        run_cmd(%w(content-view delete --name cv2 --organization-id 1))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :destroy) do |p|
+          p['id'] == 2
+        end
+        ex.returns(id: '8')
+        expect_foreman_task('8')
+        run_cmd(%w(content-view delete --name cv2 --organization org1))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :destroy) do |p|
+          p['id'] == 2
+        end
+        ex.returns(id: '8')
+        expect_foreman_task('8')
+        run_cmd(%w(content-view delete --name cv2 --organization-label org1))
+      end
+    end
+  end
+end

data/test/functional/content_view/publish_test.rb

@@ -1,4 +1,5 @@
 require_relative '../test_helper'
+require 'hammer_cli_katello/content_view_puppet_module'
 
 module HammerCLIKatello
   describe ContentViewPuppetModule do

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

@@ -1,4 +1,5 @@
 require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_puppet_module'
 
 module HammerCLIKatello
   describe ContentViewPuppetModule do

data/test/functional/content_view/purge_test.rb

@@ -0,0 +1,72 @@
+require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
+require 'hammer_cli_katello/content_view'
+require 'hammer_cli_katello/content_view_purge'
+
+module HammerCLIKatello
+  describe ContentViewPurgeCommand do
+    include ForemanTaskHelpers
+
+    let(:versions) do
+      {'versions' => [
+        {"id" => '2', "version" => "1.0", "published" => "2017-03-09 19:48:25 UTC",
+         "environment_ids" => []},
+        {"id" => '4', "version" => "2.2", "published" => "2017-03-09 19:50:29 UTC",
+         "environment_ids" => [1]},
+        {"id" => '6', "version" => "2.10", "published" => "2017-03-09 19:50:29 UTC",
+         "environment_ids" => []}
+      ]}
+    end
+
+    it 'fails gracefully if count <= 0' do
+      api_expects_no_call
+
+      r = run_cmd(%w(content-view purge --id 2 --count -1))
+      assert(r.err.include?('Invalid value for --count option'), 'Incorrect error message')
+    end
+
+    it 'fails gracefully when there are no versions to delete' do
+      ex = api_expects(:content_views, :show) do |p|
+        p['id'] == '2'
+      end
+      ex.returns(versions)
+
+      r = run_cmd(%w(content-view purge --id 2 --count 3))
+      assert(r.err.include?('No versions to delete.'), 'Incorrect error message')
+    end
+
+    it 'only deletes versions not associated with an environment' do
+      ex = api_expects(:content_views, :show) do |p|
+        p['id'] == '2'
+      end
+      ex.returns(versions)
+
+      %w(2 6).each do |id|
+        ex = api_expects(:content_view_versions, :destroy) do |p|
+          p['id'] == id
+        end
+        ex.returns('id' => '3', 'state' => 'stopped')
+        expect_foreman_task('3')
+      end
+
+      run_cmd(%w(content-view purge --id 2 --count 0))
+    end
+
+    it 'allows for async purge of versions' do
+      ex = api_expects(:content_views, :show) do |p|
+        p['id'] == '2'
+      end
+      ex.returns(versions)
+
+      %w(2 6).each do |id|
+        ex = api_expects(:content_view_versions, :destroy) do |p|
+          p['id'] == id
+        end
+        ex.returns('id' => '3')
+      end
+
+      run_cmd(%w(content-view purge --id 2 --count 0 --async))
+    end
+  end
+end

data/test/functional/content_view/remove_content_view_version_test.rb

@@ -1,8 +1,13 @@
 require_relative '../test_helper'
+require_relative '../organization/organization_helpers'
+require_relative '../content_view/content_view_helpers'
 require 'hammer_cli_katello/content_view'
 
 module HammerCLIKatello
   describe ContentView::RemoveContentViewVersionCommand do
+    include OrganizationHelpers
+    include ContentViewHelpers
+
     it 'allows minimal options' do
       ex = api_expects(:content_views, :show) do |p|
         p[:id] == '1'
@@ -16,7 +21,7 @@ module HammerCLIKatello
 
     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'
+        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])
@@ -27,8 +32,57 @@ module HammerCLIKatello
       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))
+      run_cmd(%w(content-view remove-version --id 1 --content-view-id 3 --content-view-version 2.1))
+    end
+
+    describe 'OrganizationOptions' do
+      it 'requires organization if content view name is supplied' do
+        api_expects_no_call
+        result = run_cmd(%w(content-view remove-version --name cv1))
+        assert(result.err[/--organization-id, --organization, --organization-label is required/],
+               "Organization option requirements are validated")
+      end
+
+      it 'allows organization id' do
+        expect_content_view_search('1', 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-version --name cv2 --organization-id 1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization name' do
+        expect_organization_search('org1', 1)
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-version --name cv2 --organization org1
+                   --content-view-version-id 3))
+      end
+
+      it 'allows organization label' do
+        expect_organization_search('org1', 1, field: 'label')
+        expect_content_view_search(1, 'cv2', 2)
+        ex = api_expects(:content_views, :show) do |p|
+          p[:id] == 2
+        end
+        ex.returns('id' => 2, 'component_ids' => [1, 2, 3])
+        api_expects(:content_views, :update) do |p|
+          p['id'] == 2 && p['component_ids'] == %w(1 2)
+        end
+        run_cmd(%w(content-view remove-version --name cv2 --organization-label org1
+                   --content-view-version-id 3))
+      end
     end
   end
 end