hammer_cli_katello 0.0.24 → 0.0.25

data/test/functional/organization/organization_helpers.rb

@@ -0,0 +1,10 @@
+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/ping_test.rb

@@ -0,0 +1,19 @@
+require_relative 'test_helper'
+
+describe 'ping' do
+  it 'does not require authentication' do
+    api_expects(:ping, :index).returns(
+      'status' => 'ok',
+      'services' => {
+        'foreman_tasks' => {'status' => 'ok', 'duration_ms' => '34'},
+        'foreman_auth' => {'status' => 'ok', 'duration_ms' => '34'},
+        'candlepin' => {'status' => 'ok', 'duration_ms' => '34'},
+        'candlepin_auth' => {'status' => 'ok', 'duration_ms' => '34'},
+        'pulp' => {'status' => 'ok', 'duration_ms' => '34'},
+        'pulp_auth' => {'status' => 'ok', 'duration_ms' => '34'}
+      }
+    )
+
+    run_cmd(%w(ping))
+  end
+end

data/test/functional/product/product_helpers.rb

@@ -0,0 +1,8 @@
+module ProductHelpers
+  def expect_product_search(org_id, name, id)
+    ex = api_expects(:products, :index, 'Find the Product') do |par|
+      par['name'] == name && par['organization_id'] == org_id
+    end
+    ex.returns(index_response([{'id' => id}]))
+  end
+end

data/test/functional/repository/info_test.rb

@@ -0,0 +1,46 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+require File.join(File.dirname(__FILE__), './repository_helpers')
+
+describe "get repository info" do
+  include ProductHelpers
+  include RepositoryHelpers
+
+  before do
+    @cmd = %w(repository info)
+  end
+
+  let(:org_id) { 1 }
+  let(:product_id) { 2 }
+  let(:repo_id) { 3 }
+
+  it "Shows information about a repository" do
+    params = ["--id=#{repo_id}"]
+
+    ex = api_expects(:repositories, :show, "Get info") do |par|
+      par["id"] == repo_id.to_s
+    end
+
+    ex.returns({})
+
+    result = run_cmd(@cmd + params)
+    assert(result.exit_code, 0)
+  end
+
+  it "Shows information about a repository with organization-id and product name" do
+    params = ["--name=test_repo", "--product=test_product", "--organization-id=#{org_id}"]
+
+    expect_product_search(org_id, 'test_product', product_id)
+
+    expect_repository_search(org_id, product_id, 'test_repo', repo_id)
+
+    ex2 = api_expects(:repositories, :show, "Get info") do |par|
+      par["id"] == repo_id
+    end
+
+    ex2.returns({})
+
+    result = run_cmd(@cmd + params)
+    assert(result.exit_code, 0)
+  end
+end

data/test/functional/repository/list_test.rb

@@ -0,0 +1,69 @@
+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 repositories' do
+  include LifecycleEnvironmentHelpers
+
+  before do
+    @cmd = %w(repository list)
+  end
+
+  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" => []
+  }
+  end
+
+  it "lists an organizations repositories" do
+    params = ["--organization-id=#{org_id}"]
+
+    ex = api_expects(:repositories, :index, 'Organizations repositories list') do |par|
+      par['organization_id'] == org_id && par['page'] == 1 &&
+        par['per_page'] == 1000
+    end
+
+    ex.returns(empty_response)
+
+    expected_result = success_result("---|------|---------|--------------|----
+ID | NAME | PRODUCT | CONTENT TYPE | URL
+---|------|---------|--------------|----
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+
+  it "lists the repositories belonging to a lifecycle-environment by name" do
+    params = ['--organization-id=1', '--environment=test']
+
+    expect_lifecycle_environment_search(org_id, 'test', lifecycle_env_id)
+
+    ex = api_expects(:repositories, :index, 'lifecycle repositories list') do |par|
+      par['organization_id'] == org_id && par['page'] == 1 &&
+        par['per_page'] == 1000
+    end
+
+    ex.returns(empty_response)
+
+    expected_result = success_result("---|------|---------|--------------|----
+ID | NAME | PRODUCT | CONTENT TYPE | URL
+---|------|---------|--------------|----
+")
+
+    result = run_cmd(@cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end

data/test/functional/repository/repository_helpers.rb

@@ -0,0 +1,9 @@
+module RepositoryHelpers
+  def expect_repository_search(org_id, 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
+    end
+    ex.returns(index_response([{'id' => id}]))
+  end
+end

data/test/functional/repository/synchronize_test.rb

@@ -0,0 +1,57 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+require File.join(File.dirname(__FILE__), './repository_helpers')
+
+describe 'Synchronize a repository' do
+  include ForemanTaskHelpers
+  include ProductHelpers
+  include RepositoryHelpers
+
+  before do
+    @cmd = %w(repository synchronize)
+  end
+
+  let(:org_id) { 1 }
+  let(:repo_id) { 2 }
+  let(:product_id) { 3 }
+  let(:sync_response) do
+    {
+      'id' => repo_id,
+      'state' => 'planned'
+    }
+  end
+
+  it "synchronizes a repository" do
+    params = ["--id=#{repo_id}"]
+
+    ex = api_expects(:repositories, :sync, 'Repository is synced') do |par|
+      par['id'] == repo_id.to_s
+    end
+
+    ex.returns(sync_response)
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert(result.exit_code, 0)
+  end
+
+  it "synchronizes a repository with a repository name" do
+    params = ["--name=test_repo", "--product=test_product", "--organization-id=#{org_id}"]
+
+    expect_product_search(org_id, "test_product", product_id)
+
+    expect_repository_search(org_id, product_id, "test_repo", repo_id)
+
+    ex = api_expects(:repositories, :sync, 'Repository is synced') do |par|
+      par['id'] == repo_id
+    end
+
+    ex.returns(sync_response)
+
+    expect_foreman_task('3')
+
+    result = run_cmd(@cmd + params)
+    assert(result.exit_code, 0)
+  end
+end

data/test/functional/repository/upload_test.rb

@@ -0,0 +1,86 @@
+require File.join(File.dirname(__FILE__), '../test_helper')
+require File.join(File.dirname(__FILE__), '../product/product_helpers')
+require File.join(File.dirname(__FILE__), './repository_helpers')
+
+describe 'upload repository' do
+  include ProductHelpers
+  include RepositoryHelpers
+
+  before do
+    @cmd = %w(repository upload-content)
+  end
+
+  let(:org_id) { 1 }
+  let(:product_id) { 2 }
+  let(:repo_id) { 3 }
+  let(:path) { "./test.rpm" }
+  let(:upload_id) { "1234" }
+  let(:_href) { "/pulp/api/v2/content/uploads/#{upload_id}" }
+  let(:upload_response) do
+    {
+      "upload_id" => upload_id,
+      "_href" => _href
+    }
+  end
+
+  it "uploads a package" do
+    File.new("test.rpm", "w")
+
+    params = ["--id=#{repo_id}", "--path=#{path}"]
+
+    ex = api_expects(:content_uploads, :create, "Create upload for content") do |par|
+      par[:repository_id] == repo_id.to_s
+    end
+
+    ex.returns(upload_response)
+
+    ex2 = api_expects(:repositories, :import_uploads, 'Take in an upload') do |par|
+      par[:id] == repo_id.to_s && par[:upload_ids] == ['1234']
+    end
+
+    ex2.returns("")
+
+    ex3 = api_expects(:content_uploads, :destroy, "Delete the upload") do |par|
+      par[:id] == upload_id && par[:repository_id] == repo_id.to_s
+    end
+
+    ex3.returns("")
+
+    result = run_cmd(@cmd + params)
+    assert(result.exit_code, 0)
+    File.delete("test.rpm")
+  end
+
+  it "uploads a package with an organization-id" do
+    File.new("test.rpm", "w")
+
+    params = ["--name=test_repo", "--product=test_product", "--organization-id=#{org_id}",
+              "--path=#{path}"]
+
+    expect_product_search(org_id, 'test_product', product_id)
+
+    expect_repository_search(org_id, product_id, 'test_repo', repo_id)
+
+    ex = api_expects(:content_uploads, :create, "Create upload for content") do |par|
+      par[:repository_id] == repo_id
+    end
+
+    ex.returns(upload_response)
+
+    ex2 = api_expects(:repositories, :import_uploads, 'Take in an upload') do |par|
+      par[:id] == repo_id && par[:upload_ids] == ['1234']
+    end
+
+    ex2.returns("")
+
+    ex3 = api_expects(:content_uploads, :destroy, "Delete the upload") do |par|
+      par[:id] == upload_id && par[:repository_id] == repo_id
+    end
+
+    ex3.returns("")
+
+    result = run_cmd(@cmd + params)
+    assert(result.exit_code, 0)
+    File.delete("test.rpm")
+  end
+end

data/test/task_helper.rb

@@ -0,0 +1,7 @@
+module ForemanTaskHelpers
+  def expect_foreman_task(task_id)
+    ex = api_expects(:foreman_tasks, :show, 'Show task')
+    ex.returns('id' => task_id, 'state' => 'stopped', 'progress' => 1,
+               'humanized' => {'output' => '', 'errors' => ''})
+  end
+end

data/test/test_helper.rb

@@ -1,11 +1,24 @@
+if RUBY_VERSION > "2.2"
+  # Coverage - Keep these two lines at the top of this file
+  require 'simplecov'
+  require 'coveralls'
+  SimpleCov.formatters = [SimpleCov::Formatter::HTMLFormatter, Coveralls::SimpleCov::Formatter]
+  SimpleCov.start do
+    minimum_coverage 46
+    refuse_coverage_drop
+    track_files "lib/**/*.rb"
+    add_filter '/test/'
+  end
+end
+
+require File.join(File.dirname(__FILE__), './task_helper.rb')
 require 'minitest/autorun'
 require 'minitest/spec'
-require "mocha/setup"
-
+require 'mocha/setup'
 require 'hammer_cli'
 require 'hammer_cli_foreman/commands'
 
-KATELLO_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '2.5')
+KATELLO_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '3.0')
 
 HammerCLIForeman.stubs(:resource_config).returns(
   :apidoc_cache_dir => 'test/data/' + KATELLO_VERSION.to_s,

data/test/unit/id_resolver_test.rb

@@ -0,0 +1,32 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/id_resolver'
+
+module HammerCLIKatello
+  describe IdResolver do
+    let(:api) { mock('api') }
+    let(:searchables) { mock('searchables') }
+    let(:id_resolver) { IdResolver.new api, searchables }
+    describe '#repository_ids' do
+      before(:each) do
+        api.stubs(:resources).returns([])
+      end
+      it 'accepts repository_ids' do
+        id_resolver.repository_ids(
+          'option_repository_ids' => [1, 2]
+        ).must_equal [1, 2]
+      end
+
+      it 'accepts repository_names' do
+        id_resolver.stubs(:find_resources).returns(
+          [
+            {'id' => 1, 'name' => 'repo1'},
+            {'id' => 2, 'name' => 'repo2'}
+          ]
+        )
+        id_resolver.repository_ids(
+          'option_names' => %w(repo1 repo2)
+        ).must_equal [1, 2]
+      end
+    end
+  end
+end

data/test/unit/search_options_creators_test.rb

@@ -0,0 +1,107 @@
+require_relative '../test_helper'
+require 'hammer_cli_katello/search_options_creators'
+
+class SearchOptionsCreatorsMock
+  include HammerCLIKatello::SearchOptionsCreators
+
+  def one
+  end
+
+  def two
+  end
+end
+
+describe HammerCLIKatello::SearchOptionsCreators do
+
+  let(:search_options_creators) { SearchOptionsCreatorsMock.new }
+  let(:options) { Hash.new }
+  let(:resource) { mock('ApipieBindings::Resource') }
+
+  describe '#create_repositories_search_options' do
+    it 'handles an array of names' do
+      search_options_creators.create_repositories_search_options(
+        'option_names' => ['repo1']
+      )['names'].must_equal ['repo1']
+    end
+
+    it 'handles a single value name' do
+      search_options_creators.create_repositories_search_options(
+        'option_name' => 'repo1'
+      )['name'].must_equal 'repo1'
+    end
+
+    it 'handles a single value product_id' do
+      search_options_creators.create_repositories_search_options(
+        'option_product_id' => 3
+      )['product_id'].must_equal 3
+    end
+  end
+
+  describe '#create_content_view_versions_search_options(options)' do
+    it 'handles a single value environment_id' do
+      search_options_creators.create_content_view_versions_search_options(
+        'option_environment_id' => 3,
+        'option_content_view_id' => :required_option
+      )['environment_id'].must_equal 3
+    end
+
+    it 'handles a single value content_view_id' do
+      search_options_creators.create_content_view_versions_search_options(
+        'option_content_view_id' => 3
+      )['content_view_id'].must_equal 3
+    end
+
+    it 'handles a single value version' do
+      search_options_creators.create_content_view_versions_search_options(
+        'option_version' => 3
+      )['version'].must_equal 3
+    end
+  end
+
+  describe 'without the katello api' do
+    let(:api) { mock('api') }
+
+    before(:each) do
+      search_options_creators.expects(:create_search_options_without_katello_api)
+      search_options_creators.stubs(:api).returns(api)
+      api.stubs(:resource)
+    end
+
+    describe '#create_organizations_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_organizations_search_options(:anything)
+      end
+    end
+
+    describe '#create_smart_proxies_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_organizations_search_options(:anything)
+      end
+    end
+
+    describe '#create_capsules_search_options(options)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_organizations_search_options(:anything)
+      end
+    end
+
+    describe '#create_search_options_with_katello_api(options, resource)' do
+      it 'does not use the katello api' do
+        search_options_creators.create_organizations_search_options(:anything)
+      end
+    end
+
+  end # describe 'without the katello api'
+
+  describe '#create_search_options_with_katello_api' do
+    it 'translates all searchable fields from options' do
+      search_options_creators.stubs(:searchables).
+        returns([search_options_creators.method(:one), search_options_creators.method(:two)])
+
+      search_options_creators.create_search_options_with_katello_api(
+        {'option_one' => 1, 'option_two' => 2}, resource
+      ).must_equal('one' => '1', 'two' => '2')
+    end
+  end # describe '#create_search_options_with_katello_api'
+
+end # describe HammerCLIKatello::SearchOptionsCreators