RubyGems - hammer_cli_foreman - Versions diffs - 2.4.0 → 3.1.0 - Mend

hammer_cli_foreman 2.4.0 → 3.1.0

Files changed (117) hide show

checksums.yaml +4 -4
data/doc/release_notes.md +40 -0
data/lib/hammer_cli_foreman/architecture.rb +5 -5
data/lib/hammer_cli_foreman/associating_commands.rb +0 -24
data/lib/hammer_cli_foreman/bookmark.rb +5 -5
data/lib/hammer_cli_foreman/combination.rb +3 -13
data/lib/hammer_cli_foreman/command_extensions/ping.rb +21 -2
data/lib/hammer_cli_foreman/command_extensions/subnet.rb +25 -10
data/lib/hammer_cli_foreman/command_extensions/update_common.rb +14 -0
data/lib/hammer_cli_foreman/command_extensions/user.rb +9 -5
data/lib/hammer_cli_foreman/command_extensions.rb +1 -2
data/lib/hammer_cli_foreman/commands.rb +17 -11
data/lib/hammer_cli_foreman/compute_attribute.rb +1 -1
data/lib/hammer_cli_foreman/compute_profile.rb +5 -5
data/lib/hammer_cli_foreman/compute_resource/ovirt.rb +2 -1
data/lib/hammer_cli_foreman/compute_resource.rb +23 -0
data/lib/hammer_cli_foreman/config_report.rb +2 -0
data/lib/hammer_cli_foreman/filter.rb +2 -2
data/lib/hammer_cli_foreman/host.rb +1 -47
data/lib/hammer_cli_foreman/hostgroup.rb +14 -58
data/lib/hammer_cli_foreman/hosts/common_update_options.rb +10 -31
data/lib/hammer_cli_foreman/id_resolver.rb +7 -68
data/lib/hammer_cli_foreman/location.rb +0 -6
data/lib/hammer_cli_foreman/mail_notification.rb +2 -2
data/lib/hammer_cli_foreman/model.rb +5 -5
data/lib/hammer_cli_foreman/operating_system.rb +10 -10
data/lib/hammer_cli_foreman/option_builders.rb +84 -52
data/lib/hammer_cli_foreman/option_sources/id_params.rb +41 -11
data/lib/hammer_cli_foreman/option_sources/ids_params.rb +39 -7
data/lib/hammer_cli_foreman/option_sources.rb +0 -1
data/lib/hammer_cli_foreman/organization.rb +0 -6
data/lib/hammer_cli_foreman/ping.rb +6 -1
data/lib/hammer_cli_foreman/references.rb +0 -16
data/lib/hammer_cli_foreman/registration.rb +18 -0
data/lib/hammer_cli_foreman/settings.rb +3 -3
data/lib/hammer_cli_foreman/smart_proxy.rb +7 -57
data/lib/hammer_cli_foreman/template.rb +3 -15
data/lib/hammer_cli_foreman/user.rb +4 -4
data/lib/hammer_cli_foreman/usergroup.rb +5 -5
data/lib/hammer_cli_foreman/version.rb +1 -1
data/lib/hammer_cli_foreman.rb +3 -21
data/lib/minitest/coverage_reporter.rb +1 -1
data/locale/ca/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/de/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/en/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/en_GB/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/es/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/fr/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/it/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/ja/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/ko/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/pt_BR/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/ru/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/zh_CN/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/locale/zh_TW/LC_MESSAGES/hammer-cli-foreman.mo +0 -0
data/test/data/2.4/foreman_api.json +1 -0
data/test/data/2.5/foreman_api.json +1 -0
data/test/data/3.1/foreman_api.json +1 -0
data/test/functional/architecture_test.rb +37 -5
data/test/functional/bookmark_test.rb +5 -2
data/test/functional/compute_attribute_test.rb +20 -20
data/test/functional/compute_profile_test.rb +14 -1
data/test/functional/compute_resource_test.rb +37 -0
data/test/functional/filter_test.rb +2 -1
data/test/functional/host_test.rb +65 -11
data/test/functional/hostgroup/create_test.rb +11 -117
data/test/functional/hostgroup/update_test.rb +11 -79
data/test/functional/http_proxy_test.rb +12 -0
data/test/functional/location_test.rb +13 -3
data/test/functional/mail_notification_test.rb +3 -1
data/test/functional/media_test.rb +11 -0
data/test/functional/model_test.rb +3 -1
data/test/functional/operating_system_test.rb +3 -1
data/test/functional/personal_access_token_test.rb +4 -4
data/test/functional/ping_test.rb +33 -0
data/test/functional/realm_test.rb +11 -0
data/test/functional/registration_test.rb +8 -0
data/test/functional/report_template_test.rb +11 -0
data/test/functional/settings_test.rb +4 -4
data/test/functional/template_test.rb +87 -48
data/test/functional/user_mail_notification_test.rb +3 -3
data/test/functional/user_test.rb +11 -0
data/test/functional/usergroup_test.rb +3 -1
data/test/test_helper.rb +1 -1
data/test/unit/apipie_resource_mock.rb +1 -39
data/test/unit/architecture_test.rb +10 -10
data/test/unit/bookmark_test.rb +10 -14
data/test/unit/commands_test.rb +1 -2
data/test/unit/compute_profile_test.rb +10 -10
data/test/unit/config_report_test.rb +1 -0
data/test/unit/helpers/command.rb +0 -8
data/test/unit/host_test.rb +14 -61
data/test/unit/hostgroup_test.rb +7 -26
data/test/unit/id_resolver_test.rb +0 -28
data/test/unit/mail_notification_test.rb +4 -4
data/test/unit/model_test.rb +47 -47
data/test/unit/operating_system_test.rb +10 -16
data/test/unit/option_builders_test.rb +88 -83
data/test/unit/settings_test.rb +4 -4
data/test/unit/smart_proxy_test.rb +0 -12
data/test/unit/usergroup_test.rb +10 -10
metadata +115 -128
data/lib/hammer_cli_foreman/command_extensions/puppet_environment.rb +0 -29
data/lib/hammer_cli_foreman/command_extensions/puppet_environments.rb +0 -29
data/lib/hammer_cli_foreman/config_group.rb +0 -45
data/lib/hammer_cli_foreman/option_sources/puppet_environment_params.rb +0 -59
data/lib/hammer_cli_foreman/puppet_class.rb +0 -61
data/lib/hammer_cli_foreman/puppet_environment.rb +0 -65
data/lib/hammer_cli_foreman/smart_class_parameter.rb +0 -189
data/test/functional/config_group_test.rb +0 -50
data/test/functional/proxy_test.rb +0 -86
data/test/functional/smart_class_parameter_test.rb +0 -97
data/test/reports/TEST-Minitest-Result.xml +0 -4344
data/test/unit/config_group_test.rb +0 -82
data/test/unit/puppet_class_test.rb +0 -72
data/test/unit/puppet_environment_test.rb +0 -114
data/test/unit/smart_class_parameter_test.rb +0 -113

data/test/unit/config_group_test.rb DELETED Viewed

@@ -1,82 +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
-      it_should_fail_with 'organization param', ['--organization-id=1']
-      it_should_fail_with 'location param', ['--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) { HammerCLIForeman::ConfigGroup::InfoCommand.new("", ctx) }
-    context "parameters" do
-      it_should_accept "id", ["--id=1"]
-      it_should_accept "name", ["--name=group_x"]
-      it_should_fail_with 'organization param', ['--organization-id=1']
-      it_should_fail_with 'location param', ['--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) { HammerCLIForeman::ConfigGroup::CreateCommand.new("", ctx) }
-    context "parameters" do
-      it_should_accept "name, puppetclass ids", ["--name=first_group", "--puppet-class-ids=1,2"]
-      it_should_fail_with 'organization param', ['--organization-id=1']
-      it_should_fail_with 'location param', ['--location-id=1']
-    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"]
-      it_should_fail_with 'organization param', ['--organization-id=1']
-      it_should_fail_with 'location param', ['--location-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"]
-      it_should_fail_with 'organization param', ['--organization-id=1']
-      it_should_fail_with 'location param', ['--location-id=1']
-    end
-  end
-end

data/test/unit/puppet_class_test.rb DELETED Viewed

@@ -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 DELETED Viewed

@@ -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 DELETED Viewed

@@ -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