puppet-lint 2.3.0 → 2.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5cf7e44161d54a800d685bcfd4cf152f5d8513c9
-  data.tar.gz: dedf83a5d6c65efd93413867bce94c198b622b0c
+  metadata.gz: 3516ab4e6adac864c4fb3e43ce1f90bb5d81288c
+  data.tar.gz: a518428edb88a71dd7bb0708e060d5ee76fbd16c
 SHA512:
-  metadata.gz: 40ae10c6e1b29822cbc3146c9a1fb4dcbfcabacc3d934710d9b7ca654bfdbd3ba45a44186f03b32ff77ba1dbeb147f58131a7fc197e08cccac60f4ec9fa13153
-  data.tar.gz: fb7c37f37510e8d6f6fb100e797e2d002cc539ad08c5bbaddda33c301d077ba09761d71d967692ad95f5de6b8b8e7f9a66d03f1ac12370d9e0b89fb41932effb
+  metadata.gz: 70a5239c7b36ef55024580ebcda52e5bee45db6d12b21d2e957233930555dd3c91645a3f0076a393dfd79481511ab39a555689994aa4827c6a52b4326bc3745e
+  data.tar.gz: db50c67864f94a834f141af85eaf062a7a8d62f2c74fda37510bb959685e9594dd09406ca11f3b58338ade7154ffee51ed5e72dd53597ebb8503693237addf77

data/.rubocop.yml

@@ -0,0 +1,74 @@
+---
+inherit_from: './.rubocop_todo.yml'
+AllCops:
+    TargetRubyVersion: 1.9
+
+Style/HashSyntax:
+    EnforcedStyle: hash_rockets
+
+Layout/FirstArrayElementLineBreak:
+    Enabled: true
+Layout/FirstHashElementLineBreak:
+    Enabled: true
+Layout/FirstMethodArgumentLineBreak:
+    Enabled: true
+Layout/FirstMethodParameterLineBreak:
+    Enabled: true
+Layout/IndentArray:
+    EnforcedStyle: consistent
+Layout/MultilineArrayBraceLayout:
+    EnforcedStyle: new_line
+Layout/MultilineAssignmentLayout:
+    Enabled: true
+    EnforcedStyle: same_line
+Layout/MultilineHashBraceLayout:
+    EnforcedStyle: new_line
+Layout/MultilineMethodDefinitionBraceLayout:
+    EnforcedStyle: new_line
+
+Style/AutoResourceCleanup:
+    Enabled: true
+Style/BlockDelimiters:
+    EnforcedStyle: braces_for_chaining
+Style/BracesAroundHashParameters:
+    EnforcedStyle: context_dependent
+Style/Encoding:
+    Enabled: false
+Style/MethodCallWithArgsParentheses:
+    Enabled: true
+    IgnoreMacros: true
+    IgnoredMethods:
+        - puts
+        - require
+        - include
+        - it
+        - context
+        - describe
+        - to
+        - to_not
+        - raise
+        - desc
+        - task
+        - exit
+        - should
+        - gem
+        - group
+        - attr_reader
+        - attr_accessor
+        - attr_writer
+        - source
+Style/MethodCalledOnDoEndBlock:
+    Enabled: true
+Style/RegexpLiteral:
+    EnforcedStyle: percent_r
+Style/TrailingCommaInArguments:
+    EnforcedStyleForMultiline: comma
+Style/TrailingCommaInLiteral:
+    EnforcedStyleForMultiline: comma
+Style/FormatStringToken:
+    Enabled: false
+Style/FileName:
+    Exclude:
+        - 'lib/puppet-lint.rb'
+        - 'lib/puppet-lint/tasks/puppet-lint.rb'
+        - 'spec/puppet-lint_spec.rb'

data/.rubocop_todo.yml

@@ -0,0 +1,89 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2017-08-28 12:57:58 +1000 using RuboCop version 0.49.1.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 3
+# Configuration parameters: Include.
+# Include: **/Gemfile, **/gems.rb
+Bundler/DuplicatedGem:
+  Exclude:
+    - 'Gemfile'
+
+# Offense count: 2
+# Cop supports --auto-correct.
+# Configuration parameters: Include, TreatCommentsAsGroupSeparators.
+# Include: **/Gemfile, **/gems.rb
+Bundler/OrderedGems:
+  Exclude:
+    - 'Gemfile'
+
+# Offense count: 9
+# Configuration parameters: AllowSafeAssignment.
+Lint/AssignmentInCondition:
+  Exclude:
+    - 'lib/puppet-lint/lexer.rb'
+    - 'lib/puppet-lint/plugins/check_classes/variable_scope.rb'
+
+# Offense count: 52
+Metrics/AbcSize:
+  Max: 153
+
+# Offense count: 121
+# Configuration parameters: CountComments, ExcludedMethods.
+Metrics/BlockLength:
+  Max: 1136
+
+# Offense count: 8
+# Configuration parameters: CountBlocks.
+Metrics/BlockNesting:
+  Max: 5
+
+# Offense count: 2
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 383
+
+# Offense count: 20
+Metrics/CyclomaticComplexity:
+  Max: 26
+
+# Offense count: 238
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
+# URISchemes: http, https
+Metrics/LineLength:
+  Max: 223
+
+# Offense count: 65
+# Configuration parameters: CountComments.
+Metrics/MethodLength:
+  Max: 94
+
+# Offense count: 18
+Metrics/PerceivedComplexity:
+  Max: 25
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Performance/RedundantBlockCall:
+  Exclude:
+    - 'lib/puppet-lint/tasks/puppet-lint.rb'
+
+# Offense count: 7
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+# SupportedStyles: nested, compact
+Style/ClassAndModuleChildren:
+  Exclude:
+    - 'lib/puppet-lint.rb'
+    - 'lib/puppet-lint/bin.rb'
+    - 'lib/puppet-lint/checkplugin.rb'
+    - 'lib/puppet-lint/checks.rb'
+    - 'lib/puppet-lint/data.rb'
+    - 'lib/puppet-lint/optparser.rb'
+
+# Offense count: 20
+Style/MultilineBlockChain:
+  Enabled: false

data/.travis.yml

@@ -6,16 +6,18 @@ branches:
   only:
     - master
 bundler_args: --without development system_tests
+script:
+    - bundle exec rake $CHECK
+env:
+    - CHECK=test
 rvm:
-  - 1.8.7
   - 1.9.3
   - 2.0.0
-  - 2.1.9
-  - 2.3.1
-  - 2.4.0
+  - 2.1.10
+  - 2.2.7
+  - 2.3.4
+  - 2.4.1
 matrix:
-  allow_failures:
-  - rvm: 1.8.7
-notifications:
-  email:
-    - tim@bombasticmonkey.com
+    include:
+        - rvm: 2.4.1
+          env: CHECK=rubocop

data/CHANGELOG.md

@@ -1,5 +1,59 @@
 # Change Log
 
+## [2.3.1](https://github.com/rodjek/puppet-lint/tree/2.3.1) (2017-09-27)
+[Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.3.0...2.3.1)
+
+**Fixed bugs:**
+
+- NoMethodError: undefined method `type' for nil:NilClass [\#732](https://github.com/rodjek/puppet-lint/issues/732)
+- NoMethodError: undefined method `end\_with?' for nil:NilClass [\#727](https://github.com/rodjek/puppet-lint/issues/727)
+- puppet-lint not applying some lint:ignore statements when there are more then 2 on the same line [\#726](https://github.com/rodjek/puppet-lint/issues/726)
+- optional paramter warning false positve when inheriting params [\#716](https://github.com/rodjek/puppet-lint/issues/716)
+- invalid byte sequence in UTF-8 in selmodule-example.pp [\#714](https://github.com/rodjek/puppet-lint/issues/714)
+- puppet-lint --fix encountered an error that it doesn't know how to handle [\#706](https://github.com/rodjek/puppet-lint/issues/706)
+- Mangled file after running puppet-lint due to chained function call [\#703](https://github.com/rodjek/puppet-lint/issues/703)
+- `incompatible encoding regexp match` for non-printing characters in place of space [\#693](https://github.com/rodjek/puppet-lint/issues/693)
+- Unhandled error case [\#691](https://github.com/rodjek/puppet-lint/issues/691)
+
+**Closed issues:**
+
+- puppet-lint has encountered an error that it doesn't know how to handle [\#750](https://github.com/rodjek/puppet-lint/issues/750)
+- Variable use like "a+1 = ${$a + 1}" isn't reported but silently changed by --fix to "a+1 = ${a} + 1" [\#749](https://github.com/rodjek/puppet-lint/issues/749)
+- using --fix changes line endings [\#748](https://github.com/rodjek/puppet-lint/issues/748)
+- Puppet lint syntax error - puppet parser validate no issues [\#746](https://github.com/rodjek/puppet-lint/issues/746)
+- Error not handled [\#745](https://github.com/rodjek/puppet-lint/issues/745)
+- Whoops, not sure why. ArgumentError: bad value for range [\#742](https://github.com/rodjek/puppet-lint/issues/742)
+- ArgumentError: bad value for range [\#741](https://github.com/rodjek/puppet-lint/issues/741)
+- Line numbers off after multi-line strings with variables [\#736](https://github.com/rodjek/puppet-lint/issues/736)
+- Whoops! It looks like puppet-lint has encountered an error [\#729](https://github.com/rodjek/puppet-lint/issues/729)
+- puppet lint config log\_format not working [\#725](https://github.com/rodjek/puppet-lint/issues/725)
+- Quoted booleans in Puppet5 for Enum type declarations. [\#720](https://github.com/rodjek/puppet-lint/issues/720)
+- Chaining arrow syntax fix introduces trailing whitespaces [\#695](https://github.com/rodjek/puppet-lint/issues/695)
+- Refactor check\_comments.rb [\#587](https://github.com/rodjek/puppet-lint/issues/587)
+- Puppethack 12/2016 Issues [\#583](https://github.com/rodjek/puppet-lint/issues/583)
+- Allow multiple block-level ignore comments [\#498](https://github.com/rodjek/puppet-lint/issues/498)
+- puppet-lint crashes with "invalid byte sequence in UTF-8 \(ArgumentError\)" [\#458](https://github.com/rodjek/puppet-lint/issues/458)
+
+**Merged pull requests:**
+
+- Render the ${} enclosures as part of the string tokens [\#752](https://github.com/rodjek/puppet-lint/pull/752) ([rodjek](https://github.com/rodjek))
+- Open manifest as binary when writing fixed manifest [\#751](https://github.com/rodjek/puppet-lint/pull/751) ([rodjek](https://github.com/rodjek))
+- Remove monkeypatches to implement String\#% [\#744](https://github.com/rodjek/puppet-lint/pull/744) ([rodjek](https://github.com/rodjek))
+- Add unit tests for PuppetLint::Checks [\#743](https://github.com/rodjek/puppet-lint/pull/743) ([rodjek](https://github.com/rodjek))
+- Update CI configuration [\#739](https://github.com/rodjek/puppet-lint/pull/739) ([rodjek](https://github.com/rodjek))
+- Rubocop compliance [\#738](https://github.com/rodjek/puppet-lint/pull/738) ([rodjek](https://github.com/rodjek))
+- Fix line numbers being off with multi-line strings containing variables [\#737](https://github.com/rodjek/puppet-lint/pull/737) ([cbowman0](https://github.com/cbowman0))
+- Split control comments into words before parsing [\#735](https://github.com/rodjek/puppet-lint/pull/735) ([rodjek](https://github.com/rodjek))
+- Handle unicode spaces in the tokeniser [\#734](https://github.com/rodjek/puppet-lint/pull/734) ([rodjek](https://github.com/rodjek))
+- Handle SE Linux policy package files [\#733](https://github.com/rodjek/puppet-lint/pull/733) ([rodjek](https://github.com/rodjek))
+- Take into account Optional data type when checking parameter order [\#731](https://github.com/rodjek/puppet-lint/pull/731) ([rodjek](https://github.com/rodjek))
+- Read the manifest files as UTF-8 [\#730](https://github.com/rodjek/puppet-lint/pull/730) ([rodjek](https://github.com/rodjek))
+- Improve handling of unterminated double quoted strings [\#728](https://github.com/rodjek/puppet-lint/pull/728) ([rodjek](https://github.com/rodjek))
+- Chaining arrow syntax fix introduces trailing whitespaces [\#708](https://github.com/rodjek/puppet-lint/pull/708) ([rnelson0](https://github.com/rnelson0))
+- Add helper methods to add and remove tokens while maintaining the token links [\#694](https://github.com/rodjek/puppet-lint/pull/694) ([Darhazer](https://github.com/Darhazer))
+- Code style improvements [\#661](https://github.com/rodjek/puppet-lint/pull/661) ([Darhazer](https://github.com/Darhazer))
+- Plugins: Improve code readability [\#658](https://github.com/rodjek/puppet-lint/pull/658) ([Darhazer](https://github.com/Darhazer))
+
 ## [2.3.0](https://github.com/rodjek/puppet-lint/tree/2.3.0) (2017-07-12)
 [Full Changelog](https://github.com/rodjek/puppet-lint/compare/2.2.1...2.3.0)
 

data/Gemfile

@@ -25,6 +25,9 @@ group :test do
     # requires ruby 1.9+, on 1.8 we'll fall back to the old regex parsing
     gem 'rspec-json_expectations', '~> 1.4'
   end
+
+  gem 'rubocop', '0.49.1' if RUBY_VERSION > '2.0'
+  gem 'simplecov', :require => false
 end
 
 group :development do

data/Rakefile

@@ -8,12 +8,23 @@ RSpec::Core::RakeTask.new(:test)
 
 begin
   require 'github_changelog_generator/task'
-  GitHubChangelogGenerator::RakeTask.new :changelog do |config|
+  GitHubChangelogGenerator::RakeTask.new(:changelog) do |config|
     version = PuppetLint::VERSION
-    config.future_release = "#{version}"
-    config.exclude_labels = %w{duplicate question invalid wontfix release-pr}
+    config.future_release = version.to_s
+    config.exclude_labels = %w[duplicate question invalid wontfix release-pr]
   end
 rescue LoadError
+  $stderr.puts 'Changelog generation requires Ruby 2.0 or higher'
+end
+
+begin
+  require 'rubocop/rake_task'
+
+  RuboCop::RakeTask.new(:rubocop) do |task|
+    task.options = %w[-D -E]
+  end
+rescue LoadError
+  $stderr.puts 'Rubocop is not available for this version of Ruby.'
 end
 
 # vim: syntax=ruby

data/appveyor.yml

@@ -12,6 +12,7 @@ environment:
     - RUBY_VERSION: 21
     - RUBY_VERSION: 22
     - RUBY_VERSION: 23-x64
+    - RUBY_VERSION: 24-x64
 
 install:
   - SET PATH=C:\Ruby%RUBY_VERSION%\bin;%PATH%
@@ -26,7 +27,3 @@ before_test:
 
 test_script:
   - bundle exec rake test
-
-notifications:
-  email:
-    - tim@bombasticmonkey.com

data/bin/puppet-lint

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 
-$:.unshift File.join(File.dirname(__FILE__), *%w[.. lib])
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 
 require 'puppet-lint'
 

data/lib/puppet-lint.rb

@@ -1,3 +1,5 @@
+# encoding: utf-8
+
 require 'set'
 require 'json'
 require 'puppet-lint/version'
@@ -6,7 +8,6 @@ require 'puppet-lint/configuration'
 require 'puppet-lint/data'
 require 'puppet-lint/checks'
 require 'puppet-lint/bin'
-require 'puppet-lint/monkeypatches'
 
 class PuppetLint::NoCodeError < StandardError; end
 class PuppetLint::NoFix < StandardError; end
@@ -47,7 +48,7 @@ class PuppetLint
   # Public: Initialise a new PuppetLint object.
   def initialize
     @code = nil
-    @statistics = {:error => 0, :warning => 0, :fixed => 0, :ignored => 0}
+    @statistics = { :error => 0, :warning => 0, :fixed => 0, :ignored => 0 }
     @manifest = ''
   end
 
@@ -70,10 +71,16 @@ class PuppetLint
   #
   # Returns nothing.
   def file=(path)
-    if File.exist? path
-      @path = path
-      @code = File.read(path)
+    return unless File.exist?(path)
+
+    @path = path
+    File.open(path, 'r:UTF-8') do |f|
+      @code = f.read
     end
+
+    # Check if the input is an SE Linux policy package file (which also use
+    # the .pp extension), which all have the first 4 bytes 0xf97cff8f.
+    @code = '' if @code[0..3].unpack('V').first == 0xf97cff8f
   end
 
   # Internal: Retrieve the format string to be used when writing problems to
@@ -85,12 +92,11 @@ class PuppetLint
     if configuration.log_format == ''
       ## recreate previous old log format as far as thats possible.
       format = '%{KIND}: %{message} on line %{line}'
-      if configuration.with_filename
-        format.prepend '%{path} - '
-      end
+      format.prepend('%{path} - ') if configuration.with_filename
       configuration.log_format = format
     end
-    return configuration.log_format
+
+    configuration.log_format
   end
 
   # Internal: Format a problem message and print it to STDOUT.
@@ -101,9 +107,8 @@ class PuppetLint
   def format_message(message)
     format = log_format
     puts format % message
-    if message[:kind] == :ignored && !message[:reason].nil?
-      puts "  #{message[:reason]}"
-    end
+
+    puts "  #{message[:reason]}" if message[:kind] == :ignored && !message[:reason].nil?
   end
 
   # Internal: Get the line of the manifest on which the problem was found
@@ -112,8 +117,7 @@ class PuppetLint
   #
   # Returns the problematic line as a string.
   def get_context(message)
-    line = PuppetLint::Data.manifest_lines[message[:line] - 1]
-    return line.strip
+    PuppetLint::Data.manifest_lines[message[:line] - 1].strip
   end
 
   # Internal: Print out the line of the manifest on which the problem was found
@@ -126,9 +130,9 @@ class PuppetLint
     return if message[:check] == 'documentation'
     return if message[:kind] == :fixed
     line = get_context(message)
-    offset = line.index(/\S/) || 1
+    offset = line.index(%r{\S}) || 1
     puts "\n  #{line.strip}"
-    printf "%#{message[:column] + 2 - offset}s\n\n", '^'
+    printf("%#{message[:column] + 2 - offset}s\n\n", '^')
   end
 
   # Internal: Print the reported problems with a manifest to stdout.
@@ -149,12 +153,14 @@ class PuppetLint
           message['context'] = get_context(message) if configuration.with_context
           json << message
         else
-          format_message message
+          format_message(message)
           print_context(message) if configuration.with_context
         end
       end
     end
     puts JSON.pretty_generate(json) if configuration.json
+
+    $stderr.puts 'Try running `puppet parser validate <file>`' if problems.any? { |p| p[:check] == :syntax }
   end
 
   # Public: Determine if PuppetLint found any errors in the manifest.
@@ -177,9 +183,7 @@ class PuppetLint
   # Returns nothing.
   # Raises PuppetLint::NoCodeError if no manifest code has been loaded.
   def run
-    if @code.nil?
-      raise PuppetLint::NoCodeError
-    end
+    raise PuppetLint::NoCodeError if @code.nil?
 
     if @code.empty?
       @problems = []
@@ -198,7 +202,7 @@ class PuppetLint
   #
   # Returns nothing.
   def print_problems
-    report @problems
+    report(@problems)
   end
 
   # Public: Define a new check.