hammer_cli_foreman_puppet 0.0.1

data/test/functional/template_test.rb

@@ -0,0 +1,38 @@
+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

@@ -0,0 +1,7 @@
+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

@@ -0,0 +1,29 @@
+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.0')
+
+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

@@ -0,0 +1,186 @@
+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

@@ -0,0 +1,81 @@
+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
+      it_should_accept 'organization', ['--organization-id=1']
+      it_should_accept 'location', ['--location-id=1']
+    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"]
+      it_should_accept 'organization', %w[--id=1 --organization-id=1]
+      it_should_accept 'location', %w[--id=1 --location-id=1]
+    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, location, organization',
+                       %w[--name=first_group --puppet-class-ids=1,2 --location-id=1 --organization-id=1]
+    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"]
+      it_should_accept 'organization', %w[--id=1 --organization-id=1]
+      it_should_accept 'location', %w[--id=1 --location-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"]
+      it_should_accept 'organization', %w[--id=1 --organization-id=1]
+      it_should_accept 'location', %w[--id=1 --location-id=1]
+    end
+  end
+end

data/test/unit/helpers/command.rb

@@ -0,0 +1,163 @@
+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