hammer_cli_foreman 2.5.1 → 3.2.0

data/lib/hammer_cli_foreman/command_extensions/puppet_environment.rb

@@ -1,29 +0,0 @@
-module HammerCLIForeman
-  module CommandExtensions
-    class PuppetEnvironment < HammerCLI::CommandExtensions
-      # Remove when support of --environment options is ended.
-      option_family(
-        aliased_resource: 'environment',
-        description: _('Puppet environment'),
-        deprecation: _("Use %s instead") % '--puppet-environment[-id]',
-        deprecated: { '--environment' => _("Use %s instead") % '--puppet-environment[-id]',
-                      '--environment-id' => _("Use %s instead") % '--puppet-environment[-id]'}
-      ) do
-        parent '--environment-id', 'ENVIRONMENT_ID', _(''),
-               format: HammerCLI::Options::Normalizers::Number.new,
-               attribute_name: :option_environment_id
-        child '--environment', 'ENVIRONMENT_NAME', _('Environment name'),
-              attribute_name: :option_environment_name
-      end
-
-      option_sources do |sources, command|
-        sources.find_by_name('IdResolution').insert_relative(
-          :after,
-          'IdParams',
-          HammerCLIForeman::OptionSources::PuppetEnvironmentParams.new(command)
-        )
-        sources
-      end
-    end
-  end
-end

data/lib/hammer_cli_foreman/command_extensions/puppet_environments.rb

@@ -1,29 +0,0 @@
-module HammerCLIForeman
-  module CommandExtensions
-    class PuppetEnvironments < HammerCLI::CommandExtensions
-      # Remove when support of --environments options is ended.
-      option_family(
-        aliased_resource: 'environment',
-        description: _('Puppet environments'),
-        deprecation: _("Use %s instead") % '--puppet-environment[s|-ids]',
-        deprecated: { '--environments' => _("Use %s instead") % '--puppet-environment[s|-ids]',
-                      '--environment-ids' => _("Use %s instead") % '--puppet-environment[s|-ids]' }
-      ) do
-        parent '--environment-ids', 'ENVIRONMENT_IDS', _('Environment IDs'),
-               format: HammerCLI::Options::Normalizers::List.new,
-               attribute_name: :option_environment_ids
-        child '--environments', 'ENVIRONMENT_NAMES', _(''),
-              attribute_name: :option_environment_names
-      end
-
-      option_sources do |sources, command|
-        sources.find_by_name('IdResolution').insert_relative(
-          :after,
-          'IdsParams',
-          HammerCLIForeman::OptionSources::PuppetEnvironmentParams.new(command)
-        )
-        sources
-      end
-    end
-  end
-end

data/lib/hammer_cli_foreman/config_group.rb

@@ -1,45 +0,0 @@
-module HammerCLIForeman
-  class ConfigGroup < HammerCLIForeman::Command
-    resource :config_groups
-
-    class ListCommand < HammerCLIForeman::ListCommand
-      output do
-        field :id, _("ID")
-        field :name, _("Name")
-      end
-
-      build_options expand: { except: %i[organizations locations] }, without: %i[organization_id location_id]
-    end
-
-    class InfoCommand < HammerCLIForeman::InfoCommand
-      output ListCommand.output_definition do
-        HammerCLIForeman::References.puppetclasses(self)
-      end
-
-      build_options expand: { except: %i[organizations locations] }, without: %i[organization_id location_id]
-    end
-
-    class CreateCommand < HammerCLIForeman::CreateCommand
-      success_message _("Config group created.")
-      failure_message _("Could not create the config group")
-
-      build_options expand: { except: %i[organizations locations] }, without: %i[organization_id location_id]
-    end
-
-    class UpdateCommand < HammerCLIForeman::UpdateCommand
-      success_message _("Config group updated.")
-      failure_message _("Could not update the config group")
-
-      build_options expand: { except: %i[organizations locations] }, without: %i[organization_id location_id]
-    end
-
-    class DeleteCommand < HammerCLIForeman::DeleteCommand
-      success_message _("Config group has been deleted.")
-      failure_message _("Could not delete the config group")
-
-      build_options expand: { except: %i[organizations locations] }, without: %i[organization_id location_id]
-    end
-
-    autoload_subcommands
-  end
-end

data/lib/hammer_cli_foreman/option_sources/puppet_environment_params.rb

@@ -1,59 +0,0 @@
-module HammerCLIForeman
-  module OptionSources
-    class PuppetEnvironmentParams < HammerCLI::Options::Sources::Base
-      def initialize(command)
-        @command = command
-      end
-
-      def get_options(_defined_options, result)
-        if result['option_environment_id'].nil? &&
-           (@command.respond_to?(:option_environment_id) ||
-           @command.respond_to?(:option_environment_name))
-          put_puppet_environment_id(result)
-        end
-        if result['option_environment_ids'].nil? &&
-           (@command.respond_to?(:option_environment_ids) ||
-           @command.respond_to?(:option_environment_names))
-          put_puppet_environment_ids(result)
-        end
-        result
-      end
-
-      private
-
-      def put_puppet_environment_id(result)
-        if result['option_environment_name'].nil?
-          if @command.option_environment_id
-            result['option_environment_id'] = @command.option_environment_id
-          elsif @command.option_environment_name
-            result['option_environment_name'] = @command.option_environment_name
-            result['option_environment_id'] = @command.resolver.puppet_environment_id(
-              @command.resolver.scoped_options('environment', result, :single)
-            )
-          end
-        else
-          result['option_environment_id'] = @command.resolver.puppet_environment_id(
-            @command.resolver.scoped_options('environment', result, :single)
-          )
-        end
-      end
-
-      def put_puppet_environment_ids(result)
-        if result['option_environment_names'].nil?
-          if @command.option_environment_ids
-            result['option_environment_ids'] = @command.option_environment_ids
-          elsif @command.option_environment_names
-            result['option_environment_names'] = @command.option_environment_names
-            result['option_environment_ids'] = @command.resolver.puppet_environment_ids(
-              @command.resolver.scoped_options('environment', result, :multi)
-            )
-          end
-        else
-          result['option_environment_ids'] = @command.resolver.puppet_environment_ids(
-            @command.resolver.scoped_options('environment', result, :multi)
-          )
-        end
-      end
-    end
-  end
-end

data/lib/hammer_cli_foreman/puppet_class.rb

@@ -1,61 +0,0 @@
-require 'hammer_cli_foreman/smart_class_parameter'
-
-module HammerCLIForeman
-
-  class PuppetClass < HammerCLIForeman::Command
-
-    resource :puppetclasses
-
-    class ListCommand < HammerCLIForeman::ListCommand
-
-      output do
-        field :id, _("Id")
-        field :name, _("Name")
-      end
-
-      def send_request
-        self.class.unhash_classes(super)
-      end
-
-      def self.unhash_classes(classes)
-        clss = classes.first.inject([]) { |list, (pp_module, pp_module_classes)| list + pp_module_classes }
-
-        HammerCLI::Output::RecordCollection.new(clss, :meta => classes.meta)
-
-      end
-
-      build_options
-
-      extend_with(HammerCLIForeman::CommandExtensions::PuppetEnvironment.new)
-    end
-
-
-    class InfoCommand < HammerCLIForeman::InfoCommand
-      output ListCommand.output_definition do
-        collection :smart_class_parameters, _("Smart class parameters"), :numbered => false do
-          custom_field Fields::Reference, :name_key => :parameter
-        end
-        HammerCLIForeman::References.hostgroups(self)
-        HammerCLIForeman::References.environments(self)
-        HammerCLIForeman::References.parameters(self)
-      end
-
-      build_options
-
-      extend_with(HammerCLIForeman::CommandExtensions::PuppetEnvironment.new)
-    end
-
-
-    class SCParamsCommand < HammerCLIForeman::SmartClassParametersBriefList
-      build_options_for :puppetclasses
-
-      def validate_options
-        super
-        validator.any(:option_puppetclass_name, :option_puppetclass_id).required
-      end
-    end
-
-    autoload_subcommands
-  end
-
-end

data/lib/hammer_cli_foreman/puppet_environment.rb

@@ -1,65 +0,0 @@
-require 'hammer_cli'
-require 'hammer_cli_foreman/smart_class_parameter'
-
-module HammerCLIForeman
-
-  class PuppetEnvironment < HammerCLIForeman::Command
-
-    resource :environments
-
-    class ListCommand < HammerCLIForeman::ListCommand
-
-      output do
-        field :id, _("Id")
-        field :name, _("Name")
-      end
-
-      build_options
-    end
-
-
-    class InfoCommand < HammerCLIForeman::InfoCommand
-      output ListCommand.output_definition do
-        HammerCLIForeman::References.puppetclasses(self)
-        HammerCLIForeman::References.taxonomies(self)
-        HammerCLIForeman::References.timestamps(self)
-      end
-
-      build_options
-    end
-
-
-    class CreateCommand < HammerCLIForeman::CreateCommand
-      success_message _("Environment created.")
-      failure_message _("Could not create the environment")
-
-      build_options
-    end
-
-
-    class UpdateCommand < HammerCLIForeman::UpdateCommand
-      success_message _("Environment updated.")
-      failure_message _("Could not update the environment")
-
-      build_options
-    end
-
-
-    class DeleteCommand < HammerCLIForeman::DeleteCommand
-      success_message _("Environment deleted.")
-      failure_message _("Could not delete the environment")
-
-      build_options
-    end
-
-    class SCParamsCommand < HammerCLIForeman::SmartClassParametersList
-      build_options_for :environments
-
-      extend_with(HammerCLIForeman::CommandExtensions::PuppetEnvironment.new)
-    end
-
-
-    autoload_subcommands
-  end
-
-end

data/lib/hammer_cli_foreman/smart_class_parameter.rb

@@ -1,189 +0,0 @@
-module HammerCLIForeman
-
-  class SmartClassParametersBriefList < HammerCLIForeman::ListCommand
-    resource :smart_class_parameters, :index
-    command_name 'sc-params'
-
-    output do
-      field :id, _("Id")
-
-      field :parameter, _("Parameter")
-      field :default_value, _("Default Value")
-      field :override, _("Override")
-    end
-
-    def send_request
-      res = super
-      # FIXME: API returns doubled records, probably just if filtered by puppetclasses
-      # it seems group by environment is missing
-      # having the uniq to fix that
-      HammerCLI::Output::RecordCollection.new(res.uniq, :meta => res.meta)
-    end
-
-    def self.build_options_for(resource)
-      options = {}
-      options[:without] = [:host_id, :puppetclass_id, :environment_id, :hostgroup_id]
-      options[:expand] = {}
-      options[:expand][:except] = ([:hosts, :puppetclasses, :environments, :hostgroups] - [resource])
-      build_options(options)
-    end
-  end
-
-  class SmartClassParametersList < SmartClassParametersBriefList
-
-    output do
-      field :puppetclass_name, _("Puppet class")
-      field :puppetclass_id, _("Class Id"), Fields::Id
-    end
-  end
-
-  class SmartClassParameter < HammerCLIForeman::Command
-
-    resource :smart_class_parameters
-
-    class ListCommand < HammerCLIForeman::ListCommand
-
-      output SmartClassParametersList.output_definition
-
-      def extend_data(res)
-        res['parameter_type'] ||= 'string'
-        res
-      end
-
-      build_options
-
-      extend_with(HammerCLIForeman::CommandExtensions::PuppetEnvironment.new)
-    end
-
-    class InfoCommand < HammerCLIForeman::InfoCommand
-
-      output ListCommand.output_definition do
-        field :description, _("Description")
-        field :parameter_type, _("Type")
-        field :hidden_value?, _("Hidden Value?")
-        field :omit, _("Omit"), Fields::Boolean
-        field :required, _("Required")
-
-        label _("Validator") do
-          field :validator_type, _("Type")
-          field :validator_rule, _("Rule")
-        end
-        label _("Override values") do
-          field :merge_overrides, _("Merge overrides"), Fields::Boolean
-          field :merge_default, _("Merge default value"), Fields::Boolean
-          field :avoid_duplicates, _("Avoid duplicates"), Fields::Boolean
-          field :override_value_order, _("Order"), Fields::LongText
-
-          collection :override_values, _("Values") do
-              field :id, _('Id')
-              field :match, _('Match')
-              field :value, _('Value')
-              field :omit, _('Omit'), Fields::Boolean
-          end
-        end
-        HammerCLIForeman::References.environments(self)
-        HammerCLIForeman::References.timestamps(self)
-      end
-
-      def extend_data(res)
-        res['parameter_type'] ||= 'string'
-        res['omit'] ||= false
-        res
-      end
-
-      build_options do |options|
-        options.expand.including(:puppetclasses)
-      end
-
-      validate_options do
-        if option(:option_name).exist?
-          any(:option_puppetclass_name, :option_puppetclass_id).required
-        end
-      end
-    end
-
-    class UpdateCommand < HammerCLIForeman::UpdateCommand
-
-      success_message _("Parameter updated.")
-      failure_message _("Could not update the parameter")
-
-      option '--default-value', 'VALUE', _('Value to use when there is no match')
-
-      build_options do |options|
-        options.expand.including(:puppetclasses)
-        options.without(:parameter_type, :validator_type, :override, :required, :override_value_order)
-      end
-
-      option "--override", "OVERRIDE", _("Override this parameter"),
-        :format => HammerCLI::Options::Normalizers::Bool.new
-      option "--required", "REQUIRED", _("This parameter is required"),
-        :format => HammerCLI::Options::Normalizers::Bool.new
-      option "--parameter-type", "PARAMETER_TYPE", _("Type of the parameter"),
-        :format => HammerCLI::Options::Normalizers::Enum.new(
-            ['string', 'boolean', 'integer', 'real', 'array', 'hash', 'yaml', 'json'])
-      option "--validator-type", "VALIDATOR_TYPE", _("Type of the validator"),
-        :format => HammerCLI::Options::Normalizers::Enum.new(['regexp', 'list', ''])
-      option "--override-value-order", "OVERRIDE_VALUE_ORDER", _("The order in which values are resolved"),
-             :format => HammerCLI::Options::Normalizers::List.new
-
-      validate_options do
-        if option(:option_name).exist?
-          any(:option_puppetclass_name, :option_puppetclass_id).required
-        end
-      end
-
-      def request_params
-        params = super
-        override_order = params['smart_class_parameter']['override_value_order']
-        params['smart_class_parameter']['override_value_order'] = override_order.join("\n") if override_order.is_a?(Array)
-        params
-      end
-    end
-
-    class AddMatcherCommand < HammerCLIForeman::CreateCommand
-      resource :override_values
-      command_name 'add-matcher'
-
-      option '--value', 'VALUE', _('Override value, required if omit is false')
-
-      success_message _("Override value created.")
-      failure_message _("Could not create the override value")
-
-      build_options do |options|
-        options.expand.including(:puppetclasses)
-      end
-
-      validate_options do
-        if option(:option_omit).value
-          option(:option_value).rejected(:msg => _('Cannot use --value when --omit is true.'))
-        end
-
-        if option(:option_smart_class_parameter_name).exist?
-          any(:option_puppetclass_name, :option_puppetclass_id).required
-        end
-      end
-    end
-
-    class RemoveMatcherCommand < HammerCLIForeman::DeleteCommand
-      resource :override_values
-      command_name 'remove-matcher'
-
-      success_message _("Override value deleted.")
-      failure_message _("Could not delete the override value")
-
-      build_options do |options|
-        options.expand.including(:puppetclasses)
-      end
-
-      validate_options do
-        if option(:option_smart_class_parameter_name).exist?
-          any(:option_puppetclass_name, :option_puppetclass_id).required
-        end
-      end
-    end
-
-    autoload_subcommands
-
-  end
-
-end

data/test/functional/config_group_test.rb

@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-require File.join(File.dirname(__FILE__), 'test_helper')
-
-describe 'config_group' do
-  describe 'list' do
-    before do
-      @cmd = %w[config-group list]
-      @config_groups = [{
-                          id: 1,
-                          name: 'config-group-test',
-                        }]
-    end
-
-    it 'should return a list of config groups' do
-      api_expects(:config_groups, :index, 'List config groups').returns(@config_groups)
-
-      output = IndexMatcher.new([
-                                  %w[ID NAME],
-                                  %w[1 config-group-test]
-                                ])
-      expected_result = success_result(output)
-
-      result = run_cmd(@cmd)
-      assert_cmd(expected_result, result)
-    end
-
-    it 'should run list command with defaults' do
-      providers = { 'foreman' => HammerCLIForeman::Defaults.new(api_connection({}, '2.1')) }
-      defaults = HammerCLI::Defaults.new(
-        {
-          organization_id: {
-            provider: 'foreman'
-          },
-          location_id: {
-            provider: 'foreman'
-          }
-        }
-      )
-      defaults.stubs(:write_to_file).returns(true)
-      defaults.stubs(:providers).returns(providers)
-      api_expects(:config_groups, :index, 'List config groups').returns(@config_groups)
-
-      result = run_cmd(@cmd, { use_defaults: true, defaults: defaults })
-      _(result.exit_code).must_equal HammerCLI::EX_OK
-    end
-  end
-end
-
-

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