puppet-lint 4.2.0 → 4.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69069ebb59ecc423674d2918bf4493ae0da816b8d9a6f3da7db8a34cc7e657bf
-  data.tar.gz: 28cfb8db332d53efcaab17e412a1f965a382a341aa3696c2148116ce18b4570b
+  metadata.gz: e762e8a97a52868c81c71f50b4a168b2ffe6057019d7bae74de03c7c5b2e39ff
+  data.tar.gz: 29f70f084a46c9e218b33447cc0b5573b0c1a7a6d451e7a051977906a83de0cc
 SHA512:
-  metadata.gz: 3f951b0291881ea53286adf57085a8cc0ef5164028ca2c815b05bd196f4f35f038d10910fb417d03465590ba0af20a835fceaa1037808f5a697d8c1ce0ce3b76
-  data.tar.gz: 4c6347b7c83ab801cc03994c4422239ebec41ce851179f4362c43b0aa012f61ea362aebc27b0a8da5abd3c42fea1274d10c97092782f24635d95eb9b96612b03
+  metadata.gz: 298c4387c06213f7ecaf8a84b4367762d1efbbf7752b33f09b4b6064d62a1e567b467bbdf91e6c9009c1052f78c6b2bf7570fc2c7efba454a961ffc386c3ffda
+  data.tar.gz: e8425d0f5b5f1c0ab14d7b9dc8de13bfefd4e57b5209839f2523a1b3701e807182f5c41c2d99a04bc18e74bbf0507eda8eb495ee550521fa3cfe9f5d89a406fe

data/README.md

@@ -1,13 +1,15 @@
 # Puppet Lint
 
+[![Code Owners](https://img.shields.io/badge/owners-DevX--team-blue)](https://github.com/puppetlabs/puppet-lint/blob/main/CODEOWNERS)
+![ci](https://github.com/puppetlabs/puppet-lint/actions/workflows/ci.yml/badge.svg)
+![GitHub release (latest by date)](https://img.shields.io/github/v/release/puppetlabs/puppet-lint)
+
 Puppet Lint tests Puppet code against the recommended [Puppet language style
 guide](http://puppet.com/docs/puppet/latest/style_guide.html). Puppet Lint validates only code style; it does not validate syntax. To test syntax, use Puppet's `puppet parser validate` command.
 
 ## Compatibility warning
 
-Version 3.0.0 and above will no longer support Puppet 6 environments.
-
-In cases where Puppet Lint is required in an environment with Puppet 6, we recommend pinning to version 2.5.2.
+This tool is only supported on Puppet 7 & 8 environments. In cases where Puppet Lint is required in an environment with Puppet 6, we recommend pinning to version 2.5.2.
 
 ## Installation
 
@@ -112,7 +114,7 @@ Any flag that can be specified on the command line can also be specified in the
 --no-hard_tabs-check
 ```
 
-Or to specify a whitelist of allowed checks, include a line like:
+Or to specify an allowlist of allowed checks, include a line like:
 
 ```
 --only-checks=trailing_whitespace,hard_tabs,duplicate_params,double_quoted_strings,unquoted_file_mode,only_variable_string,variables_not_enclosed,single_quote_string_with_variables,variable_contains_dash,ensure_not_symlink_target,unquoted_resource_title,relative_classname_inclusion,file_mode,resource_reference_without_title_capital,leading_zero,arrow_alignment,variable_is_lowercase,ensure_first_param,resource_reference_without_whitespace,file_ensure,trailing_comma,leading_zero
@@ -293,6 +295,18 @@ If you find a bug in Puppet Lint or its results, please create an issue in the
 [repo issues tracker](https://github.com/puppetlabs/puppet-lint/issues/). Bonus
 points will be awarded if you also include a patch that fixes the issue.
 
+## Development
+
+Acceptance tests for this tool leverage [puppet_litmus](https://github.com/puppetlabs/puppet_litmus).
+To run the acceptance tests follow the instructions [here](https://github.com/puppetlabs/puppet_litmus/wiki/Tutorial:-use-Litmus-to-execute-acceptance-tests-with-a-sample-module-(MoTD)#install-the-necessary-gems-for-the-module).
+You can also find a tutorial and walkthrough of using Litmus and the PDK on [YouTube](https://www.youtube.com/watch?v=FYfR7ZEGHoE).
+
+If you run into an issue with this tool or would like to request a feature you can [raise a PR](https://github.com/puppetlabs/puppet-lint/pulls) with your suggested changes. Alternatively, you can [raise a Github issue](https://github.com/puppetlabs/puppet-lint/issues) with a feature request or to report any bugs.
+Every other Tuesday the DevX team holds [office hours](https://puppet.com/community/office-hours) in the [Puppet Community Slack](http://slack.puppet.com/), where you can ask questions about this and any other supported tools.
+This session runs at 15:00 (GMT) for about an hour.
+
+If you have problems getting this tool up and running, please [contact Support](http://puppetlabs.com/services/customer-support).
+
 ## Thank you
 
 Many thanks to the following people for contributing to puppet-lint

data/lib/puppet-lint/data.rb

@@ -1,24 +1,30 @@
 require 'singleton'
 require 'set'
 
-# Public: A singleton class storing all the information about the manifest
-# being analysed.
+# A singleton class storing all the information about the manifest being
+# analysed.
+#
+# @api public
 class PuppetLint::Data
   include Singleton
 
   class << self
-    # Internal: Get/Set the full expanded path to the manifest file being
-    # checked.
+    # Get/Set the full expanded path to the manifest file being checked.
+    #
+    # @api private
     attr_reader :path, :fullpath, :filename
 
-    # Internal: Get/Set the raw manifest data, split by \n.
+    # Get/Set the raw manifest data, split by \n.
+    #
+    # @api private
     attr_accessor :manifest_lines
 
-    # Internal: Store the tokenised manifest.
+    # Store the tokenised manifest.
     #
-    # tokens - The Array of PuppetLint::Lexer::Token objects to store.
+    # @param [Array[PuppetLint::Lexer::Token]] tokens The Array of PuppetLint::Lexer::Token objects to store.
+    # @return [nil]
     #
-    # Returns nothing.
+    # @api private
     def tokens=(tokens)
       @tokens = tokens
       @title_tokens = nil
@@ -32,14 +38,17 @@ class PuppetLint::Data
       @defaults_indexes = nil
     end
 
+    # @api private
     def ruby1?
       @ruby1 = RbConfig::CONFIG['MAJOR'] == '1' if @ruby1.nil?
       @ruby1
     end
 
-    # Public: Get the tokenised manifest.
+    # Get the tokenised manifest.
+    #
+    # @return [Array[PuppetLint::Lexer::Token]]
     #
-    # Returns an Array of PuppetLint::Lexer::Token objects.
+    # @api public
     def tokens
       calling_method = if ruby1?
                          begin
@@ -62,7 +71,12 @@ class PuppetLint::Data
       end
     end
 
-    # Public: Add new token.
+    # Add new token.
+    #
+    # @param [Integer] index
+    # @param [PuppetLint::Lexer::Token] token
+    #
+    # @api public
     def insert(index, token)
       current_token = tokens[index - 1]
       token.next_token = current_token.next_token
@@ -97,7 +111,11 @@ class PuppetLint::Data
       tokens.insert(index, token)
     end
 
-    # Public: Removes a token
+    # Remove a token
+    #
+    # @param [PuppetLint::Lexer::Token] token
+    #
+    # @api public
     def delete(token)
       token.next_token.prev_token = token.prev_token unless token.next_token.nil?
       token.prev_token.next_token = token.next_token unless token.prev_token.nil?
@@ -109,12 +127,13 @@ class PuppetLint::Data
       tokens.delete(token)
     end
 
-    # Internal: Store the path to the manifest file and populate fullpath and
-    # filename.
+    # Store the path to the manifest file and populate fullpath and filename.
     #
-    # val - The path to the file as a String.
+    # @param [String] val The path to the file
     #
-    # Returns nothing.
+    # @return [nil]
+    #
+    # @api private
     def path=(val)
       @path = val
       if val.nil?
@@ -126,9 +145,11 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Retrieve a list of tokens that represent resource titles
+    # Retrieve a list of tokens that represent resource titles
+    #
+    # @return [Array[PuppetLint::Lexer::Token]]
     #
-    # Returns an Array of PuppetLint::Lexer::Token objects.
+    # @api private
     def title_tokens
       @title_tokens ||= begin
         result = []
@@ -153,10 +174,11 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Determine if the given token contains a CLASSREF in
-    # the token chain..
+    # Determine if the given token contains a CLASSREF in the token chain..
     #
-    # Returns a Boolean.
+    # @param [PuppetLint::Lexer::Token] token
+    #
+    # @api private
     def classref?(token)
       current_token = token
       while (current_token = current_token.prev_code_token)
@@ -165,15 +187,15 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Calculate the positions of all resource declarations within the
+    # Calculate the positions of all resource declarations within the
     # tokenised manifest. These positions only point to the content of the
     # resource declarations, they do not include resource types or titles.
     #
-    # Returns an Array of Hashes, each containing:
-    #   :start - An Integer position in the `tokens` Array pointing to the
-    #            first Token of a resource declaration.
-    #   :end   - An Integer position in the `tokens` Array pointing to the last
-    #            Token of a resource declaration.
+    # @return [Array[Hash[Symbol, Integer]]] each hash contains :start and
+    #   :end pointing to the first and last {PuppetLint::Lexer::Token} of a
+    #   resource declaration
+    #
+    # @api private
     def resource_indexes
       @resource_indexes ||= begin
         marker = 0
@@ -204,12 +226,15 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Find the Token representing the type of a resource definition.
+    # Find the Token representing the type of a resource definition.
+    #
+    # @param [Integer] index
+    #   The Integer pointing to the start of the resource in the `tokens`
+    #   array.
     #
-    # index - The Integer pointing to the start of the resource in the `tokens`
-    #         array.
+    # @return [PuppetLint::Lexer::Token]
     #
-    # Returns a Token object.
+    # @api private
     def find_resource_type_token(index)
       lbrace_idx = tokens[0..index].rindex do |token|
         token.type == :LBRACE && token.prev_code_token.type != :QMARK
@@ -220,13 +245,15 @@ class PuppetLint::Data
       tokens[lbrace_idx].prev_code_token
     end
 
-    # Internal: Find all the Token objects representing the parameter names in
+    # Find all the Token objects representing the parameter names in
     # a resource definition.
     #
-    # resource_tokens - An Array of Token objects that comprise the resource
-    #                   definition.
+    # @param [Array[PuppetLint::Lexer::Token]] resource_tokens
+    #   An Array of Token objects that comprise the resource definition.
     #
-    # Returns an Array of Token objects.
+    # @return [Array[PuppetLint::Lexer::Token]]
+    #
+    # @api private
     def find_resource_param_tokens(resource_tokens)
       param_tokens = []
 
@@ -243,8 +270,8 @@ class PuppetLint::Data
       param_tokens
     end
 
-    # Internal: Calculate the positions of all class definitions within the
-    # `tokens` Array.
+    # Calculate the positions of all class definitions within the `tokens`
+    # Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -253,12 +280,14 @@ class PuppetLint::Data
     #             Token of a class definition.
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             class definition.
+    #
+    # @api private
     def class_indexes
       @class_indexes ||= definition_indexes(:CLASS)
     end
 
-    # Internal: Calculate the positions of all defined type definitions within
-    # the `tokens` Array.
+    # Calculate the positions of all defined type definitions within the
+    # `tokens` Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -267,12 +296,14 @@ class PuppetLint::Data
     #             Token of a defined type definition.
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             defined type.
+    #
+    # @api private
     def defined_type_indexes
       @defined_type_indexes ||= definition_indexes(:DEFINE)
     end
 
-    # Internal: Calculate the positions of all node definitions within the
-    # `tokens` Array.
+    # Calculate the positions of all node definitions within the `tokens`
+    # Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -281,12 +312,14 @@ class PuppetLint::Data
     #             Token of a defined type definition.
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             defined type.
+    #
+    # @api private
     def node_indexes
       @node_indexes ||= definition_indexes(:NODE)
     end
 
-    # Internal: Calculate the positions of all the specified defintion types
-    # within the `tokens` Array.
+    # Calculate the positions of all the specified defintion types within the
+    # `tokens` Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -295,6 +328,8 @@ class PuppetLint::Data
     #             Token of a definition.
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             definition.
+    #
+    # @api private
     def definition_indexes(type)
       result = []
       tokens.each_with_index do |token, i|
@@ -336,8 +371,7 @@ class PuppetLint::Data
       result
     end
 
-    # Internal: Calculate the positions of all function calls within
-    # `tokens` Array.
+    # Calculate the positions of all function calls within `tokens` Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -346,6 +380,8 @@ class PuppetLint::Data
     #             Token of a function call
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             function call.
+    #
+    # @api private
     def function_indexes
       @function_indexes ||= begin
         functions = []
@@ -385,8 +421,7 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Calculate the positions of all array values within
-    # `tokens` Array.
+    # Calculate the positions of all array values within `tokens` Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -395,6 +430,8 @@ class PuppetLint::Data
     #             Token of an array value
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             array value.
+    #
+    # @api private
     def array_indexes
       @array_indexes ||= begin
         arrays = []
@@ -421,8 +458,7 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Calculate the positions of all hash values within
-    # `tokens` Array.
+    # Calculate the positions of all hash values within `tokens` Array.
     #
     # Returns an Array of Hashes, each containing:
     #   :start  - An Integer position in the `tokens` Array pointing to the
@@ -431,6 +467,8 @@ class PuppetLint::Data
     #             Token of an hash value
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             hash value.
+    #
+    # @api private
     def hash_indexes
       @hash_indexes ||= begin
         hashes = []
@@ -459,7 +497,7 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Calculate the positions of all defaults declarations within
+    # Calculate the positions of all defaults declarations within
     # `tokens` Array.
     #
     # Returns an Array of Hashes, each containing:
@@ -469,6 +507,8 @@ class PuppetLint::Data
     #             Token of the defaults declaration
     #   :tokens - An Array consisting of all the Token objects that make up the
     #             defaults declaration.
+    #
+    # @api private
     def defaults_indexes
       @defaults_indexes ||= begin
         defaults = []
@@ -493,14 +533,17 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Finds all the tokens that make up the defined type or class
+    # Finds all the tokens that make up the defined type or class
     # definition parameters.
     #
-    # these_tokens - An Array of PuppetLint::Lexer::Token objects that make up
-    #                the defined type or class definition.
+    # @param [Array[PuppetLint::Lexer::Token]] these_tokens
+    #   An Array of PuppetLint::Lexer::Token objects that make up the defined
+    #   type or class definition.
     #
-    # Returns an Array of PuppetLint::Lexer::Token objects or nil if it takes
-    # no parameters.
+    # @return [Array[PuppetLint::Lexer::Token], nil] objects or nil if it takes
+    #   no parameters.
+    #
+    # @api private
     def param_tokens(these_tokens)
       depth = 0
       lparen_idx = nil
@@ -529,28 +572,34 @@ class PuppetLint::Data
       end
     end
 
-    # Internal: Retrieves a list of token types that are considered to be
+    # Retrieves a list of token types that are considered to be
     # formatting tokens (whitespace, newlines, etc).
     #
-    # Returns an Array of Symbols.
+    # @return [Array[Symbol]]
+    #
+    # @api private
     def formatting_tokens
       @formatting_tokens ||= PuppetLint::Lexer::FORMATTING_TOKENS
     end
 
-    # Internal: Retrieves a Hash of Sets. Each key is a check name Symbol and
+    # Retrieves a Hash of Sets. Each key is a check name Symbol and
     # the Set of Integers returned lists all the lines that the check results
     # should be ignored on.
     #
-    # Returns a Hash of Sets of Integers.
+    # @return [Hash[Symbol, Set[Integer]]]
+    #
+    # @api private
     def ignore_overrides
       @ignore_overrides ||= {}
     end
 
-    # Internal: Parses all COMMENT, MLCOMMENT and SLASH_COMMENT tokens looking
+    # Parses all COMMENT, MLCOMMENT and SLASH_COMMENT tokens looking
     # for control comments (comments that enable or disable checks). Builds the
     # contents of the `ignore_overrides` hash.
     #
-    # Returns nothing.
+    # @return [nil]
+    #
+    # @api private
     def parse_control_comments
       @ignore_overrides.each_key { |check| @ignore_overrides[check].clear }
 

data/lib/puppet-lint/lexer.rb

@@ -50,7 +50,7 @@ class PuppetLint::Lexer
 
   # Internal: A Hash whose keys are Strings representing reserved keywords in
   # the Puppet DSL.
-  # From https://github.com/puppetlabs/puppet/blob/master/lib/puppet/pops/parser/lexer2.rb#L116-L137
+  # From https://github.com/puppetlabs/puppet/blob/main/lib/puppet/pops/parser/lexer2.rb#L116-L137
   # or thereabouts
   KEYWORDS = {
     'case' => true,
@@ -78,7 +78,7 @@ class PuppetLint::Lexer
 
   # Internal: A Hash whose keys are Strings representing reserved keywords in
   # the Puppet DSL when Application Management is enabled
-  # From https://github.com/puppetlabs/puppet/blob/master/lib/puppet/pops/parser/lexer2.rb#L142-L159
+  # From https://github.com/puppetlabs/puppet/blob/main/lib/puppet/pops/parser/lexer2.rb#L142-L159
   # or therabouts
   # Currently unused
   APP_MANAGEMENT_TOKENS = {

data/lib/puppet-lint/plugins/check_whitespace/line_length.rb

@@ -9,7 +9,7 @@ class PuppetLint::LineLengthCheck
   # @param content [String] The content of the current line.
   # @param character_count [Integer] The maximum number of characters allowed
   #
-  # @return problem [Array] An array containing a description of the problem.
+  # @return [Array] An array containing a description of the problem.
   # Can be passed directly to notify..
   def self.check(line_number, content, character_count)
     return if content.include? '://'

data/lib/puppet-lint/plugins/legacy_facts/legacy_facts.rb

@@ -116,8 +116,8 @@ PuppetLint.new_check(:legacy_facts) do
       fact_name = ''
 
       # This matches legacy facts defined in the fact hash.
-      if (match = (token.value.match(%r{(::)?facts\['(.*)'\]}) || token.value.match(%r{(::)?facts\[(.*)\]})))
-        fact_name = match[2]
+      if (match = token.value.match(%r{^(?:::)?facts\[(?<quote>'?)(?<value>.+)\k<quote>\]$}))
+        fact_name = match[:value]
 
       # This matches using legacy facts in a the new structured fact. For
       # example this would match 'uuid' in $facts['uuid'] so it can be converted

data/lib/puppet-lint/plugins/top_scope_facts/top_scope_facts.rb

@@ -7,7 +7,7 @@
 # If you have top scope variables that aren't facts you should configure the
 # linter to ignore them.
 #
-# You can whitelist top scope variables to ignore via the Rake task.
+# You can allowlist top scope variables to ignore via the Rake task.
 # You should insert the following line to your Rakefile.
 # `PuppetLint.configuration.top_scope_variables = ['location', 'role']`
 #
@@ -17,7 +17,7 @@ TOP_SCOPE_FACTS_VAR_TYPES = Set[:VARIABLE, :UNENC_VARIABLE]
 
 PuppetLint.new_check(:top_scope_facts) do
   def check
-    whitelist = ['trusted', 'facts', 'architecture', 'augeasversion', 'bios_release_date', 'bios_vendor', 'bios_version',
+    allowlist = ['trusted', 'facts', 'architecture', 'augeasversion', 'bios_release_date', 'bios_vendor', 'bios_version',
                  'boardassettag', 'boardmanufacturer', 'boardproductname', 'boardserialnumber', 'chassisassettag', 'chassistype', 'domain',
                  'fqdn', 'gid', 'hardwareisa', 'hardwaremodel', 'hostname', 'id', 'ipaddress', 'ipaddress6', 'lsbdistcodename',
                  'lsbdistdescription', 'lsbdistid', 'lsbdistrelease', 'lsbmajdistrelease', 'lsbminordistrelease', 'lsbrelease',
@@ -28,10 +28,10 @@ PuppetLint.new_check(:top_scope_facts) do
                  'selinux_config_policy', 'selinux_current_mode', 'selinux_enforced', 'selinux_policyversion', 'serialnumber',
                  'swapencrypted', 'swapfree', 'swapsize', 'system32', 'uptime', 'uptime_days', 'uptime_hours', 'uptime_seconds',
                  'uuid', 'xendomains', 'zonename'] + (PuppetLint.configuration.top_scope_variables || [])
-    whitelist = whitelist.join('|')
+    allowlist = allowlist.join('|')
     tokens.select { |x| TOP_SCOPE_FACTS_VAR_TYPES.include?(x.type) }.each do |token|
       next unless %r{^::}.match?(token.value)
-      next if %r{^::(#{whitelist})\[?}.match?(token.value)
+      next if %r{^::(#{allowlist})\[?}.match?(token.value)
       next if %r{^::[a-z0-9_][a-zA-Z0-9_]+::}.match?(token.value)
 
       notify :warning, {

data/lib/puppet-lint/version.rb

@@ -1,3 +1,3 @@
 class PuppetLint
-  VERSION = '4.2.0'.freeze
+  VERSION = '4.2.2'.freeze
 end

data/rubocop_baseline.yml

@@ -81,3 +81,10 @@ Style/Documentation:
   - spec/**/*
 Style/WordArray:
   EnforcedStyle: brackets
+####################################################
+# Cops below here due for deprecation
+####################################################
+# ``Rspec/FilePath`` is going to be deprecated in the next major release of rubocop >=3.0.0: see <https://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/FilePath>
+# As the new cops are already present, e.g., Rspec/SpecFilePathPathFormat, then disabling this in preparation
+RSpec/FilePath:
+  Enabled: false

data/spec/spec_helper.rb

@@ -12,7 +12,6 @@ end
 
 require 'puppet-lint'
 require 'rspec/its'
-require 'rspec/collection_matchers'
 begin
   require 'rspec/json_expectations'
 rescue LoadError, SyntaxError

data/spec/unit/puppet-lint/checks_spec.rb

@@ -31,7 +31,7 @@ describe PuppetLint::Checks do
         end
 
         it 'creates a syntax error problem for the file' do
-          expect(instance.problems).to have(1).problem
+          expect(instance.problems.size).to eq(1)
           expect(instance.problems.first).to include(
             kind: :error,
             check: :syntax,
@@ -53,7 +53,7 @@ describe PuppetLint::Checks do
         end
 
         it 'creates a syntax error problem for the file' do
-          expect(instance.problems).to have(1).problem
+          expect(instance.problems.size).to eq(1)
           expect(instance.problems.first).to include(
             kind: :error,
             check: :syntax,
@@ -148,9 +148,14 @@ describe PuppetLint::Checks do
 
       it 'prints out information about the puppet-lint version and ruby environment' do
         expected_info = [
+          'Whoops! It looks like puppet-lint has encountered an error that it doesn\'t',
+          'know how to handle. Please open an issue at https://github.com/puppetlabs/puppet-lint',
+          'and paste the following output into the issue description.',
+          '---',
           "puppet-lint version: #{PuppetLint::VERSION}",
           "ruby version: #{RUBY_VERSION}-p#{RUBY_PATCHLEVEL}",
           "platform: #{RUBY_PLATFORM}",
+          "file path: #{fileinfo}",
         ]
         pattern = expected_info.map { |r| Regexp.escape(r) }.join('\s+')
         expect($stdout).to receive(:puts).with(a_string_matching(%r{#{pattern}}m))

data/spec/unit/puppet-lint/ignore_overrides_spec.rb

@@ -13,7 +13,7 @@ describe 'quoted_booleans', type: :lint do
     end
 
     it 'detects three problems' do
-      expect(problems).to have(3).problems
+      expect(problems.size).to eq(3)
     end
 
     it 'has two warnings' do
@@ -36,7 +36,7 @@ describe 'quoted_booleans', type: :lint do
     end
 
     it 'detects three problems' do
-      expect(problems).to have(3).problems
+      expect(problems.size).to eq(3)
     end
 
     it 'has two warnings' do
@@ -62,7 +62,7 @@ describe 'quoted_booleans', type: :lint do
     end
 
     it 'detects four problems' do
-      expect(problems).to have(4).problems
+      expect(problems.size).to eq(4)
     end
 
     it 'has two warnings' do
@@ -89,7 +89,7 @@ describe 'quoted_booleans', type: :lint do
     end
 
     it 'detects four problems' do
-      expect(problems).to have(4).problems
+      expect(problems.size).to eq(4)
     end
 
     it 'has two warnings' do
@@ -107,7 +107,7 @@ describe 'quoted_booleans', type: :lint do
     let(:code) { '"true" # lint:ignore:quoted_booleans lint:ignore:double_quoted_string a reason' }
 
     it 'detects 1 problems' do
-      expect(problems).to have(1).problems
+      expect(problems.size).to eq(1)
     end
 
     it 'has one ignored problems' do
@@ -129,7 +129,7 @@ describe 'quoted_booleans', type: :lint do
     end
 
     it 'detects 2 problems' do
-      expect(problems).to have(2).problems
+      expect(problems.size).to eq(2)
     end
 
     it 'ignores both problems' do