hammer_cli_foreman_puppet 0.0.7 → 0.1.1

data/test/functional/proxy_test.rb

@@ -1,86 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-
-describe 'proxy' do
-  describe 'import-classes' do
-    let(:cmd) { ['proxy', 'import-classes'] }
-    let(:params) { ['--id=83'] }
-    let(:report) do
-      {
-        'message' => 'Successfully updated environment and puppetclasses from the on-disk puppet installation',
-        'environments_with_new_puppetclasses' => 2,
-        'environments_updated_puppetclasses' => 0,
-        'environments_obsolete' => 0,
-        'environments_ignored' => 0,
-        'results' => [{
-          'name' => 'development',
-          'actions' => [ 'new', 'update' ],
-          'new_puppetclasses' => [
-            'motd',
-            'hammer'
-          ],
-          'updated_puppetclasses' => [
-            'stdlib',
-            'vim'
-          ]
-        },{
-          'name' => 'production',
-          'actions' => [ 'obsolete', 'ignore' ],
-          'obsolete_puppetclasses' => [
-            'apache'
-          ],
-          'ignored_puppetclasses' => [
-            'hammer'
-          ]
-        }]
-      }
-    end
-    let(:no_change_report) do
-      {
-        'message' => 'No changes to your environments detected'
-      }
-    end
-
-    it 'prints detailed report' do
-      api_expects(:smart_proxies, :import_puppetclasses, 'Import classes')
-        .with_params('id' => '83')
-        .returns(report)
-
-      output = OutputMatcher.new([
-        'Result:',
-        '  Successfully updated environment and puppetclasses from the on-disk puppet installation',
-        'Changed environments:',
-        ' 1) development',
-        '    New classes:',
-        '        motd',
-        '        hammer',
-        '    Updated classes:',
-        '        stdlib',
-        '        vim',
-        ' 2) production',
-        '    Removed classes:',
-        '        apache',
-        '    Ignored classes:',
-        '        hammer'
-      ])
-      expected_result = success_result(output)
-
-      result = run_cmd(cmd + params)
-      assert_cmd(expected_result, result)
-    end
-
-    it 'prints a message when nothig has changed' do
-      api_expects(:smart_proxies, :import_puppetclasses, 'Import classes')
-        .with_params('id' => '83')
-        .returns(no_change_report)
-
-      output = OutputMatcher.new([
-        'Result:',
-        '  No changes to your environments detected'
-      ])
-      expected_result = success_result(output)
-
-      result = run_cmd(cmd + params)
-      assert_cmd(expected_result, result)
-    end
-  end
-end

data/test/functional/smart_class_parameter_test.rb

@@ -1,97 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-
-describe 'sc-params update' do
-  let(:cmd) { %w[sc-param update] }
-
-  it 'does not allow to update parameter name' do
-    params = ['--id=1', '--new-name=name2']
-
-    expected_result = usage_error_result(
-      cmd,
-      "Unrecognised option '--new-name'.",
-      'Could not update the parameter'
-    )
-
-    api_expects_no_call
-    result = run_cmd(cmd + params)
-    assert_cmd(expected_result, result)
-  end
-end
-
-describe 'sc-params add-matcher' do
-  let(:cmd) { %w[sc-param add-matcher] }
-  let(:override_value) { 'something' }
-  let(:match) { 'domain = example.com' }
-  let(:puppet_class) { { 'name' => 'motd', 'id' => '1' } }
-  let(:parameter) { { 'name' => 'content', 'id' => '2' } }
-  let(:base) do
-    [
-      '--puppet-class', puppet_class['name'],
-      '--smart-class-parameter', parameter['name'],
-      '--match', match
-    ]
-  end
-
-  def api_expects_parameter_search(puppet_class, parameter)
-    api_expects(:puppetclasses, :index, 'Find puppet class') do |par|
-      par[:search] == %Q(name = "#{puppet_class['name']}")
-    end.returns(index_response('motd' => [puppet_class]))
-
-    api_expects(:smart_class_parameters, :index, 'Find smart parameter') do |par|
-      par[:search] == %Q(key = "#{parameter['name']}") &&
-        par[:puppetclass_id] == puppet_class['id']
-    end.returns(index_response([parameter]))
-  end
-
-  it 'allows to set value' do
-    params = ['--value', override_value]
-    expected_result = success_result("Override value created.\n")
-
-    api_expects_parameter_search(puppet_class, parameter)
-
-    api_expects(:override_values, :create, 'Create override value').with_params(
-      smart_class_parameter_id: parameter['id'],
-      override_value: {
-        match: match,
-        value: override_value
-      }
-    )
-
-    result = run_cmd(cmd + base + params)
-    assert_cmd(expected_result, result)
-  end
-
-  it 'allows to set value with disabled puppet default' do
-    params = ['--value', override_value, '--omit', false]
-    expected_result = success_result("Override value created.\n")
-
-    api_expects_parameter_search(puppet_class, parameter)
-
-    api_expects(:override_values, :create, 'Create override value').with_params(
-      smart_class_parameter_id: parameter['id'],
-      override_value: {
-        match: match,
-        value: override_value,
-        omit: false
-      }
-    )
-
-    result = run_cmd(cmd + base + params)
-    assert_cmd(expected_result, result)
-  end
-
-  it 'does not allow to use puppet default and value at the same time' do
-    params = ['--value', 'something', '--omit', true]
-
-    expected_result = usage_error_result(
-      cmd,
-      'Cannot use --value when --omit is true.',
-      'Could not create the override value'
-    )
-
-    api_expects_parameter_search(puppet_class, parameter)
-
-    result = run_cmd(cmd + base + params)
-    assert_cmd(expected_result, result)
-  end
-end

data/test/functional/template_test.rb

@@ -1,38 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-
-describe 'template' do
-  describe 'combinations' do
-    before do
-      @cmd = %w[template combination]
-    end
-
-    it 'should create new combination' do
-      params = ['create', '--provisioning-template-id=10', '--hostgroup-id=1', '--puppet-environment-id=1']
-      expected_result = success_result("Template combination created.\n")
-      api_expects(:template_combinations, :create, 'Create template combination') do |p|
-        p['provisioning_template_id'] == '10' &&
-          p['hostgroup_id'] == '1' &&
-          p['environment_id'] == 1 &&
-          p['template_combination'] == { 'environment_id' => 1, 'hostgroup_id' => '1' }
-      end
-
-      result = run_cmd(@cmd + params)
-      assert_cmd(expected_result, result)
-    end
-
-    it 'should update combination' do
-      params = ['update', '--id=3', '--provisioning-template-id=10', '--hostgroup-id=1', '--puppet-environment-id=1']
-      expected_result = success_result("Template combination updated.\n")
-      api_expects(:template_combinations, :update, 'Update template combination') do |p|
-        p['id'] == '3' &&
-          p['provisioning_template_id'] == '10' &&
-          p['hostgroup_id'] == '1' &&
-          p['environment_id'] == 1 &&
-          p['template_combination'] == { 'environment_id' => 1, 'hostgroup_id' => '1' }
-      end
-
-      result = run_cmd(@cmd + params)
-      assert_cmd(expected_result, result)
-    end
-  end
-end

data/test/functional/test_helper.rb

@@ -1,7 +0,0 @@
-require File.join(File.dirname(__FILE__), '../test_helper')
-
-require 'hammer_cli/testing/command_assertions'
-require 'hammer_cli/testing/output_matchers'
-
-include HammerCLI::Testing::CommandAssertions
-include HammerCLI::Testing::OutputMatchers

data/test/test_helper.rb

@@ -1,29 +0,0 @@
-require 'simplecov'
-require 'pathname'
-
-SimpleCov.use_merging true
-SimpleCov.start do
-  command_name 'MiniTest'
-  add_filter 'test'
-end
-SimpleCov.root Pathname.new(File.dirname(__FILE__) + "../../../")
-
-
-require 'minitest/autorun'
-require 'minitest/spec'
-require "minitest-spec-context"
-require "mocha/minitest"
-
-require 'hammer_cli'
-require 'hammer_cli/testing/command_assertions'
-require 'hammer_cli_foreman/testing/api_expectations'
-FOREMAN_VERSION = Gem::Version.new(ENV['TEST_API_VERSION'] || '3.3')
-
-include HammerCLI::Testing::CommandAssertions
-include HammerCLIForeman::Testing::APIExpectations
-HammerCLI.context[:api_connection].create('foreman') do
-  api_connection({}, FOREMAN_VERSION)
-end
-
-require 'hammer_cli_foreman'
-require 'hammer_cli_foreman_puppet'

data/test/unit/apipie_resource_mock.rb

@@ -1,186 +0,0 @@
-module ResourceMocks
-
-  def self.mock_action_call(resource, action, value, params=:default)
-    response = ApipieBindings::Example.new('GET', '/', '', 200, JSON.dump(value))
-    @mocks ||= {}
-    @mocks[[resource, action]] ||= {}
-    @mocks[[resource, action]][params] = response
-    ApipieBindings::API.any_instance.stubs(:fake_responses).returns(@mocks)
-  end
-
-  def self.clear_mocks
-    @mocks = {}
-    ApipieBindings::API.any_instance.stubs(:fake_responses).returns(@mocks)
-  end
-
-  def self.mock_action_calls(*calls)
-    calls.each do |(resource, action, value, params)|
-      mock_action_call(resource, action, value, (params || :default))
-    end
-  end
-
-  def self.smart_class_parameters_index
-    ResourceMocks.mock_action_call(:smart_class_parameters, :index,
-      { "results" => [ { 'parameter' => 'config', 'id' => '1'} ] })
-  end
-
-  def self.smart_class_parameters_show
-    ResourceMocks.mock_action_call(:smart_class_parameters, :show, { 'smart_class_parameter' => { 'override_value_order' => '', 'environments' => [] }})
-  end
-
-  def self.smart_variables_index
-    ResourceMocks.mock_action_call(:smart_variables, :index,
-      { "results" => [ { 'variable' => 'var', 'id' => '1'} ] })
-  end
-
-  def self.smart_variables_show
-    ResourceMocks.mock_action_call(:smart_variables, :show, { "id" => 1, "override_value_order" => "fqdn" })
-  end
-
-  def self.organizations_index
-    ResourceMocks.mock_action_call(:organizations, :index, {
-     "results" => [
-        {
-                 "label" => "Default_Organization",
-                    "id" => 1,
-                  "name" => "Default_Organization",
-                 "title" => "Default_Organization"
-        }
-      ]})
-  end
-
-  def self.hosts_show
-    ResourceMocks.mock_action_call(:hosts, :show, {
-      "ip" => "192.168.122.51",
-      "ip6" => nil,
-      "environment_id" => 1,
-      "environment_name" => "production",
-      "last_report" => "2016-10-24 12:06:31 UTC",
-      "mac" => "52:54:00:ce:b2:b9",
-      "realm_id" => nil,
-      "realm_name" => nil,
-      "sp_mac" => nil,
-      "sp_ip" => nil,
-      "sp_name" => nil,
-      "domain_id" => 1,
-      "domain_name" => "tstrachota.usersys.redhat.com",
-      "architecture_id" => 1,
-      "architecture_name" => "x86_64",
-      "operatingsystem_id" => 1,
-      "operatingsystem_name" => "CentOS 7.2.1511",
-      "build" => false,
-      "model_id" => 1,
-      "hostgroup_id" => nil,
-      "owner_id" => nil,
-      "owner_type" => nil,
-      "enabled" => true,
-      "managed" => false,
-      "use_image" => nil,
-      "image_file" => "",
-      "uuid" => nil,
-      "compute_resource_id" => nil,
-      "compute_resource_name" => nil,
-      "compute_profile_id" => nil,
-      "compute_profile_name" => nil,
-      "capabilities" => ["build"],
-      "provision_method" => "build",
-      "certname" => "foreman.example.com",
-      "image_id" => nil,
-      "image_name" => nil,
-      "created_at" => "2016-10-24 08:36:43 UTC",
-      "updated_at" => "2016-10-24 12:06:46 UTC",
-      "last_compile" => "2016-10-24 12:06:41 UTC",
-      "global_status" => 0,
-      "global_status_label" => "Warning",
-      "organization_id" => nil,
-      "organization_name" => nil,
-      "location_id" => nil,
-      "location_name" => nil,
-      "puppet_status" => 0,
-      "model_name" => "KVM",
-      "configuration_status" => 0,
-      "configuration_status_label" => "No reports",
-      "name" => "foreman.example.com",
-      "id" => 1,
-      "puppet_proxy_id" => 1,
-      "puppet_proxy_name" => "foreman.example.com",
-      "puppet_ca_proxy_id" => 1,
-      "puppet_ca_proxy_name" => "foreman.example.com",
-      "puppet_proxy" => {
-        "name" => "foreman.example.com",
-        "id" => 1,
-        "url" => "https://foreman.example.com:9090"
-      },
-      "puppet_ca_proxy" => {
-        "name" => "foreman.example.com",
-        "id" => 1,
-        "url" => "https://foreman.example.com:9090"
-      },
-      "hostgroup_name" => nil,
-      "hostgroup_title" => nil,
-      "parameters" => [],
-      "all_parameters" => [],
-      "interfaces" => [{
-        "id" => 1,
-        "name" => "foreman.example.com",
-        "ip" => "192.168.122.51",
-        "mac" => "52:54:00:ce:b2:b9",
-        "identifier" => "eth0",
-        "primary" => true,
-        "provision" => true,
-        "type" => "interface"
-      },
-      {
-        "id" => 2,
-        "name" => nil,
-        "ip" => "10.34.130.105",
-        "mac" => "52:54:00:f5:1b:57",
-        "identifier" => "eth1",
-        "primary" => false,
-        "provision" => false,
-        "type" => "interface"
-      }],
-      "puppetclasses" => [],
-      "config_groups" => [],
-      "all_puppetclasses" => []
-    })
-  end
-
-
-  def self.organizations_show
-    ResourceMocks.mock_action_calls(
-      [:organizations, :index, [{ "id" => 2, "name" => "ACME" }]],
-      [:organizations, :show, { "id" => 2, "name" => "ACME" }]
-      )
-  end
-
-  def self.locations_index
-    ResourceMocks.mock_action_call(:locations, :index, {
-      "results" => [
-        {
-              "ancestry" => nil,
-            "created_at" => "2014-07-17T17:21:49+02:00",
-            "updated_at" => "2015-06-17T13:18:10+02:00",
-                    "id" => 2,
-                  "name" => "Default_Location",
-                 "title" => "Default_Location"
-        }
-      ]})
-  end
-
-  def self.locations_show
-    ResourceMocks.mock_action_calls(
-      [:locations, :index, [{ "id" => 2, "name" => "Rack" }]],
-      [:locations, :show, { "id" => 2, "name" => "Rack" }]
-      )
-  end
-
-  def self.config_groups_index
-    ResourceMocks.mock_action_call(:config_groups, :index, [{
-      id: 15,
-      name: "test config group",
-      puppetclasses: [ { name: "My puppetclass" } ]
-    }])
-  end
-
-end

data/test/unit/config_group_test.rb

@@ -1,73 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
-
-require 'hammer_cli_foreman_puppet/config_group'
-
-describe HammerCLIForemanPuppet::ConfigGroup do
-  include CommandTestHelper
-
-  context "ListCommand" do
-    let(:cmd) { HammerCLIForemanPuppet::ConfigGroup::ListCommand.new("", ctx) }
-
-    before :each do
-      ResourceMocks.config_groups_index
-    end
-
-    context "parameters" do
-      it_should_accept "no arguments"
-      it_should_accept_search_params
-    end
-
-    context "output" do
-      let(:expected_record_count) { cmd.resource.call(:index).length }
-      it_should_print_n_records
-      it_should_print_columns ["ID", "Name"]
-    end
-  end
-
-  context "InfoCommand" do
-    let(:cmd) { HammerCLIForemanPuppet::ConfigGroup::InfoCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "id", ["--id=1"]
-      it_should_accept "name", ["--name=group_x"]
-    end
-
-    context "output" do
-      with_params ["--id=1"] do
-        it_should_print_n_records 1
-        it_should_print_column "Name"
-        it_should_print_column "ID"
-        it_should_print_column "Puppetclasses"
-      end
-    end
-  end
-
-  context "CreateCommand" do
-    let(:cmd) { HammerCLIForemanPuppet::ConfigGroup::CreateCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept 'name, puppetclass ids',
-                       %w[--name=first_group --puppet-class-ids=1,2]
-    end
-  end
-
-  context "DeleteCommand" do
-    let(:cmd) { HammerCLIForemanPuppet::ConfigGroup::DeleteCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "name", ["--name=group_x"]
-      it_should_accept "id", ["--id=1"]
-    end
-  end
-
-  context "UpdateCommand" do
-
-    let(:cmd) { HammerCLIForemanPuppet::ConfigGroup::UpdateCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "name", ["--name=group_x"]
-      it_should_accept "id", ["--id=1"]
-    end
-  end
-end

data/test/unit/helpers/command.rb

@@ -1,163 +0,0 @@
-require File.join(File.dirname(__FILE__), '../test_output_adapter')
-
-class IdResolverTestProxy
-
-  attr_reader :api
-
-  def initialize(original_resolver)
-    @original_resolver = original_resolver
-    define_id_finders
-  end
-
-  def scoped_options(scope, options, mode = nil)
-    @original_resolver.scoped_options(scope, options, mode)
-  end
-
-  def searchables(resource)
-    @original_resolver.searchables(resource)
-  end
-
-  def puppet_environment_id(options)
-    environment_id(options)
-  end
-
-  def puppet_environment_ids(options)
-    environment_ids(options)
-  end
-
-  protected
-
-  def define_id_finders
-    @original_resolver.api.resources.each do |resource|
-      method_name = "#{resource.singular_name}_id"
-      self.class.send(:define_method, method_name) do |options|
-        value = options[HammerCLI.option_accessor_name("id")]
-        value ||= HammerCLI::NilValue if searchables(resource).any? do |s|
-          options[HammerCLI.option_accessor_name(s.name)] == HammerCLI::NilValue
-        end
-        value ||= 1 if searchables(resource).any? do |s|
-          !options[HammerCLI.option_accessor_name(s.name)].nil?
-        end
-        value
-      end
-
-      method_name = "#{resource.singular_name}_ids"
-      self.class.send(:define_method, method_name) do |options|
-        options["option_#{resource.singular_name}_ids"].nil? ? nil : [1]
-      end
-    end
-  end
-
-end
-
-
-module CommandTestHelper
-
-  def self.included(base)
-    base.extend(ClassMethods)
-
-    base.before :each do
-      resolver = cmd.resolver
-      cmd.stubs(:resolver).returns(IdResolverTestProxy.new(resolver))
-    end
-  end
-
-  def count_records(data)
-    HammerCLIForeman.collection_to_common_format(data['results']).count
-  end
-
-  module ClassMethods
-
-    def with_params(params, &block)
-      context "with params "+params.to_s do
-        let(:with_params) { params }
-        self.instance_eval &block
-      end
-    end
-
-    def it_should_call_action(action, params, headers={})
-      it "should call action "+action.to_s do
-        arguments ||= respond_to?(:with_params) ? with_params : []
-        ApipieBindings::API.any_instance.expects(:call).with() do |r,a,p,h,o|
-          (r == cmd.resource.name && a == action && p == params && h == headers)
-        end
-        cmd.run(arguments)
-      end
-    end
-
-    def it_should_call_action_and_test_params(action, &block)
-      it "should call action "+action.to_s do
-        arguments ||= respond_to?(:with_params) ? with_params : []
-        ApipieBindings::API.any_instance.expects(:call).with() do |r,a,p,h,o|
-          (r == cmd.resource.name && a == action && yield(p))
-        end
-        cmd.run(arguments)
-      end
-    end
-
-    def it_should_fail_with(message, arguments=[])
-      it "should fail with " + message.to_s do
-        _(cmd.run(arguments)).must_equal HammerCLI::EX_USAGE
-      end
-    end
-
-    def it_should_accept(message, arguments=[])
-      it "should accept " + message.to_s do
-        out, err = capture_io do
-          _(cmd.run(arguments)).must_equal HammerCLI::EX_OK
-        end
-      end
-    end
-
-    def it_should_output(message, adapter=:base)
-      it "should output '" + message.to_s + "'" do
-        arguments ||= respond_to?(:with_params) ? with_params : []
-        cmd.stubs(:context).returns(ctx.update(:adapter => adapter))
-        out, err = capture_io do
-          cmd.run(arguments)
-        end
-        _(out).must_include message
-      end
-    end
-
-    def it_should_print_column(column_name, arguments=nil)
-      it "should print column " + column_name do
-        arguments ||= respond_to?(:with_params) ? with_params : []
-
-        cmd.stubs(:context).returns(ctx.update(:adapter => :test))
-        out, err = capture_io do
-          cmd.run(arguments)
-        end
-
-        _(out.split("\n")[0]).must_match /.*##{column_name}#.*/
-      end
-    end
-
-    def it_should_print_columns(column_names, arguments=nil)
-      column_names.each do |name|
-        it_should_print_column name, arguments
-      end
-    end
-
-    def it_should_print_n_records(count=nil, arguments=nil)
-      it "should print correct count of records" do
-        arguments ||= respond_to?(:with_params) ? with_params : []
-
-        cmd.stubs(:context).returns(ctx.update(:adapter => :test))
-        count ||= expected_record_count rescue 0
-        out, err = capture_io do
-          cmd.run(arguments)
-        end
-        _(out.split(/\n/).length).must_equal count+1 # plus 1 for line with column headers
-      end
-    end
-
-    def it_should_accept_search_params
-      it_should_accept "search", ["--search=some_search"]
-      it_should_accept "per page", ["--per-page=1"]
-      it_should_accept "page", ["--page=2"]
-      it_should_accept "order", ["--order=order"]
-    end
-  end
-
-end