pdk 2.7.1 → 3.0.0

data/lib/pdk/cli/set/config.rb

@@ -1,119 +1,121 @@
-module PDK::CLI
-  module Set
-    module Config
-      ALLOWED_TYPE_NAMES = %w[array boolean number string].freeze
-
-      # :nocov:
-      def self.pretty_allowed_names
-        ALLOWED_TYPE_NAMES.map { |name| "'#{name}'" }.join(', ')
-      end
-      # :nocov:
-
-      def self.transform_value(type_name, value)
-        normalized_name = type_name.downcase.strip
-        unless ALLOWED_TYPE_NAMES.include?(normalized_name)
-          raise PDK::CLI::ExitWithError, 'Unknown type %{type_name}. Expected one of %{allowed}' % { type_name: type_name, allowed: pretty_allowed_names }
+module PDK
+  module CLI
+    module Set
+      module Config
+        ALLOWED_TYPE_NAMES = ['array', 'boolean', 'number', 'string'].freeze
+
+        # :nocov:
+        def self.pretty_allowed_names
+          ALLOWED_TYPE_NAMES.map { |name| "'#{name}'" }.join(', ')
         end
+        # :nocov:
 
-        # Short circuit string conversions as it's trivial
-        if normalized_name == 'string'
-          raise PDK::CLI::ExitWithError, 'An error occured converting \'%{value}\' into a %{type_name}' % { value: value.nil? ? 'nil' : value, type_name: type_name } unless value.is_a?(String)
-          return value
-        end
+        def self.transform_value(type_name, value)
+          normalized_name = type_name.downcase.strip
+          unless ALLOWED_TYPE_NAMES.include?(normalized_name)
+            raise PDK::CLI::ExitWithError, format('Unknown type %{type_name}. Expected one of %{allowed}', type_name: type_name, allowed: pretty_allowed_names)
+          end
 
-        begin
-          case normalized_name
-          when 'array'
-            convert_to_array(value)
-          when 'boolean'
-            convert_to_boolean(value)
-          when 'number'
-            convert_to_number(value)
-          else
-            value
+          # Short circuit string conversions as it's trivial
+          if normalized_name == 'string'
+            raise PDK::CLI::ExitWithError, format('An error occured converting \'%{value}\' into a %{type_name}', value: value.nil? ? 'nil' : value, type_name: type_name) unless value.is_a?(String)
+
+            return value
+          end
+
+          begin
+            case normalized_name
+            when 'array'
+              convert_to_array(value)
+            when 'boolean'
+              convert_to_boolean(value)
+            when 'number'
+              convert_to_number(value)
+            else
+              value
+            end
+          rescue ArgumentError, TypeError
+            raise PDK::CLI::ExitWithError, format('An error occured converting \'%{value}\' into a %{type_name}', value: value.nil? ? 'nil' : value, type_name: type_name)
           end
-        rescue ArgumentError, TypeError
-          raise PDK::CLI::ExitWithError, 'An error occured converting \'%{value}\' into a %{type_name}' % { value: value.nil? ? 'nil' : value, type_name: type_name }
         end
-      end
 
-      def self.convert_to_array(value)
-        return [] if value.nil?
-        value.is_a?(Array) ? value : [value]
-      end
-      private_class_method :convert_to_array
+        def self.convert_to_array(value)
+          return [] if value.nil?
 
-      def self.convert_to_boolean(value)
-        string_val = value.to_s.strip.downcase
+          value.is_a?(Array) ? value : [value]
+        end
+        private_class_method :convert_to_array
 
-        return true  if %w[yes true -1 1].include?(string_val)
-        return false if %w[no false 0].include?(string_val)
+        def self.convert_to_boolean(value)
+          string_val = value.to_s.strip.downcase
 
-        raise ArgumentError
-      end
-      private_class_method :convert_to_boolean
+          return true  if ['yes', 'true', '-1', '1'].include?(string_val)
+          return false if ['no', 'false', '0'].include?(string_val)
 
-      def self.convert_to_number(value)
-        float_val = Float(value)
-        # Return an Integer if this is actually and Integer, otherwise return the float
-        (float_val.truncate == float_val) ? float_val.truncate : float_val
-      end
-      private_class_method :convert_to_number
+          raise ArgumentError
+        end
+        private_class_method :convert_to_boolean
+
+        def self.convert_to_number(value)
+          float_val = Float(value)
+          # Return an Integer if this is actually and Integer, otherwise return the float
+          float_val.truncate == float_val ? float_val.truncate : float_val
+        end
+        private_class_method :convert_to_number
 
-      def self.run(opts, args)
-        item_name = (args.count > 0) ? args[0] : nil
-        item_value = (args.count > 1) ? args[1] : nil
+        def self.run(opts, args)
+          item_name = args.count.positive? ? args[0] : nil
+          item_value = args.count > 1 ? args[1] : nil
 
-        opts[:type] = opts[:as] if opts[:type].nil? && !opts[:as].nil?
-        force = opts[:force] || false
+          opts[:type] = opts[:as] if opts[:type].nil? && !opts[:as].nil?
+          force = opts[:force] || false
 
-        # Transform the value if we need to
-        item_value = PDK::CLI::Set::Config.transform_value(opts[:type], item_value) unless opts[:type].nil?
+          # Transform the value if we need to
+          item_value = PDK::CLI::Set::Config.transform_value(opts[:type], item_value) unless opts[:type].nil?
 
-        raise PDK::CLI::ExitWithError, 'Configuration name is required' if item_name.nil?
-        raise PDK::CLI::ExitWithError, 'Configuration value is required. If you wish to remove a value use \'pdk remove config\'' if item_value.nil?
+          raise PDK::CLI::ExitWithError, 'Configuration name is required' if item_name.nil?
+          raise PDK::CLI::ExitWithError, 'Configuration value is required. If you wish to remove a value use \'pdk remove config\'' if item_value.nil?
 
-        current_value = PDK.config.get(item_name)
-        raise PDK::CLI::ExitWithError, "The configuration item '%{name}' can not have a value set." % { name: item_name } if current_value.is_a?(PDK::Config::Namespace)
+          current_value = PDK.config.get(item_name)
+          raise PDK::CLI::ExitWithError, format("The configuration item '%{name}' can not have a value set.", name: item_name) if current_value.is_a?(PDK::Config::Namespace)
 
-        # If we're forcing the value, don't do any munging
-        unless force
+          # If we're forcing the value, don't do any munging
           # Check if the setting already exists
-          if current_value.is_a?(Array) && current_value.include?(item_value)
-            PDK.logger.info("No changes made to '%{name}' as it already contains value '%{to}'" % { name: item_name, to: item_value })
+          if !force && (current_value.is_a?(Array) && current_value.include?(item_value))
+            PDK.logger.info(format("No changes made to '%{name}' as it already contains value '%{to}'", name: item_name, to: item_value))
             return 0
           end
-        end
 
-        new_value = PDK.config.set(item_name, item_value, force: opts[:force])
-        if current_value.nil? || force
-          PDK.logger.info("Set initial value of '%{name}' to '%{to}'" % { name: item_name, to: new_value })
-        elsif current_value.is_a?(Array)
-          # Arrays have a special output format
-          PDK.logger.info("Added new value '%{to}' to '%{name}'" % { name: item_name, to: item_value })
-        else
-          PDK.logger.info("Changed existing value of '%{name}' from '%{from}' to '%{to}'" % { name: item_name, from: current_value, to: new_value })
-        end
+          new_value = PDK.config.set(item_name, item_value, force: opts[:force])
+          if current_value.nil? || force
+            PDK.logger.info(format("Set initial value of '%{name}' to '%{to}'", name: item_name, to: new_value))
+          elsif current_value.is_a?(Array)
+            # Arrays have a special output format
+            PDK.logger.info(format("Added new value '%{to}' to '%{name}'", name: item_name, to: item_value))
+          else
+            PDK.logger.info(format("Changed existing value of '%{name}' from '%{from}' to '%{to}'", name: item_name, from: current_value, to: new_value))
+          end
 
-        # Same output as `get config`
-        $stdout.puts '%{name}=%{value}' % { name: item_name, value: PDK.config.get(item_name) }
-        0
+          # Same output as `get config`
+          $stdout.puts format('%{name}=%{value}', name: item_name, value: PDK.config.get(item_name))
+          0
+        end
       end
     end
-  end
 
-  @set_config_cmd = @set_cmd.define_command do
-    name 'config'
-    usage 'config [name] [value]'
-    summary 'Set or update the configuration for <name>'
+    @set_config_cmd = @set_cmd.define_command do
+      name 'config'
+      usage 'config [name] [value]'
+      summary 'Set or update the configuration for <name>'
 
-    option :f, :force, 'Force the configuration setting to be overwitten.', argument: :forbidden
+      option :f, :force, 'Force the configuration setting to be overwitten.', argument: :forbidden
 
-    option :t, :type, 'The type of value to set. Acceptable values: %{values}' % { values: PDK::CLI::Set::Config.pretty_allowed_names }, argument: :required
-    option nil, :as, 'Alias of --type', argument: :required
+      option :t, :type, format('The type of value to set. Acceptable values: %{values}', values: PDK::CLI::Set::Config.pretty_allowed_names), argument: :required
+      option nil, :as, 'Alias of --type', argument: :required
 
-    run do |opts, args, _cmd|
-      exit PDK::CLI::Set::Config.run(opts, args)
+      run do |opts, args, _cmd|
+        exit PDK::CLI::Set::Config.run(opts, args)
+      end
     end
   end
 end

data/lib/pdk/cli/set.rb

@@ -1,20 +1,22 @@
-module PDK::CLI
-  @set_cmd = @base_cmd.define_command do
-    name 'set'
-    usage 'set [subcommand] [options]'
-    summary 'Set or update information about the PDK or current project.'
-    default_subcommand 'help'
+module PDK
+  module CLI
+    @set_cmd = @base_cmd.define_command do
+      name 'set'
+      usage 'set [subcommand] [options]'
+      summary 'Set or update information about the PDK or current project.'
+      default_subcommand 'help'
 
-    run do |_opts, args, _cmd|
-      if args == ['help']
-        PDK::CLI.run(%w[set --help])
-        exit 0
-      end
+      run do |_opts, args, _cmd|
+        if args == ['help']
+          PDK::CLI.run(['set', '--help'])
+          exit 0
+        end
 
-      PDK::CLI.run(%w[set help]) if args.empty?
+        PDK::CLI.run(['set', 'help']) if args.empty?
+      end
     end
+    @set_cmd.add_command Cri::Command.new_basic_help
   end
-  @set_cmd.add_command Cri::Command.new_basic_help
 end
 
 require 'pdk/cli/set/config'

data/lib/pdk/cli/test/unit.rb

@@ -1,89 +1,91 @@
-module PDK::CLI
-  @test_unit_cmd = @test_cmd.define_command do
-    name 'unit'
-    usage 'unit [options]'
-    summary 'Run unit tests.'
-
-    PDK::CLI.puppet_version_options(self)
-    PDK::CLI.puppet_dev_option(self)
-    flag nil, :list, 'List all available unit test files.'
-    flag nil, :parallel, 'Run unit tests in parallel.'
-    flag :v, :verbose, 'More verbose --list output. Displays a list of examples in each unit test file.'
-    flag :c, 'clean-fixtures', 'Clean up downloaded fixtures after the test run.'
-
-    option nil, :tests, 'Specify a comma-separated list of unit test files to run.', argument: :required, default: '' do |values|
-      require 'pdk/cli/util/option_validator'
-      PDK::CLI::Util::OptionValidator.comma_separated_list?(values)
-    end
+module PDK
+  module CLI
+    @test_unit_cmd = @test_cmd.define_command do
+      name 'unit'
+      usage 'unit [options]'
+      summary 'Run unit tests.'
+
+      PDK::CLI.puppet_version_options(self)
+      PDK::CLI.puppet_dev_option(self)
+      flag nil, :list, 'List all available unit test files.'
+      flag nil, :parallel, 'Run unit tests in parallel.'
+      flag :v, :verbose, 'More verbose --list output. Displays a list of examples in each unit test file.'
+      flag :c, 'clean-fixtures', 'Clean up downloaded fixtures after the test run.'
+
+      option nil, :tests, 'Specify a comma-separated list of unit test files to run.', argument: :required, default: '' do |values|
+        require 'pdk/cli/util/option_validator'
+        PDK::CLI::Util::OptionValidator.comma_separated_list?(values)
+      end
 
-    # TODO
-    # option nil, :runner_options, "options to pass through to the actual test-runner", argument: :required
+      # TODO
+      # option nil, :runner_options, "options to pass through to the actual test-runner", argument: :required
 
-    run do |opts, _args, _cmd|
-      require 'pdk/tests/unit'
-      require 'pdk/report'
-      require 'pdk/util/bundler'
+      run do |opts, _args, _cmd|
+        require 'pdk/tests/unit'
+        require 'pdk/report'
+        require 'pdk/util/bundler'
 
-      PDK::CLI::Util.validate_puppet_version_opts(opts)
+        PDK::CLI::Util.validate_puppet_version_opts(opts)
 
-      PDK::CLI::Util.ensure_in_module!(
-        message:   'Unit tests can only be run from inside a valid module directory.',
-        log_level: :info,
-      )
+        PDK::CLI::Util.ensure_in_module!(
+          message: 'Unit tests can only be run from inside a valid module directory.',
+          log_level: :info
+        )
 
-      PDK::CLI::Util.module_version_check
+        PDK::CLI::Util.module_version_check
 
-      PDK::CLI::Util.analytics_screen_view('test_unit', opts)
+        PDK::CLI::Util.analytics_screen_view('test_unit', opts)
 
-      # Ensure that the bundled gems are up to date and correct Ruby is activated before running or listing tests.
-      puppet_env = PDK::CLI::Util.puppet_from_opts_or_env(opts)
-      PDK::Util::RubyVersion.use(puppet_env[:ruby_version])
+        # Ensure that the bundled gems are up to date and correct Ruby is activated before running or listing tests.
+        puppet_env = PDK::CLI::Util.puppet_from_opts_or_env(opts)
+        PDK::Util::RubyVersion.use(puppet_env[:ruby_version])
 
-      opts.merge!(puppet_env[:gemset])
+        opts.merge!(puppet_env[:gemset])
 
-      PDK::Util::Bundler.ensure_bundle!(puppet_env[:gemset])
+        PDK::Util::Bundler.ensure_bundle!(puppet_env[:gemset])
 
-      report = nil
+        report = nil
 
-      if opts[:list]
-        examples = PDK::Test::Unit.list(opts)
+        if opts[:list]
+          examples = PDK::Test::Unit.list(opts)
 
-        if examples.empty?
-          puts 'No unit test files with examples were found.'
-        else
-          puts 'Unit Test Files:'
-          files = examples.map { |example| example[:file_path] }
-          files.uniq.each do |file|
-            puts file
+          if examples.empty?
+            puts 'No unit test files with examples were found.'
+          else
+            puts 'Unit Test Files:'
+            files = examples.map { |example| example[:file_path] }
+            files.uniq.each do |file|
+              puts file
 
-            next unless opts[:verbose]
+              next unless opts[:verbose]
 
-            file_examples = examples.select { |example| example[:file_path] == file }
-            file_examples.each do |file_example|
-              puts "\t%{id}\t%{description}" % { id: file_example[:id], description: file_example[:full_description] }
+              file_examples = examples.select { |example| example[:file_path] == file }
+              file_examples.each do |file_example|
+                puts format("\t%{id}\t%{description}", id: file_example[:id], description: file_example[:full_description])
+              end
             end
           end
-        end
-      else
-        report = PDK::Report.new
-        report_formats = if opts[:format]
-                           opts[:interactive] = false
-                           PDK::CLI::Util::OptionNormalizer.report_formats(opts[:format])
-                         else
-                           opts[:interactive] = true
-                           [{
-                             method: PDK::Report.default_format,
-                             target: PDK::Report.default_target,
-                           }]
-                         end
-
-        exit_code = PDK::Test::Unit.invoke(report, opts)
-
-        report_formats.each do |format|
-          report.send(format[:method], format[:target])
-        end
+        else
+          report = PDK::Report.new
+          report_formats = if opts[:format]
+                             opts[:interactive] = false
+                             PDK::CLI::Util::OptionNormalizer.report_formats(opts[:format])
+                           else
+                             opts[:interactive] = true
+                             [{
+                               method: PDK::Report.default_format,
+                               target: PDK::Report.default_target
+                             }]
+                           end
+
+          exit_code = PDK::Test::Unit.invoke(report, opts)
+
+          report_formats.each do |format|
+            report.send(format[:method], format[:target])
+          end
 
-        exit exit_code
+          exit exit_code
+        end
       end
     end
   end

data/lib/pdk/cli/test.rb

@@ -1,11 +1,13 @@
-module PDK::CLI
-  @test_cmd = @base_cmd.define_command do
-    name 'test'
-    usage 'test [subcommand] [options]'
-    summary 'Run tests.'
-    default_subcommand 'help'
+module PDK
+  module CLI
+    @test_cmd = @base_cmd.define_command do
+      name 'test'
+      usage 'test [subcommand] [options]'
+      summary 'Run tests.'
+      default_subcommand 'help'
+    end
+    @test_cmd.add_command Cri::Command.new_basic_help
   end
-  @test_cmd.add_command Cri::Command.new_basic_help
 end
 
 require 'pdk/cli/test/unit'

data/lib/pdk/cli/update.rb

@@ -1,54 +1,49 @@
-module PDK::CLI
-  @update_cmd = @base_cmd.define_command do
-    name 'update'
-    usage 'update [options]'
-    summary 'Update a module that has been created by or converted for use by PDK.'
+module PDK
+  module CLI
+    @update_cmd = @base_cmd.define_command do
+      name 'update'
+      usage 'update [options]'
+      summary 'Update a module that has been created by or converted for use by PDK.'
 
-    flag nil, :noop, 'Do not update the module, just output what would be done.'
-    flag nil, :force, 'Update the module automatically, with no prompts.'
+      flag nil, :noop, 'Do not update the module, just output what would be done.'
+      flag nil, :force, 'Update the module automatically, with no prompts.'
 
-    PDK::CLI.template_ref_option(self)
+      PDK::CLI.template_ref_option(self)
 
-    run do |opts, _args, _cmd|
-      # Write the context information to the debug log
-      PDK.context.to_debug_log
+      run do |opts, _args, _cmd|
+        # Write the context information to the debug log
+        PDK.context.to_debug_log
 
-      unless PDK.context.is_a?(PDK::Context::Module)
-        raise PDK::CLI::ExitWithError, '`pdk update` can only be run from inside a valid module directory.'
-      end
+        raise PDK::CLI::ExitWithError, '`pdk update` can only be run from inside a valid module directory.' unless PDK.context.is_a?(PDK::Context::Module)
 
-      raise PDK::CLI::ExitWithError, 'This module does not appear to be PDK compatible. To make the module compatible with PDK, run `pdk convert`.' unless PDK::Util.module_pdk_compatible?
+        raise PDK::CLI::ExitWithError, 'This module does not appear to be PDK compatible. To make the module compatible with PDK, run `pdk convert`.' unless PDK::Util.module_pdk_compatible?
 
-      if opts[:noop] && opts[:force]
-        raise PDK::CLI::ExitWithError, 'You can not specify --noop and --force when updating a module'
-      end
+        raise PDK::CLI::ExitWithError, 'You can not specify --noop and --force when updating a module' if opts[:noop] && opts[:force]
 
-      if Gem::Version.new(PDK::VERSION) < Gem::Version.new(PDK::Util.module_pdk_version)
-        PDK.logger.warn "This module has been updated to PDK #{PDK::Util.module_pdk_version} which is newer than your PDK version (#{PDK::VERSION}), proceed with caution!"
+        if Gem::Version.new(PDK::VERSION) < Gem::Version.new(PDK::Util.module_pdk_version)
+          PDK.logger.warn "This module has been updated to PDK #{PDK::Util.module_pdk_version} which is newer than your PDK version (#{PDK::VERSION}), proceed with caution!"
 
-        unless opts[:force]
-          raise PDK::CLI::ExitWithError,
-                'Please update your PDK installation and try again. ' \
-                'You may also use the --force flag to override this and ' \
-                'continue at your own risk.'
+          unless opts[:force]
+            raise PDK::CLI::ExitWithError,
+                  'Please update your PDK installation and try again. ' \
+                  'You may also use the --force flag to override this and ' \
+                  'continue at your own risk.'
+          end
         end
-      end
 
-      PDK::CLI::Util.analytics_screen_view('update', opts)
+        PDK::CLI::Util.analytics_screen_view('update', opts)
 
-      updater = PDK::Module::Update.new(PDK.context.root_path, opts)
+        updater = PDK::Module::Update.new(PDK.context.root_path, opts)
 
-      if updater.pinned_to_puppetlabs_template_tag?
-        PDK.logger.info 'This module is currently pinned to version %{current_version} ' \
-                        'of the default template. If you would like to update your ' \
-                        'module to the latest version of the template, please run `pdk ' \
-                        'update --template-ref %{new_version}`.' % {
-                          current_version: updater.template_uri.uri_fragment,
-                          new_version: PDK::TEMPLATE_REF,
-                        }
-      end
+        if updater.pinned_to_puppetlabs_template_tag?
+          PDK.logger.info format('This module is currently pinned to version %{current_version} ' \
+                                 'of the default template. If you would like to update your ' \
+                                 'module to the latest version of the template, please run `pdk ' \
+                                 'update --template-ref %{new_version}`.', current_version: updater.template_uri.uri_fragment, new_version: PDK::TEMPLATE_REF)
+        end
 
-      updater.run
+        updater.run
+      end
     end
   end
 end

data/lib/pdk/cli/util/command_redirector.rb

@@ -7,6 +7,15 @@ module PDK
       class CommandRedirector < TTY::Prompt::AnswersCollector
         attr_accessor :command
 
+        # Override the initialize method because the original one
+        # doesn't work with Ruby 3.
+        # rubocop:disable Lint/MissingSuper
+        def initialize(prompt, options = {})
+          @prompt  = prompt
+          @answers = options.fetch(:answers) { {} }
+        end
+        # rubocop:enable Lint/MissingSuper
+
         def pastel
           @pastel ||= Pastel.new
         end
@@ -16,7 +25,7 @@ module PDK
         end
 
         def run
-          @prompt.puts 'Did you mean \'%{command}\'?' % { command: pastel.bold(@command) }
+          @prompt.puts "Did you mean '#{pastel.bold(@command)}'?"
           @prompt.yes?('-->')
         rescue PDK::CLI::Util::Interview::READER::InputInterrupt
           nil

data/lib/pdk/cli/util/interview.rb

@@ -7,6 +7,15 @@ module PDK
       class Interview < TTY::Prompt::AnswersCollector
         READER = defined?(TTY::Reader) ? TTY::Reader : TTY::Prompt::Reader
 
+        # Override the initialize method because the original one
+        # doesn't work with Ruby 3.
+        # rubocop:disable Lint/MissingSuper
+        def initialize(prompt, options = {})
+          @prompt  = prompt
+          @answers = options.fetch(:answers) { {} }
+        end
+        # rubocop:enable Lint/MissingSuper
+
         def pastel
           @pastel ||= Pastel.new
         end
@@ -30,7 +39,7 @@ module PDK
           num_questions = @questions.count
           @questions.each do |question_name, question|
             @name = question_name
-            @prompt.print pastel.bold('[Q %{current_number}/%{questions_total}]' % { current_number: i, questions_total: num_questions }) + ' '
+            @prompt.print "#{pastel.bold(format('[Q %{current_number}/%{questions_total}]', current_number: i, questions_total: num_questions))} "
             @prompt.puts pastel.bold(question[:question])
             @prompt.puts question[:help] if question.key?(:help)
 
@@ -52,9 +61,7 @@ module PDK
               ask('-->') do |q|
                 q.required(question.fetch(:required, false))
 
-                if question.key?(:validate_pattern)
-                  q.validate(question[:validate_pattern], question[:validate_message])
-                end
+                q.validate(question[:validate_pattern], question[:validate_message]) if question.key?(:validate_pattern)
 
                 q.default(question[:default]) if question.key?(:default)
               end

data/lib/pdk/cli/util/option_normalizer.rb

@@ -6,6 +6,7 @@ module PDK
       class OptionNormalizer
         def self.comma_separated_list_to_array(list, _options = {})
           raise 'Error: expected comma separated list' unless OptionValidator.comma_separated_list?(list)
+
           list.split(',').compact
         end
 
@@ -31,10 +32,7 @@ module PDK
             begin
               OptionValidator.enum(format, PDK::Report.formats)
             rescue ArgumentError
-              raise PDK::CLI::ExitWithError, "'%{name}' is not a valid report format (%{valid})" % {
-                name:  format,
-                valid: PDK::Report.formats.join(', '),
-              }
+              raise PDK::CLI::ExitWithError, format("'%{name}' is not a valid report format (%{valid})", name: format, valid: PDK::Report.formats.join(', '))
             end
 
             case target