hammer_cli_katello 1.0.1 → 1.2.0

data/lib/hammer_cli_katello/cv_import_export_helper.rb

@@ -1,187 +0,0 @@
-# rubocop:disable Metrics/ModuleLength
-module HammerCLIKatello
-  module CVImportExportHelper
-    PUBLISHED_REPOS_DIR = "/var/lib/pulp/published/yum/https/repos/".freeze
-
-    def fetch_exportable_cvv_repositories(cvv)
-      immediate = []
-      non_immediate_names = []
-
-      cvv['repositories'].each do |repo|
-        next unless repo['content_type'] == 'yum'
-
-        api_repo = show(:repositories, 'id' => repo['id'], :full_result => true)
-
-        download_policy = if api_repo['library_instance_id']
-                            library = show(:repositories, 'id' => api_repo['library_instance_id'])
-                            library['download_policy']
-                          else
-                            api_repo['download_policy']
-                          end
-
-        if download_policy == 'immediate'
-          immediate << api_repo
-        else
-          non_immediate_names << api_repo['name']
-        end
-      end
-
-      warn_repo_download_policy(non_immediate_names)
-
-      return immediate
-    end
-
-    def find_local_component_id(component_from_export)
-      *name, version = component_from_export.split(' ')
-      name = name.join(' ')
-      existing_component_cv = content_view(name, options['option_organization_id'])
-      found_composite_version = existing_component_cv['versions'].select do |v|
-        v['version'] == version
-      end
-      if found_composite_version.empty?
-        raise _("Unable to find CV version %{cvv} on system. Please ensure it " \
-                "is already imported." % {'cvv' => component_from_export})
-      end
-      found_composite_version.first['id']
-    end
-
-    def warn_repo_download_policy(repository_names)
-      return if repository_names.empty?
-
-      msg = <<~MSG
-        The following repositories could not be exported due to the download policy
-        not being set to 'immediate':
-          #{repository_names.join(', ')}
-      MSG
-      print_message msg
-    end
-
-    def collect_packages(repositories)
-      repositories.each do |repo|
-        per_page = 50
-        repo['packages'] = []
-        repo['errata'] = []
-        repo_packages = repo['content_counts']['rpm']
-        errata_count = repo['content_counts']['erratum']
-        pkg_pages = (repo_packages / per_page.to_f).ceil
-        errata_pages = (errata_count / per_page.to_f).ceil
-        (1..pkg_pages).each do |page|
-          repo['packages'] += index(:packages, 'repository_id' => repo['id'],
-                                               :page => page, :per_page => 50)
-        end
-        (1..errata_pages).each do |page|
-          repo['errata'] += index(:errata, 'repository_id' => repo['id'],
-                                           :page => page, :per_page => 50)
-        end
-      end
-    end
-
-    def import_checks(cv, import_cv, major, minor)
-      version = "#{major}.#{minor}".to_f
-
-      if import_cv.nil?
-        raise _("The Content View #{cv['name']} is not present on this server,"\
-        " please create the Content View and try the import again.")
-      end
-
-      unless import_cv['default']
-        if import_cv['latest_version'].to_f >= version
-          raise _("The latest version (#{import_cv['latest_version']}) of"\
-          " the Content View '#{cv['name']}'"\
-          " is greater or equal to the version you are trying to import (#{version})")
-        end
-      end
-
-      unless import_cv['repository_ids'].nil?
-        repositories = import_cv['repository_ids'].collect do |repo_id|
-          show(:repositories, 'id' => repo_id)
-        end
-        repositories.each do |repo|
-          if repo['mirror_on_sync'] == true
-            raise _("The Repository '#{repo['name']}' is set with Mirror-on-Sync to YES."\
-            " Please change Mirror-on-Sync to NO and try the import again.")
-          end
-        end
-      end
-    end
-
-    def untar_export(options)
-      export_tar_file = options[:filename]
-      export_tar_dir =  options[:dirname]
-      export_tar_prefix = options[:prefix]
-
-      Dir.chdir(export_tar_dir) do
-        `tar -xf #{export_tar_file}`
-      end
-
-      Dir.chdir("#{export_tar_dir}/#{export_tar_prefix}") do
-        if File.exist?(export_tar_file.gsub('.tar', '-repos.tar'))
-          `tar -xf #{export_tar_file.gsub('.tar', '-repos.tar')}`
-        end
-      end
-    end
-
-    def obtain_export_params(option_export_tar)
-      export_tar_file = File.basename(option_export_tar)
-      {:filename => export_tar_file,
-       :dirname => File.dirname(option_export_tar),
-       :prefix => export_tar_file.gsub('.tar', '')}
-    end
-
-    def read_json(options)
-      export_tar_file = options[:filename]
-      export_tar_dir =  options[:dirname]
-      export_tar_prefix = options[:prefix]
-
-      json_file = export_tar_file.gsub('tar', 'json')
-      json_file = "#{export_tar_dir}/#{export_tar_prefix}/#{json_file}"
-      json_file = File.read(json_file)
-      JSON.parse(json_file)
-    end
-
-    def export_json(export_json_options)
-      content_view_version = export_json_options[:cvv]
-      repositories = export_json_options[:repositories]
-      json = {
-        "name" => content_view_version['content_view']['name'],
-        "major" => content_view_version['major'],
-        "minor" => content_view_version['minor']
-      }
-      json["composite_components"] = export_json_options[:component_cvvs]
-      json["repositories"] = repositories.collect do |repo|
-        {
-          "id" => repo['id'],
-          "label" => repo['label'],
-          "content_type" => repo['content_type'],
-          "backend_identifier" => repo['backend_identifier'],
-          "relative_path" => repo['relative_path'],
-          "on_disk_path" => "#{PUBLISHED_REPOS_DIR}/#{repo['relative_path']}",
-          "rpm_filenames" => repo['packages'].collect { |package| package['filename'] },
-          "errata_ids" => repo['errata'].collect { |errata| errata['errata_id'] }
-        }
-      end
-      json
-    end
-
-    def fail_msg_import
-      _("This command is not supported with Pulp 3. Use `hammer content-import` instead.")
-    end
-
-    def fail_msg_export
-      _("This command is not supported with Pulp 3. Use `hammer content-export` instead.")
-    end
-
-    def validate_pulp3_not_enabled(failure_message)
-      pulp3_enabled = HammerCLIForeman
-                      .foreman_api_connection
-                      .resource(:content_exports)
-                      .call(:api_status)['api_usable']
-      if pulp3_enabled
-        raise failure_message
-      end
-    rescue NoMethodError
-      # if the api endpoint isn't there, the validation passes
-      true
-    end
-  end
-end

data/test/functional/content_view/version/cv_import_export_helper_test.rb

@@ -1,20 +0,0 @@
-require_relative '../../test_helper'
-describe 'CVImportExportHelper' do
-  describe 'validate_pulp3_not_enabled' do
-    include HammerCLIKatello::CVImportExportHelper
-    it 'returns true when the api endpoint does not return a value' do
-      api_expects(:content_exports, :api_status)
-      assert_equal true, validate_pulp3_not_enabled('fail_msg')
-    end
-
-    it 'raises error when the api reports Pulp 3 is enabled' do
-      api_expects(:content_exports, :api_status).returns('api_usable' => true)
-      assert_raises(RuntimeError, 'fail msg') { validate_pulp3_not_enabled('fail msg') }
-    end
-
-    it 'returns nil when the api reports Pulp 3 is not enabled' do
-      api_expects(:content_exports, :api_status).returns('api_usable' => false)
-      assert_equal nil, validate_pulp3_not_enabled('fail_msg')
-    end
-  end
-end

data/test/functional/content_view/version/default_export_test.rb

@@ -1,40 +0,0 @@
-require File.join(File.dirname(__FILE__), '../../test_helper')
-
-describe 'content-view version export' do
-  include ForemanTaskHelpers
-
-  before do
-    @cmd = %w(content-view version export-default)
-  end
-
-  it "performs export with bad SELinux" do
-    params = [
-      '--export-dir=/tmp/default'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    Dir.expects(:mkdir).with('/tmp/default').returns(0)
-    Kernel.expects(:system).with("rsync -aL /var/lib/pulp/published/yum/https/repos/ /tmp/default")
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_CANTCREAT, result.exit_code)
-  end
-
-  it "performs export" do
-    params = [
-      '--export-dir=/tmp/default'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    Dir.expects(:exist?).with('/tmp/default').returns(0)
-    Kernel.expects(:system).with("rsync -aL /var/lib/pulp/published/yum/https/repos/ /tmp/default")
-          .returns(true)
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_OK, result.exit_code)
-  end
-end

data/test/functional/content_view/version/export_test.rb

@@ -1,148 +0,0 @@
-require File.join(File.dirname(__FILE__), '../../test_helper')
-
-describe 'content-view version export' do
-  include ForemanTaskHelpers
-
-  before do
-    @cmd = %w(content-view version export)
-  end
-
-  it "performs export" do
-    HammerCLIKatello::ContentViewVersion::ExportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--id=5',
-      '--export-dir=/tmp/exports'
-    ]
-
-    ex = api_expects(:content_view_versions, :show)
-    ex.returns(
-      'id' => '5',
-      'repositories' => [{'id' => '2', 'content_type' => 'yum'}],
-      'major' => 1,
-      'minor' => 0,
-      'content_view' => {'name' => 'cv'},
-      'content_view_id' => 4321
-    )
-
-    ex = api_expects(:content_views, :show)
-    ex.returns(
-      'id' => '4321',
-      'composite' => false,
-      'label' => 'cv'
-    )
-
-    ex = api_expects(:repositories, :show).with_params('id' => '2')
-    ex.returns(
-      'id' => '2',
-      'label' => 'Test_Repo',
-      'content_type' => 'yum',
-      'download_policy' => 'immediate',
-      'backend_identifier' => 'Default_Organization-Library-Test_Repo',
-      'relative_path' => 'Default_Organization/Library/Test_Repo',
-      'library_instance_id' => '1',
-      'content_counts' => {
-        'rpm' => 1,
-        'erratum' => 1
-      }
-    )
-
-    api_expects(:repositories, :show).with_params('id' => '1').returns(
-      'id' => '1',
-      'download_policy' => 'immediate'
-    )
-    api_expects(:packages, :index).returns('results' => [])
-    api_expects(:errata, :index).returns('results' => [])
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    Dir.expects(:chdir).with("/var/lib/pulp/published/yum/https/repos/").returns(true)
-    Dir.expects(:mkdir).with('/tmp/exports/export-cv-1.0').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-cv-1.0').returns(0)
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_OK, result.exit_code)
-  end
-
-  it "performs composite export" do
-    HammerCLIKatello::ContentViewVersion::ExportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--id=999',
-      '--export-dir=/tmp/exports'
-    ]
-
-    ex = api_expects(:content_view_versions, :show)
-    ex.returns(
-      'id' => '999',
-      'repositories' => [{'id' => '2'}],
-      'major' => 1,
-      'minor' => 0,
-      'content_view' => {'name' => 'cv'},
-      'content_view_id' => 4321
-    )
-
-    ex = api_expects(:content_views, :show)
-    ex.returns(
-      'id' => '4321',
-      'composite' => true,
-      'components' => [{ 'name' => "injera 95.5" }, {'name' => 'carrot wot 87.0'}],
-      'label' => 'cv'
-    )
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    Dir.expects(:chdir).with("/var/lib/pulp/published/yum/https/repos/").never
-    Dir.expects(:mkdir).with('/tmp/exports/export-cv-1.0').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-cv-1.0').returns(0)
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_OK, result.exit_code)
-  end
-
-  it "fails export if content view version has no repository" do
-    HammerCLIKatello::ContentViewVersion::ExportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--id=5',
-      '--export-dir=/tmp/exports'
-    ]
-
-    ex = api_expects(:content_view_versions, :show)
-    ex.returns(
-      'id' => '5',
-      'name' => 'Test_version',
-      'repositories' => [],
-      'major' => 1,
-      'minor' => 0,
-      'content_view' => {'name' => 'cv'},
-      'content_view_id' => 4321,
-      'puppet_modules' => []
-    )
-
-    ex = api_expects(:content_views, :show)
-    ex.returns(
-      'id' => '4321',
-      'composite' => false
-    )
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    result = run_cmd(@cmd + params)
-    assert_equal(result.err, "Could not export the content view:\n"\
-                             "  Error: Ensure the content view version 'Test_version'"\
-                             " has at least one repository.\n")
-    assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
-  end
-end

data/test/functional/content_view/version/import_test.rb

@@ -1,346 +0,0 @@
-require File.join(File.dirname(__FILE__), '../../test_helper')
-
-describe 'content-view version import' do
-  include ForemanTaskHelpers
-
-  before do
-    @cmd = %w(content-view version import)
-  end
-
-  it "performs import" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-2.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
-    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
-      JSON.dump(
-        'name' => 'Foo View',
-        'major' => '5',
-        'minor' => '0',
-        'repositories' => [{
-          'label' => 'foo',
-          'rpm_filenames' => ['foo-1.0-1.el7']
-        }]
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{
-        'id' => '5',
-        'repositories' => [{'id' => '2', 'label' => 'foo'}],
-        'content_view' => {'name' => 'cv'}
-      }]
-    )
-
-    ex = api_expects(:repositories, :index)
-    ex = ex.with_params('organization_id' => '1', 'library' => true)
-    ex.returns(
-      'results' => [{
-        'id' => '2',
-        'label' => 'foo'
-      }]
-    )
-
-    ex = api_expects(:repositories, :sync)
-    ex = ex.with_params('id' => '2', 'source_url' => "file:///tmp/exports/export-2/")
-    ex.returns('id' => '2', 'state' => 'planned')
-
-    expect_foreman_task('3')
-
-    ex = api_expects(:content_views, :publish)
-    ex = ex.with_params(
-      'id' => '5',
-      'major' => '5',
-      'minor' => '0',
-      'repos_units' => [{
-        'label' => 'foo',
-        'rpm_filenames' => ['foo-1.0-1.el7']
-      }]
-    )
-    ex.returns('id' => '2', 'state' => 'planned')
-
-    expect_foreman_task('3')
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_OK, result.exit_code)
-  end
-
-  it "performs composite import" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-999.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-999.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-999').returns(0)
-    File.expects(:read).with("/tmp/exports/export-999/export-999.json").returns(
-      JSON.dump(
-        'name' => 'Foo Composite View',
-        'major' => '10',
-        'minor' => '0',
-        'composite_components' => ["berbere 55.32"]
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'berbere', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{'versions' => [{'version' => '10.0', 'id' => '654'},
-                                    {'version' => '55.32', 'id' => '876'}]
-                    }]
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo Composite View', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{
-        'id' => '5',
-        'repositories' => [{'id' => '2', 'label' => 'foo'}],
-        'content_view' => {'name' => 'cv'}
-      }]
-    )
-
-    ex = api_expects(:content_views, :update)
-    ex = ex.with_params(
-      'id' => '5',
-      'component_ids' => ['876']
-    )
-    ex.returns('id' => '5', 'state' => 'planned')
-
-    ex = api_expects(:content_views, :publish)
-    ex = ex.with_params(
-      'id' => '5',
-      'major' => '10',
-      'minor' => '0'
-    )
-    ex.returns('id' => '2', 'state' => 'planned')
-
-    expect_foreman_task('3')
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_OK, result.exit_code)
-  end
-
-  it "performs composite import, component not found" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-999.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-999.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-999').returns(0)
-    File.expects(:read).with("/tmp/exports/export-999/export-999.json").returns(
-      JSON.dump(
-        'name' => 'Foo Composite View',
-        'major' => '10',
-        'minor' => '0',
-        'composite_components' => ["berbere 55.32", "unicorn 99.99"]
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'berbere', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{'versions' => [{'version' => '10.0', 'id' => '654'},
-                                    {'version' => '55.32', 'id' => '876'}]
-                    }]
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'unicorn', 'organization_id' => '1')
-    ex.returns(
-      'results' => []
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo Composite View', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{
-        'id' => '5',
-        'repositories' => [{'id' => '2', 'label' => 'foo'}],
-        'content_view' => {'name' => 'cv'}
-      }]
-    )
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
-  end
-
-  it "fails import if cv has not been created" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-2.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
-    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
-      JSON.dump(
-        'name' => 'Foo View'
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
-    ex.returns([])
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
-  end
-
-  it "fails import if repo is set to mirror-on-sync" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-2.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
-    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
-      JSON.dump(
-        'name' => 'Foo View',
-        'major' => '2',
-        'minor' => '1'
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{
-        'id' => '5',
-        'repositories' => [{'id' => '2', 'label' => 'foo', 'mirror_on_sync' => 'true'}],
-        'content_view' => {'name' => 'cv'}
-      }]
-    )
-
-    result = run_cmd(@cmd + params)
-    assert_equal(result.exit_code, 70)
-  end
-
-  it "fails import if cv version already exists" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-2.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
-    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
-      JSON.dump(
-        'name' => 'Foo View',
-        'major' => '2',
-        'minor' => '1'
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{
-        'id' => '5',
-        'content_view' => {'name' => 'cv'},
-        'versions' => [{'version' => '2.1', 'id' => '654'}]
-      }]
-    )
-
-    result = run_cmd(@cmd + params)
-    assert_equal(result.exit_code, 70)
-  end
-
-  it "fails import if any repository does not exist" do
-    HammerCLIKatello::ContentViewVersion::ImportCommand
-      .any_instance
-      .expects(:validate_pulp3_not_enabled)
-      .returns(true)
-    params = [
-      '--export-tar=/tmp/exports/export-2.tar',
-      '--organization-id=1'
-    ]
-
-    File.expects(:exist?).with('/usr/share/foreman').returns(true)
-    File.stubs(:exist?).with('/var/log/hammer/hammer.log._copy_').returns(false)
-
-    File.expects(:exist?).with("/tmp/exports/export-2.tar").returns(true)
-    Dir.expects(:chdir).with('/tmp/exports').returns(0)
-    Dir.expects(:chdir).with('/tmp/exports/export-2').returns(0)
-    File.expects(:read).with("/tmp/exports/export-2/export-2.json").returns(
-      JSON.dump(
-        'name' => 'Foo View',
-        'repositories' => ['label' => 'foo'],
-        'major' => '2',
-        'minor' => '1'
-      )
-    )
-
-    ex = api_expects(:content_views, :index)
-    ex = ex.with_params('name' => 'Foo View', 'organization_id' => '1')
-    ex.returns(
-      'results' => [{
-        'id' => '5',
-        'repositories' => [{'id' => '2', 'label' => 'foo'}],
-        'content_view' => {'name' => 'cv'}
-      }]
-    )
-
-    ex = api_expects(:repositories, :index)
-    ex = ex.with_params('organization_id' => '1', 'library' => true)
-    ex.returns([])
-
-    result = run_cmd(@cmd + params)
-    assert_equal(HammerCLI::EX_SOFTWARE, result.exit_code)
-  end
-end