hammer_cli_foreman 2.3.2 → 3.0.0

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) { [
-    '--puppet-class', puppet_class['name'],
-    '--smart-class-parameter', parameter['name'],
-    '--match', match
-  ] }
-
-  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/unit/config_group_test.rb

@@ -1,72 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
-
-require 'hammer_cli_foreman/config_group'
-
-describe HammerCLIForeman::ConfigGroup do
-  include CommandTestHelper
-
-  context "ListCommand" do
-    let(:cmd) { HammerCLIForeman::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) { HammerCLIForeman::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) { HammerCLIForeman::ConfigGroup::CreateCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "name, puppetclass ids", ["--name=first_group", "--puppet-class-ids=1,2"]
-    end
-  end
-
-  context "DeleteCommand" do
-    let(:cmd) { HammerCLIForeman::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) { HammerCLIForeman::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/puppet_class_test.rb

@@ -1,72 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
-
-require 'hammer_cli_foreman/puppet_class'
-
-describe HammerCLIForeman::PuppetClass do
-
-  include CommandTestHelper
-
-  context "ListCommand" do
-    before do
-      ResourceMocks.mock_action_call(:puppetclasses, :index, {})
-    end
-
-    let(:cmd) { HammerCLIForeman::PuppetClass::ListCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "no arguments"
-      # FIXME: the command should accept search parameters in future
-      # it_should_accept_search_params
-    end
-
-    context "output" do
-      let(:expected_record_count) do
-        # data are retuned in specific format
-        HammerCLIForeman.collection_to_common_format(cmd.resource.call(:index)).first.keys.count
-      end
-
-      it_should_print_n_records
-      it_should_print_column "Id"
-      it_should_print_column "Name"
-    end
-
-  end
-
-
-  context "InfoCommand" do
-
-    let(:cmd) { HammerCLIForeman::PuppetClass::InfoCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "id", ["--id=1"]
-      # it_should_fail_with "no arguments" # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-    context "output" do
-      with_params ["--id=1"] do
-        it_should_print_n_records 1
-        it_should_print_column "Id"
-        it_should_print_column "Name"
-      end
-    end
-
-  end
-
-  context "SCParamsCommand" do
-
-    before :each do
-      ResourceMocks.smart_class_parameters_index
-    end
-
-    let(:cmd) { HammerCLIForeman::PuppetClass::SCParamsCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "puppet-class", ["--puppet-class=cls"]
-      it_should_accept "puppet-class-id", ["--puppet-class-id=1"]
-      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-  end
-
-end

data/test/unit/puppet_environment_test.rb

@@ -1,114 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
-
-require 'hammer_cli_foreman/puppet_environment'
-
-describe HammerCLIForeman::PuppetEnvironment do
-
-  include CommandTestHelper
-
-  context "ListCommand" do
-    before do
-      ResourceMocks.mock_action_call(:environments, :index, [])
-    end
-
-    let(:cmd) { HammerCLIForeman::PuppetEnvironment::ListCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "no arguments"
-      it_should_accept_search_params
-    end
-
-    context "output" do
-      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
-
-      it_should_print_n_records
-      it_should_print_column "Name"
-      it_should_print_column "Id"
-    end
-
-  end
-
-
-  context "InfoCommand" do
-
-    let(:cmd) { HammerCLIForeman::PuppetEnvironment::InfoCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "id", ["--id=1"]
-      it_should_accept "name", ["--name=env"]
-      # it_should_fail_with "no arguments"
-      # TODO: temporarily disabled, parameters are checked in the id resolver
-    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 "Created at"
-        it_should_print_column "Updated at"
-      end
-    end
-
-  end
-
-
-  context "CreateCommand" do
-
-    let(:cmd) { HammerCLIForeman::PuppetEnvironment::CreateCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "name", ["--name=env"]
-      # it_should_fail_with "name missing", []
-      # TODO: temporarily disabled, parameters are checked by the api
-    end
-
-  end
-
-
-  context "DeleteCommand" do
-
-    let(:cmd) { HammerCLIForeman::PuppetEnvironment::DeleteCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "name", ["--name=env"]
-      it_should_accept "id", ["--id=1"]
-      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-  end
-
-
-  context "UpdateCommand" do
-
-    let(:cmd) { HammerCLIForeman::PuppetEnvironment::UpdateCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "name", ["--name=env", "--new-name=env2"]
-      it_should_accept "id", ["--id=1", "--new-name=env2"]
-      # it_should_fail_with "no params", [] # TODO: temporarily disabled, parameters are checked in the id resolver
-      # it_should_fail_with "name or id missing", ["--new-name=env2"] # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-  end
-
-  context "SCParamsCommand" do
-
-    before :each do
-      ResourceMocks.smart_class_parameters_index
-    end
-
-    let(:cmd) { HammerCLIForeman::PuppetEnvironment::SCParamsCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "environment", ["--environment=env"]
-      it_should_accept "environment-id", ["--environment-id=1"]
-      it_should_accept "puppet-environment", ["--puppet-environment=env"]
-      it_should_accept "puppet-environment-id", ["--puppet-environment-id=1"]
-      # it_should_fail_with "name or id missing", [] # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-  end
-
-end

data/test/unit/smart_class_parameter_test.rb

@@ -1,113 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-require File.join(File.dirname(__FILE__), 'apipie_resource_mock')
-
-require 'hammer_cli_foreman/smart_class_parameter'
-
-describe HammerCLIForeman::SmartClassParameter do
-
-  include CommandTestHelper
-
-  context "ListCommand" do
-
-    before :each do
-      ResourceMocks.smart_class_parameters_index
-    end
-
-    let(:cmd) { HammerCLIForeman::SmartClassParameter::ListCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "no arguments"
-      it_should_accept "hostgroup id", ["--hostgroup-id=1"]
-      it_should_accept "host id", ["--host-id=1"]
-      it_should_accept "environment id", ["--environment-id=1"]
-      it_should_accept "puppet-class-id", ["--puppet-class-id=1"]
-      it_should_accept_search_params
-    end
-
-    context "output" do
-      let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
-
-      it_should_print_n_records
-      it_should_print_column "Id"
-      it_should_print_column "Class Id"
-      it_should_print_column "Puppet class"
-      it_should_print_column "Parameter"
-      it_should_print_column "Default Value"
-      it_should_print_column "Override"
-    end
-
-  end
-
-
-  context "InfoCommand" do
-
-    before :each do
-      ResourceMocks.smart_class_parameters_show
-    end
-
-    let(:cmd) { HammerCLIForeman::SmartClassParameter::InfoCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "id", ["--id=1"]
-      it_should_accept "name, puppet-class", ["--name=par", "--puppet-class=ntp"]
-      it_should_fail_with "name", ["--name=par"]
-      # it_should_fail_with "no arguments"
-      # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-    context "output" do
-      with_params ["--id=1"] do
-        it_should_print_n_records 1
-        it_should_print_columns ["Description", "Type", "Omit", "Required"]
-      end
-    end
-  end
-
-
-  context "UpdateCommand" do
-
-    let(:cmd) { HammerCLIForeman::SmartClassParameter::UpdateCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "id", ["--id=1"]
-      it_should_accept "name, puppet-class", ["--name=par", "--puppet-class=ntp"]
-      it_should_fail_with "name", ["--name=par"]
-      it_should_accept "override", ["--id=1","--override=true"]
-      it_should_accept "description", ["--id=1","--description=descr"]
-      it_should_accept "default-value", ["--id=1","--default-value=1"]
-      it_should_accept "path  ", ["--id=1","--path=path"]
-      it_should_accept "validator-type", ["--id=1","--validator-type=list"]
-      it_should_accept "validator-rule ", ["--id=1","--validator-rule=''"]
-      it_should_accept "override-value-order", ["--id=1","--override-value-order=fqdn"]
-      it_should_accept "parameter-type ", ["--id=1","--parameter-type=string"]
-      it_should_accept "required", ["--id=1","--required=true"]
-
-      # it_should_fail_with "no params", []
-      # TODO: temporarily disabled, parameters are checked in the id resolver
-    end
-
-  end
-
-  context "AddMatcherCommand" do
-
-    let(:cmd) { HammerCLIForeman::SmartClassParameter::AddMatcherCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "smart-class-parametr-id, match, value", ["--smart-class-parameter-id=1", "--match='domain=my.lan'", "--value=1"]
-      it_should_accept "smart-class-parameter, puppet-class, match, value", ["--smart-class-parameter=par", "--puppet-class=ntp", "--match='domain=my.lan'", "--value=1"]
-      it_should_fail_with "smart-class-parameter, match, value", ["--smart-class-parameter=par", "--match='domain=my.lan'", "--value=1"]
-    end
-  end
-
-  context "RemoveMatcherCommand" do
-
-    let(:cmd) { HammerCLIForeman::SmartClassParameter::RemoveMatcherCommand.new("", ctx) }
-
-    context "parameters" do
-      it_should_accept "smart-class-parametr-id, id", ["--smart-class-parameter-id=1", "--id=1"]
-      it_should_accept "smart-class-parameter, puppet-class, id", ["--smart-class-parameter=par", "--puppet-class=ntp", "--id=1"]
-      it_should_fail_with "smart-class-parameter, id", ["--smart-class-parameter=par", "--id=1"]
-    end
-  end
-
-end