hammer_cli_katello 1.1.3 → 1.2.0

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

@@ -0,0 +1,177 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+# rubocop:disable Metrics/ModuleLength
+module HammerCLIKatello
+  describe ContentViewComponent::AddComponents do
+    before do
+      @cmd = %w(content-view component add)
+      @composite = OpenStruct.new(:id => 2, :name => "composite")
+      @component_version = OpenStruct.new(:id => 666, :version => "1.0")
+      @component = OpenStruct.new(:id => 6, :name => "component",
+                                  :content_view_version => @component_version)
+      @organization = OpenStruct.new(:id => 1, :name => "great", :label => "label")
+    end
+
+    def setup_org_expectations
+      org_expect = api_expects(:organizations, :index) do |par|
+        par[:search] == "name = \"#{@organization.name}\""
+      end
+      org_expect.at_least_once.returns(index_response([{'id' => @organization.id.to_s}]))
+    end
+
+    def setup_content_view_expectations(content_view)
+      cv_expect = api_expects(:content_views, :index) do |p|
+        p['name'] == content_view.name && p["organization_id"].to_s == @organization.id.to_s
+      end
+      cv_expect.at_least_once.returns(index_response([{'id' => content_view.id}]))
+    end
+
+    describe 'allows adding a component with' do
+      it 'component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--latest"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view-version-id and latest=false' do
+        latest = false
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-version-id=#{@component_version.id}"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_version_id] == @component_version.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+
+    describe 'allows adding a component with composite name and org' do
+      before do
+        setup_org_expectations
+        setup_content_view_expectations(@composite)
+      end
+
+      it 'component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--organization=#{@organization.name}",
+                  "--latest"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view-version-id and latest=false' do
+        latest = false
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--organization=#{@organization.name}",
+                  "--component-content-view-version-id=#{@component_version.id}"]
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_version_id] == @component_version.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+
+    describe 'allows adding a component with component name and org' do
+      it 'component-content-view and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view=#{@component.name}",
+                  "--organization=#{@organization.name}",
+                  "--latest"]
+        setup_org_expectations
+        setup_content_view_expectations(@component)
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+
+    describe 'allows adding a component with component version number org component content_view' do
+      before do
+        cvv_expect = api_expects(:content_view_versions, :index) do |p|
+          p['content_view_id'].to_s == @component.id.to_s &&
+            p["version"] == @component.content_view_version.version
+        end
+        cvv_expect.at_least_once.
+          returns(index_response([{'id' => @component.content_view_version.id}]))
+      end
+
+      it 'component-content-view-id and component version number' do
+        latest = false
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--component-content-view-version=#{@component.content_view_version.version}"]
+
+        # 1) Should fetch the component_content_view_version_id
+        #    using cv specified in --component-content-view-id and version specified in
+        #    --component-content-view-version
+        # 2) Should set the component[:content_view_id] && component[:content_view_version_id]
+        #    correctly
+
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id.to_s &&
+            component[:content_view_version_id] == @component.content_view_version.id
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view and component version number' do
+        latest = false
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-view=#{@component.name}",
+                  "--component-content-view-version=#{@component.content_view_version.version}",
+                  "--organization=#{@organization.name}"]
+        # 1) Should fetch the org using org name specified in --organization
+        # 2) Should fetch composite content view using name in --composite-content-view
+        # 3) Should fetch component content view using name in --component-content-view
+        # 4) Should fetch the component_content_view_version_id
+        #    using cv specified in --component-content-view-id and version specified in
+        #    --component-content-view-version
+        # 5) Should set the component[:content_view_id] && component[:content_view_version_id]
+        #    correctly
+        setup_org_expectations
+        setup_content_view_expectations(@component)
+        setup_content_view_expectations(@composite)
+        api_expects(:content_view_components, :add_components) do |p|
+          component = p['components'].first
+          p['composite_content_view_id'] == @composite.id &&
+            component[:latest] == latest &&
+            component[:content_view_id] == @component.id &&
+            component[:content_view_version_id] == @component.content_view_version.id
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+  end
+end

data/test/functional/content_view/component/list_test.rb

@@ -0,0 +1,55 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+module HammerCLIKatello
+  describe ContentViewComponent::ListCommand do
+    before do
+      @cmd = %w(content-view component list)
+    end
+
+    it 'expects composite_content_view_id as minimal options' do
+      api_expects(:content_view_components, :index, "component list") do |p|
+        p["composite_content_view_id"].to_s == "1"
+      end
+      run_cmd(@cmd + ["--composite-content-view-id=1"])
+    end
+
+    it 'expects composite_content_view and organization_id as other options' do
+      api_expects(:content_views, :index) do |p|
+        p['name'] == "lol" && p["organization_id"].to_s == "1"
+      end
+      run_cmd(@cmd + %w(--composite-content-view=lol --organization-id=1))
+    end
+
+    it 'label' do
+      api_expects(:organizations, :index) { |par| par[:search] == "label = \"org1\"" }
+        .at_least_once.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_components, :index) do |p|
+        p["composite_content_view_id"].to_s == "1"
+      end
+
+      api_expects(:content_views, :index).
+        at_least_once.returns(index_response([{'id' => 1}])) do |p|
+        p['name'] == "lol" && p["organization_id"].to_s == "1"
+      end
+
+      run_cmd(@cmd + %w(--composite-content-view=lol --organization-label=org1))
+    end
+
+    it 'name' do
+      api_expects(:organizations, :index) { |par| par[:search] == "name = \"org1\"" }
+        .at_least_once.returns(index_response([{'id' => 1}]))
+
+      api_expects(:content_view_components, :index) do |p|
+        p["composite_content_view_id"].to_s == "1"
+      end
+
+      api_expects(:content_views, :index).
+        at_least_once.returns(index_response([{'id' => 1}])) do |p|
+        p['name'] == "lol" && p["organization_id"].to_s == "1"
+      end
+
+      run_cmd(@cmd + %w(--composite-content-view=lol --organization=org1))
+    end
+  end
+end

data/test/functional/content_view/component/remove_test.rb

@@ -0,0 +1,107 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+module HammerCLIKatello
+  describe ContentViewComponent::RemoveComponents do
+    before do
+      @cmd = %w(content-view component remove)
+
+      @component = OpenStruct.new(:id => 6, :name => "component")
+      @component2 = OpenStruct.new(:id => 7, :name => "component2")
+
+      @content_view_component = OpenStruct.new(:id => 1444,
+                                               :content_view => @component)
+
+      @content_view_component2 = OpenStruct.new(:id => 1445,
+                                                :content_view => @component2)
+
+      @composite = OpenStruct.new(:id => 2, :name => "composite",
+                                  :content_view_components => [@content_view_component])
+      @organization = OpenStruct.new(:id => 1, :name => "great", :label => "label")
+      @content_view_components = [@content_view_component, @content_view_component2]
+    end
+
+    def setup_org_expectations
+      org_expect = api_expects(:organizations, :index) do |par|
+        par[:search] == "name = \"#{@organization.name}\""
+      end
+      org_expect.at_least_once.returns(index_response([{'id' => @organization.id.to_s}]))
+    end
+
+    def setup_content_view_expectations(content_view)
+      cv_expect = api_expects(:content_views, :index) do |p|
+        p['name'] == content_view.name && p["organization_id"].to_s == @organization.id.to_s
+      end
+      cv_expect.at_least_once.returns(index_response([{"id" => content_view.id}]))
+    end
+
+    def setup_content_view_components_expectations
+      cvc_expect = api_expects(:content_view_components, :index) do |p|
+        p[:composite_content_view_id].to_s == @composite.id.to_s
+      end
+      ret = @content_view_components.map do |cvc|
+        { "id" => cvc.id,
+          "content_view" => {"id" => cvc.content_view.id,
+                             "name" => cvc.content_view.name}}
+      end
+      cvc_expect.at_least_once.returns(index_response(ret))
+    end
+
+    describe 'allows deleting component with' do
+      it 'id and component-ids' do
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-ids=#{@content_view_components.map(&:id).join(',')}"]
+
+        api_expects(:content_view_components, :remove_components) do |p|
+          component_ids = p['component_ids'].map(&:to_s)
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            component_ids.include?(@content_view_component.id.to_s) &&
+            component_ids.include?(@content_view_component2.id.to_s)
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'id and component-content-view-id' do
+        component_cv_ids = @content_view_components.map do |cvc|
+          cvc["content_view"]["id"]
+        end
+
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-ids=#{component_cv_ids.join(',')}"]
+        setup_content_view_components_expectations
+
+        api_expects(:content_view_components, :remove_components) do |p|
+          component_ids = p['component_ids'].map(&:to_s)
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            component_ids.include?(@content_view_component.id.to_s) &&
+            component_ids.include?(@content_view_component2.id.to_s)
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'orgs and component-content-view-names' do
+        component_cv_names = @content_view_components.map do |cvc|
+          cvc["content_view"]["name"]
+        end
+
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-views=#{component_cv_names.join(',')}",
+                  "--organization=#{@organization.name}"]
+        # 1) Should fetch the org using org name specified in --organization
+        # 2) Should fetch composite content view using name in --composite-content-view
+        # 3) Should fetch component content view using name in --component-content-view
+
+        setup_org_expectations
+        setup_content_view_expectations(@composite)
+        setup_content_view_components_expectations
+
+        api_expects(:content_view_components, :remove_components) do |p|
+          component_ids = p['component_ids'].map(&:to_s)
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            component_ids.include?(@content_view_component.id.to_s) &&
+            component_ids.include?(@content_view_component2.id.to_s)
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,134 @@
+require_relative '../../test_helper'
+require 'hammer_cli_katello/content_view_component'
+# rubocop:disable Metrics/ModuleLength
+module HammerCLIKatello
+  describe ContentViewComponent::UpdateCommand do
+    before do
+      @cmd = %w(content-view component update)
+      @component_version = OpenStruct.new(:id => 666, :version => "1.0")
+      @component = OpenStruct.new(:id => 6, :name => "component",
+                                  :content_view_version => @component_version)
+      @content_view_component = OpenStruct.new(:id => 1444,
+                                               :content_view => @component,
+                                               :content_view_version => @component_version)
+
+      @composite = OpenStruct.new(:id => 2, :name => "composite",
+                                  :content_view_components => [@content_view_component])
+      @organization = OpenStruct.new(:id => 1, :name => "great", :label => "label")
+    end
+
+    def setup_org_expectations
+      org_expect = api_expects(:organizations, :index) do |par|
+        par[:search] == "name = \"#{@organization.name}\""
+      end
+      org_expect.at_least_once.returns(index_response([{'id' => @organization.id.to_s}]))
+    end
+
+    def setup_content_view_expectations(content_view)
+      cv_expect = api_expects(:content_views, :index) do |p|
+        p['name'] == content_view.name && p["organization_id"].to_s == @organization.id.to_s
+      end
+      cv_expect.at_least_once.returns(index_response([{"id" => content_view.id}]))
+    end
+
+    def setup_content_view_components_expectations
+      cvc_expect = api_expects(:content_view_components, :index) do |p|
+        p[:composite_content_view_id].to_s == @composite.id.to_s
+      end
+      ret = {
+        "id" => @content_view_component.id,
+        "content_view" => {"id" => @content_view_component.content_view.id},
+        "content_view_version" => {"id" => @content_view_component.content_view_version.id}
+      }
+
+      cvc_expect.at_least_once.returns(index_response([ret]))
+    end
+
+    describe 'allows updating a component with' do
+      it 'id and component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--id=#{@content_view_component.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--latest"]
+
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_id'].to_s == @component.id.to_s &&
+            !p.key?('content_view_version_id')
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'id and component-content-view-version-id and latest=false' do
+        latest = false
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--id=#{@content_view_component.id}",
+                  "--component-content-view-version-id=#{@component.content_view_version.id}"]
+
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_version_id'].to_s == @component.content_view_version.id.to_s &&
+            !p.key?('content_view_id')
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'component-content-view-id and latest=true' do
+        latest = true
+        params = ["--composite-content-view-id=#{@composite.id}",
+                  "--component-content-view-id=#{@component.id}",
+                  "--latest"]
+        setup_content_view_components_expectations
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_id'].to_s == @component.id.to_s &&
+            !p.key?('content_view_version_id')
+        end
+        run_cmd(@cmd + params)
+      end
+
+      it 'by names and orgs' do
+        latest = false
+        params = ["--composite-content-view=#{@composite.name}",
+                  "--component-content-view=#{@component.name}",
+                  "--component-content-view-version=#{@component.content_view_version.version}",
+                  "--organization=#{@organization.name}"]
+        # 1) Should fetch the org using org name specified in --organization
+        # 2) Should fetch composite content view using name in --composite-content-view
+        # 3) Should fetch component content view using name in --component-content-view
+        # 4) Should fetch the component_content_view_version_id
+        #    using cv specified in --component-content-view-id and version specified in
+        #    --component-content-view-version
+        # 5) Should set the component[:content_view_id] && component[:content_view_version_id]
+        #    correctly
+
+        setup_org_expectations
+        setup_content_view_expectations(@component)
+        setup_content_view_expectations(@composite)
+        setup_content_view_components_expectations
+        cvv_expect = api_expects(:content_view_versions, :index) do |p|
+          p['content_view_id'].to_s == @component.id.to_s &&
+            p["version"].to_s == @component.content_view_version.version.to_s
+        end
+        cvv_expect.at_least_once.
+          returns(index_response([{'id' => @component.content_view_version.id}]))
+
+        api_expects(:content_view_components, :update) do |p|
+          p['composite_content_view_id'].to_s == @composite.id.to_s &&
+            p['id'].to_s == @content_view_component.id.to_s &&
+            p['latest'] == latest &&
+            p['content_view_id'].to_s == @component.id.to_s &&
+            p['content_view_version_id'].to_s == @component.content_view_version.id.to_s
+        end
+        run_cmd(@cmd + params)
+      end
+    end
+  end
+end

data/test/functional/content_view/content_view_helpers.rb

@@ -0,0 +1,20 @@
+require_relative '../search_helpers'
+
+module ContentViewHelpers
+  include SearchHelpers
+
+  def expect_content_view_search(org_id, name, id)
+    expect_generic_content_view_search(params: {'name' => name, 'organization_id' => org_id},
+                                       returns: {'id' => id})
+  end
+
+  def expect_generic_content_view_search(args)
+    expect_generic_search(:content_views, args)
+  end
+
+  def expect_content_view_version_search(params, returns)
+    expect_lenient_search(:content_view_versions,
+                          params: params,
+                          returns: returns)
+  end
+end

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/create_test.rb

@@ -0,0 +1,34 @@
+require_relative '../test_helper'
+
+describe 'content-view create' do
+  before do
+    @cmd = %w(content-view create)
+    @base_params = ["--organization-id=#{org_id}", "--name=#{name}"]
+  end
+  let(:error_heading) { "Could not create the content view" }
+  let(:name) { 'test-cv' }
+  let(:org_id) { 1 }
+  let(:repositories) do
+    [
+      {'id' => '1'},
+      {'id' => '2'},
+      {'id' => '3'}
+    ]
+  end
+
+  it 'creates the content view with repositories specified by ids' do
+    wanted = repositories.take(2)
+    ids = wanted.map { |repo| repo['id'] }
+    params = %W(--repository-ids=#{ids.join(',')})
+
+    api_expects(:content_views, :create, 'Create content view') do |par|
+      par['name'] == name &&
+        par['repository_ids'] == ids &&
+        par['organization_id'] == org_id
+    end
+
+    expected_result = success_result("Content view created.\n")
+    result = run_cmd(@cmd + @base_params + params)
+    assert_cmd(expected_result, result)
+  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