puppet-syntax 2.6.1 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5015bb344c1e58bce99c9eb8c93c6f011c85275c9e0748f34199441c9f965ba0
-  data.tar.gz: 1316c65fe87f5adc8bdd2faa7febd073298d2a8401203efbdca2eb3054c477f1
+  metadata.gz: 5efa461330416f6cffe0518ff6af0c111b5c7731d0f61ad10fc52947181a5d7d
+  data.tar.gz: 5050d5e596af634f1280d41155e5397e9dd8e21e21cae6f9c5ed464b5ab76026
 SHA512:
-  metadata.gz: 3b528fdd903665e128d3171b7e40459d37e21907049185b1db37408c5e12548dcd1695391f5343279ddee1aa0e79a2721228faa410edd2b94e8c282b6fb92c04
-  data.tar.gz: 06f8d2a184c2fd74349fcb937609a12526558a1cdba1071d6dbf4693b71b428ef913d814862a1c7311f54913e2a240159daebaf1b3b7437d40080e1552f3c3b3
+  metadata.gz: ba169dc6d36d2d4fc68a661b8fbd4e1f090ecb94d978c77c692003487b82b86646e3965a42afe5e50b654aaae904ec89c4fbfb5576f4d3014adabeedaf90b7cb
+  data.tar.gz: b1be324753efdaa626f242d8d53ffad7e81804bb02334a8200d610a880c038f79bda94af125b7188c6d52ff4f00589cd93637614c3d5a37549bd18d3ffb15db9

data/.gitignore

@@ -16,4 +16,5 @@ test/tmp
 test/version_tmp
 tmp
 vendor/
-.idea/
+.idea/
+.vendor/

data/.travis.yml

@@ -3,7 +3,7 @@ dist: bionic
 language: ruby
 # Workaround https://github.com/bundler/bundler/issues/3558
 before_install: gem install bundler
-install: bundle install --jobs 4 --retry 2 --without development
+install: bundle install --jobs 4 --retry 2 --without development release
 script: bundle exec rake
 rvm:
   - 2.4.4

data/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 All notable changes to this project will be documented in this file.
 
+## [v3.0.0](https://github.com/voxpupuli/puppet-syntax/tree/v3.0.0) (2020-05-09)
+
+[Full Changelog](https://github.com/voxpupuli/puppet-syntax/compare/v2.6.1...v3.0.0)
+
+**Breaking changes:**
+
+- Drop legacy code to support Puppet \< 5 and Ruby \< 2.4 [\#120](https://github.com/voxpupuli/puppet-syntax/pull/120) ([ekohl](https://github.com/ekohl))
+
+**Implemented enhancements:**
+
+- Migrate Changelog to GCG [\#93](https://github.com/voxpupuli/puppet-syntax/issues/93)
+
+**Merged pull requests:**
+
+- cleanup README.md / fix markdown linter warnings [\#119](https://github.com/voxpupuli/puppet-syntax/pull/119) ([bastelfreak](https://github.com/bastelfreak))
+
 ## [v2.6.1](https://github.com/voxpupuli/puppet-syntax/tree/v2.6.1) (2020-01-11)
 
 [Full Changelog](https://github.com/voxpupuli/puppet-syntax/compare/v2.6.0...v2.6.1)
@@ -26,7 +42,6 @@ All notable changes to this project will be documented in this file.
 
 **Merged pull requests:**
 
-- Test on ruby 2.6 [\#111](https://github.com/voxpupuli/puppet-syntax/pull/111) ([alexjfisher](https://github.com/alexjfisher))
 - Adding KMS tags to allowed EYAML methods [\#105](https://github.com/voxpupuli/puppet-syntax/pull/105) ([craigwatson](https://github.com/craigwatson))
 
 ## [v2.5.0](https://github.com/voxpupuli/puppet-syntax/tree/v2.5.0) (2019-07-07)

data/Gemfile

@@ -21,13 +21,10 @@ gemspec
 # But only if the environment variable is set
 gem 'puppet', *location_for(ENV['PUPPET_VERSION'] || '>= 5') if ENV['PUPPET_VERSION']
 
-# older version required for ruby 1.9 compat, as it is pulled in as dependency of puppet, this has to be carried by the module
-gem 'json_pure', '<= 2.0.1'
-
 group :test do
   gem 'rspec'
 end
 
 group :release do
-  gem 'github_changelog_generator',  :require => false, :git => 'https://github.com/github-changelog-generator/github-changelog-generator' if RUBY_VERSION >= '2.2.2'
+  gem 'github_changelog_generator',  :require => false, :git => 'https://github.com/github-changelog-generator/github-changelog-generator'
 end

data/README.md

@@ -4,36 +4,40 @@
 
 # Puppet::Syntax
 
-Puppet::Syntax checks for correct syntax in Puppet manifests, templates, and Hiera YAML.
+Puppet::Syntax checks for correct syntax in Puppet manifests, templates, and
+Hiera YAML.
 
 ## Version support
 
 Puppet::Syntax is supported with:
 
-- Puppet >= 2.7 that provides the `validate` face.
-- Ruby >= 1.8 with `erb` from Ruby stdlib.
+- Puppet >= 5.0 that provides the `validate` face.
+- Ruby >= 2.4
 
 For the specific versions that we test against, see the [TravisCI config](.travis.yml).
 
-If you're using `puppetlabs_spec_helper/rake_tasks` and getting unexpected non-zero exit codes, upgrade to [puppetlabs_spec_helper][psh] version 0.8.0 or greater. Versions of `puppetlabs_spec_helper` prior to 0.8.0 conflicted with Puppet::Syntax.
-
-[psh]: https://github.com/puppetlabs/puppetlabs_spec_helper
-
 ## Installation
 
-To install Puppet::Syntax, either add it to your module's Gemfile or install the gem manually.
+To install Puppet::Syntax, either add it to your module's Gemfile or install
+the gem manually.
 
 * To install with the Gemfile, add:
 
-        gem 'puppet-syntax'
+```ruby
+gem 'puppet-syntax'
+```
 
   And then execute:
 
-        $ bundle install
+```sh
+bundle install
+```
 
 * To install the gem yourself, run:
 
-        $ gem install puppet-syntax
+```sh
+gem install puppet-syntax
+```
 
 ## Configuration
 
@@ -41,75 +45,85 @@ To configure Puppet::Syntax, add any of the following settings to your `Rakefile
 
 * To exclude certain paths from the syntax checks, set:
 
-        PuppetSyntax.exclude_paths = ["vendor/**/*"]
-
-* To use the Puppet 4 ("future") parser in Puppet 3.2 through 3.8, set:
-
-        PuppetSyntax.future_parser = true
+```ruby
+PuppetSyntax.exclude_paths = ["vendor/**/*"]
+```
 
 * To configure specific paths for the Hiera syntax check, specify `hieradata_paths`. This is useful if you use Hiera data inside your module.
 
-        PuppetSyntax.hieradata_paths = ["**/data/**/*.yaml", "hieradata/**/*.yaml", "hiera*.yaml"]
+```ruby
+PuppetSyntax.hieradata_paths = ["**/data/**/*.yaml", "hieradata/**/*.yaml", "hiera*.yaml"]
+```
 
 * To configure specific paths for the Puppet syntax checks or for the templates checks, specify `manifests_paths` or `templates_paths` respectively. This is useful if you want to check specific paths only.
 
-        PuppetSyntax.manifests_paths = ["**/environments/future/*.pp"]
-        PuppetSyntax.templates_paths = ["**/modules/**/templates/*.erb"]
-
-* To validate the syntax of code written for application orchestration, enable `app_management`:
-
-        PuppetSyntax.app_management = true
-
-  The `app_management` setting is supported with Puppet 4.3 or greater and is off by default. In Puppet 5, app_management is always enabled.
+```ruby
+PuppetSyntax.manifests_paths = ["**/environments/future/*.pp"]
+PuppetSyntax.templates_paths = ["**/modules/**/templates/*.erb"]
+```
 
 * To ignore deprecation warnings, disable `fail_on_deprecation_notices`. By default, `puppet-syntax` fails if it encounters Puppet deprecation notices. If you are working with a legacy code base and want to ignore such non-fatal warnings, you might want to override the default behavior.
 
-        PuppetSyntax.fail_on_deprecation_notices = false
+```ruby
+PuppetSyntax.fail_on_deprecation_notices = false
+```
 
 * To enable a syntax check on Hiera keys, set:
 
-        PuppetSyntax.check_hiera_keys = true
+```ruby
+PuppetSyntax.check_hiera_keys = true
+```
 
-   This reports common mistakes in key names in Hiera files, such as:
+This reports common mistakes in key names in Hiera files, such as:
 
-  - Leading `::` in keys, such as: `::notsotypical::warning2: true`.
-  - Single colon scope separators, such as: `:picky::warning5: true`.
-  - Invalid camel casing, such as: `noCamelCase::warning3: true`.
-  - Use of hyphens, such as: `no-hyphens::warning4: true`.
+* Leading `::` in keys, such as: `::notsotypical::warning2: true`.
+* Single colon scope separators, such as: `:picky::warning5: true`.
+* Invalid camel casing, such as: `noCamelCase::warning3: true`.
+* Use of hyphens, such as: `no-hyphens::warning4: true`.
 
 ## Usage
 
 * To enable Puppet::Syntax, include the following in your module's `Rakefile`:
 
-        require 'puppet-syntax/tasks/puppet-syntax'
+```ruby
+require 'puppet-syntax/tasks/puppet-syntax'
+```
 
-  For Continuous Integration, use Puppet::Syntax in conjunction with `puppet-lint` and spec tests. Add the following to your module's `Rakefile`:
+For Continuous Integration, use Puppet::Syntax in conjunction with `puppet-lint`
+and spec tests. Add the following to your module's `Rakefile`:
 
-        task :test => [
-          :syntax,
-          :lint,
-          :spec,
-        ]
+```ruby
+task :test => [
+  :syntax,
+  :lint,
+  :spec,
+]
+```
 
 * To test all manifests and templates, relative to the location of the `Rakefile`, run:
 
-        $ bundle exec rake syntax
-        ---> syntax:manifests
-        ---> syntax:templates
-        ---> syntax:hiera:yaml
+```
+$ bundle exec rake syntax
+---> syntax:manifests
+---> syntax:templates
+---> syntax:hiera:yaml
+```
 
 * To return a non-zero exit code and an error message on any failures, run:
 
-        $ bundle exec rake syntax
-        ---> syntax:manifests
-        rake aborted!
-        Could not parse for environment production: Syntax error at end of file at demo.pp:2
-        Tasks: TOP => syntax => syntax:manifests
-        (See full trace by running task with --trace)
+```
+$ bundle exec rake syntax
+---> syntax:manifests
+rake aborted!
+Could not parse for environment production: Syntax error at end of file at demo.pp:2
+Tasks: TOP => syntax => syntax:manifests
+(See full trace by running task with --trace)
+```
 
 ## Checks
 
-Puppet::Syntax makes the following checks in the directories and subdirectories of the module, relative to the location of the `Rakefile`.
+Puppet::Syntax makes the following checks in the directories and subdirectories
+of the module, relative to the location of the `Rakefile`.
 
 ### Hiera
 
@@ -135,7 +149,8 @@ Checks `.erb` files in the module for syntax errors.
 
 Checks `.epp` files in the module for syntax errors.
 
-EPP checks are supported in Puppet 4 or greater, or in Puppet 3 with the future parser enabled.
+EPP checks are supported in Puppet 4 or greater, or in Puppet 3 with the future
+parser enabled.
 
 ## Contributing
 

data/lib/puppet-syntax.rb

@@ -6,7 +6,6 @@ require "puppet/version"
 
 module PuppetSyntax
   @exclude_paths = []
-  @future_parser = false
   @hieradata_paths = [
     "**/data/**/*.*{yaml,yml}",
     "hieradata/**/*.*{yaml,yml}",
@@ -20,23 +19,15 @@ module PuppetSyntax
     '**/templates/**/*.epp'
   ]
   @fail_on_deprecation_notices = true
-  @app_management = Puppet.version.to_i >= 5 ? true : false
   @check_hiera_keys = false
 
   class << self
     attr_accessor :exclude_paths,
-                  :future_parser,
                   :hieradata_paths,
                   :manifests_paths,
                   :templates_paths,
                   :fail_on_deprecation_notices,
                   :epp_only,
                   :check_hiera_keys
-    attr_reader :app_management
-
-    def app_management=(app_management)
-      raise 'app_management cannot be disabled on Puppet 5 or higher' if Puppet.version.to_i >= 5 && !app_management
-      @app_management = app_management
-    end
   end
 end

data/lib/puppet-syntax/manifests.rb

@@ -8,9 +8,7 @@ module PuppetSyntax
 
       output = []
 
-      if Puppet::Test::TestHelper.respond_to?(:initialize) # 3.1+
-        Puppet::Test::TestHelper.initialize
-      end
+      Puppet::Test::TestHelper.initialize
       Puppet::Test::TestHelper.before_all_tests
       called_before_all_tests = true
 
@@ -62,8 +60,6 @@ module PuppetSyntax
 
     private
     def validate_manifest(file)
-      Puppet[:parser] = 'future' if PuppetSyntax.future_parser and Puppet.version.to_i < 4
-      Puppet[:app_management] = true if PuppetSyntax.app_management && (Puppet::Util::Package.versioncmp(Puppet.version, '4.3.0') >= 0 && Puppet.version.to_i < 5)
       Puppet[:tasks] = true if Puppet::Util::Package.versioncmp(Puppet.version, '5.4.0') >= 0 and file.match(/.*plans\/.*\.pp$/)
       Puppet::Face[:parser, :current].validate(file)
     end

data/lib/puppet-syntax/tasks/puppet-syntax.rb

@@ -27,42 +27,14 @@ module PuppetSyntax
     def initialize(*args)
       desc 'Syntax check Puppet manifests and templates'
       task :syntax => [
-        'syntax:check_puppetlabs_spec_helper',
         'syntax:manifests',
         'syntax:templates',
         'syntax:hiera',
       ]
 
       namespace :syntax do
-        task :check_puppetlabs_spec_helper do
-          psh_present = task(:syntax).actions.any? { |a|
-            a.inspect.match(/puppetlabs_spec_helper\/rake_tasks\.rb:\d+/)
-          }
-
-          if psh_present
-            $stderr.puts <<-EOS
-[WARNING] A conflicting :syntax rake task has been defined by
-puppetlabs_spec_helper/rake_tasks. You should either disable this or upgrade
-to puppetlabs_spec_helper >= 0.8.0 which now uses puppet-syntax.
-            EOS
-          end
-        end
-
         desc 'Syntax check Puppet manifests'
         task :manifests do |t|
-          if Puppet.version.to_i >= 4 and PuppetSyntax.future_parser
-            $stderr.puts <<-EOS
-[INFO] Puppet 4 has been detected and `future_parser` has been set to
-'true'. The `future_parser setting will be ignored.
-            EOS
-          end
-          if Puppet::Util::Package.versioncmp(Puppet.version, '4.3.0') < 0 and PuppetSyntax.app_management
-            $stderr.puts <<-EOS
-[WARNING] Puppet `app_management` has been detected but the Puppet
-version is less then 4.3.  The `app_management` setting will be ignored.
-            EOS
-          end
-
           $stderr.puts "---> #{t.name}"
 
           c = PuppetSyntax::Manifests.new

data/lib/puppet-syntax/templates.rb

@@ -27,10 +27,6 @@ module PuppetSyntax
     end
 
     def validate_epp(filename)
-      if Puppet.version.to_f < 3.7
-        raise "Cannot validate EPP without Puppet 4 or future parser (3.7+)"
-      end
-
       require 'puppet/pops'
       errors = []
       begin

data/lib/puppet-syntax/version.rb

@@ -1,3 +1,3 @@
 module PuppetSyntax
-  VERSION = '2.6.1'
+  VERSION = '3.0.0'
 end

data/puppet-syntax.gemspec

@@ -18,6 +18,8 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
+  spec.required_ruby_version = ">= 2.4"
+
   spec.add_dependency "rake"
   spec.add_dependency "puppet", ">= 5"
 

data/spec/puppet-syntax/manifests_spec.rb

@@ -28,15 +28,9 @@ describe PuppetSyntax::Manifests do
     files = fixture_manifests('fail_error.pp')
     output, has_errors = subject.check(files)
 
-    if Puppet.version.to_i >= 4
-      expect(output.size).to eq(3)
-      expect(output[2]).to match(/2 errors. Giving up/)
-      expect(has_errors).to eq(true)
-    else
-      expect(output.size).to eq(1)
-      expect(output[0]).to match(/Syntax error at .*:3$/)
-      expect(has_errors).to eq(true)
-    end
+    expect(output.size).to eq(3)
+    expect(output[2]).to match(/2 errors. Giving up/)
+    expect(has_errors).to eq(true)
   end
 
   it 'should return a warning from an invalid file' do
@@ -72,181 +66,22 @@ describe PuppetSyntax::Manifests do
     output, has_errors = subject.check(files)
 
     expect(has_errors).to eq(true)
-    if Puppet.version.to_i >= 4
-      expect(output.size).to eq(5)
-      expect(output[0]).to match(/This Name has no effect. A Host Class Definition can not end with a value-producing expression without other effect \(file: \S*\/fail_error.pp, line: 2, column: 32\)$/)
-      expect(output[1]).to match(/This Name has no effect. A value was produced and then forgotten \(one or more preceding expressions may have the wrong form\) \(file: \S*\/fail_error.pp, line: 2, column: 3\)$/)
-      expect(output[2]).to match('2 errors. Giving up')
-      expect(output[3]).to match(/Unrecogni(s|z)ed escape sequence '\\\['/)
-      expect(output[4]).to match(/Unrecogni(s|z)ed escape sequence '\\\]'/)
-    else
-      expect(output.size).to eq(3)
-      expect(output[0]).to match(/Syntax error at '\}' .*:3$/)
-      expect(output[1]).to match(/Unrecogni(s|z)ed escape sequence '\\\['/)
-      expect(output[2]).to match(/Unrecogni(s|z)ed escape sequence '\\\]'/)
-    end
+    expect(output.size).to eq(5)
+    expect(output[0]).to match(/This Name has no effect. A Host Class Definition can not end with a value-producing expression without other effect \(file: \S*\/fail_error.pp, line: 2, column: 32\)$/)
+    expect(output[1]).to match(/This Name has no effect. A value was produced and then forgotten \(one or more preceding expressions may have the wrong form\) \(file: \S*\/fail_error.pp, line: 2, column: 3\)$/)
+    expect(output[2]).to match('2 errors. Giving up')
+    expect(output[3]).to match(/Unrecogni(s|z)ed escape sequence '\\\['/)
+    expect(output[4]).to match(/Unrecogni(s|z)ed escape sequence '\\\]'/)
   end
 
   describe 'deprecation notices' do
-    case Puppet.version.to_f
-    when 4.0..4.99
-      context 'on puppet 4.0.0 and above' do
-        it 'should instead be failures' do
-          files = fixture_manifests('deprecation_notice.pp')
-          output, has_errors = subject.check(files)
-
-          expect(has_errors).to eq(true)
-          expect(output.size).to eq(1)
-          expect(output[0]).to match (/Node inheritance is not supported in Puppet >= 4.0.0/)
-        end
-      end
-    when 3.7, 3.8
-      context 'on puppet 3.7 and 3.8' do
-        it 'should return deprecation notices as warnings' do
-          files = fixture_manifests('deprecation_notice.pp')
-          output, has_errors = subject.check(files)
-
-          expect(has_errors).to eq(false)
-          expect(output.size).to eq(2)
-          expect(output[0]).to match(/The use of 'import' is deprecated/)
-          expect(output[1]).to match(/Deprecation notice:/)
-        end
-      end
-    when 3.5, 3.6
-      context 'on puppet 3.5 and 3.6' do
-        it 'should return deprecation notices as warnings' do
-          files = fixture_manifests('deprecation_notice.pp')
-          output, has_errors = subject.check(files)
-
-          expect(has_errors).to eq(false)
-          expect(output.size).to eq(1)
-          expect(output[0]).to match(/The use of 'import' is deprecated/)
-        end
-      end
-    when 3.0..3.4
-      context 'on puppet 3.0 to 3.4' do
-        it 'should not print deprecation notices' do
-          files = fixture_manifests('deprecation_notice.pp')
-          output, has_errors = subject.check(files)
-
-          expect(output).to eq([])
-          expect(has_errors).to eq(false)
-        end
-      end
-    end
-  end
-
-  describe 'app_management' do
-    after do
-      PuppetSyntax.app_management = false if Puppet.version.to_i < 5
-    end
-
-    context 'app_management = false (default)', :if => (Puppet.version.to_i < 5) do
-      it 'should fail to parse an application manifest' do
-
-        files = fixture_manifests(['test_app.pp'])
-        output,has_errors = subject.check(files)
-
-        expect(has_errors).to eq(true)
-        expect(output).to include(/error/)
-      end
-    end
-    context 'app_management = true' do
-      before(:each) {
-        PuppetSyntax.app_management = true
-      }
-      if Puppet::Util::Package.versioncmp(Puppet.version, '4.3.0') >= 0
-        it 'should successfully parse an application manifest on Puppet >= 4.3.0' do
-          expect(PuppetSyntax.app_management).to eq(true)
-
-          files = fixture_manifests(['test_app.pp'])
-          output,has_errors = subject.check(files)
-
-          expect(output.size).to eq(0)
-          expect(has_errors).to eq(false)
-        end
-      else
-        it 'should fail to parse an application manifest on Puppet < 4.3.0' do
-          expect(PuppetSyntax.app_management).to eq(true)
-
-          files = fixture_manifests(['test_app.pp'])
-          output,has_errors = subject.check(files)
-
-          expect(output).to include(/error/)
-          expect(has_errors).to eq(true)
-        end
-      end
-    end
-  end
-
-
-  describe 'future_parser' do
-    context 'future_parser = false (default)' do
-      if Puppet::Util::Package.versioncmp(Puppet.version, '4.0') < 0
-        it 'should fail without setting future option to true on future manifest on Puppet < 4.0.0' do
-          expect(PuppetSyntax.future_parser).to eq(false)
-
-          files = fixture_manifests(['future_syntax.pp'])
-          output, has_errors = subject.check(files)
-
-          expect(output.size).to eq(1)
-          expect(output[0]).to match(/Syntax error at '='; expected '\}' .*:2$/)
-          expect(has_errors).to eq(true)
-        end
-      else
-        it 'should succeed on Puppet >= 4.0.0' do
-          expect(PuppetSyntax.future_parser).to eq(false)
+    it 'should instead be failures' do
+      files = fixture_manifests('deprecation_notice.pp')
+      output, has_errors = subject.check(files)
 
-          files = fixture_manifests(['future_syntax.pp'])
-          output, has_errors = subject.check(files)
-
-          expect(output.size).to eq(0)
-          expect(has_errors).to eq(false)
-        end
-      end
-    end
-
-    context 'future_parser = true' do
-      before(:each) {
-        PuppetSyntax.future_parser = true
-      }
-
-      if Puppet::Util::Package.versioncmp(Puppet.version, '3.2') >= 0 and Puppet.version.to_i < 4
-        context 'Puppet >= 3.2 < 4' do
-          it 'should pass with future option set to true on future manifest' do
-            files = fixture_manifests(['future_syntax.pp'])
-            output, has_errors = subject.check(files)
-
-            expect(output).to eq([])
-            expect(has_errors).to eq(false)
-          end
-        end
-        context 'Puppet >= 3.7 < 4' do
-          # Certain elements of the future parser weren't added until 3.7
-          if Puppet::Util::Package.versioncmp(Puppet.version, '3.7') >= 0
-            it 'should fail on what were deprecation notices in the non-future parser' do
-              files = fixture_manifests('deprecation_notice.pp')
-              output, has_errors = subject.check(files)
-
-              expect(output.size).to eq(1)
-              expect(output[0]).to match(/Node inheritance is not supported/)
-              expect(has_errors).to eq(true)
-            end
-          end
-        end
-      elsif Puppet::Util::Package.versioncmp(Puppet.version, '3.2') < 0
-        context 'Puppet < 3.2' do
-          it 'should return an error that the parser option is not supported' do
-            files = fixture_manifests(['future_syntax.pp'])
-            output, has_errors = subject.check(files)
-
-            expect(output.size).to eq(1)
-            expect(output[0]).to match("Attempt to assign a value to unknown configuration parameter :parser")
-            expect(has_errors).to eq(true)
-          end
-        end
-      end
+      expect(has_errors).to eq(true)
+      expect(output.size).to eq(1)
+      expect(output[0]).to match (/Node inheritance is not supported in Puppet >= 4.0.0/)
     end
   end
-
 end

data/spec/puppet-syntax/templates_spec.rb

@@ -74,71 +74,47 @@ describe PuppetSyntax::Templates do
     expect(res).to match([])
   end
 
-  if Puppet.version.to_f < 3.7
-    context 'on Puppet < 3.7' do
-      it 'should throw an exception when parsing EPP files' do
-        file = fixture_templates('pass.epp')
-        expect{ subject.check(file) }.to raise_error(/Cannot validate EPP without Puppet 4/)
-      end
-
-      context "when the 'epp_only' options is set" do
-        before(:each) {
-          PuppetSyntax.epp_only = true
-        }
-
-        it 'should throw an exception for any file' do
-          file = fixture_templates('pass.erb')
-          expect{ subject.check(file) }.to raise_error(/Cannot validate EPP without Puppet 4/)
-        end
-      end
-    end
-  end
-
-  if Puppet.version.to_f >= 3.7
-    context 'on Puppet >= 3.7' do
-      it 'should return nothing from a valid file' do
-        files = fixture_templates('pass.epp')
-        res = subject.check(files)
+  it 'should return nothing from a valid file' do
+    files = fixture_templates('pass.epp')
+    res = subject.check(files)
 
-        expect(res).to match([])
-      end
+    expect(res).to match([])
+  end
 
-      it 'should catch SyntaxError' do
-        files = fixture_templates('fail_error.epp')
-        res = subject.check(files)
+  it 'should catch SyntaxError' do
+    files = fixture_templates('fail_error.epp')
+    res = subject.check(files)
 
-        expect(res.size).to eq(1)
-        expect(res[0]).to match(/This Type-Name has no effect/)
-      end
+    expect(res.size).to eq(1)
+    expect(res[0]).to match(/This Type-Name has no effect/)
+  end
 
-      it 'should read more than one valid file' do
-        files = fixture_templates(['pass.epp', 'pass_also.epp'])
-        res = subject.check(files)
+  it 'should read more than one valid file' do
+    files = fixture_templates(['pass.epp', 'pass_also.epp'])
+    res = subject.check(files)
 
-        expect(res).to match([])
-      end
+    expect(res).to match([])
+  end
 
-      it 'should continue after finding an error in the first file' do
-        files = fixture_templates(['fail_error.epp', 'fail_error_also.epp'])
-        res = subject.check(files)
+  it 'should continue after finding an error in the first file' do
+    files = fixture_templates(['fail_error.epp', 'fail_error_also.epp'])
+    res = subject.check(files)
 
-        expect(res.size).to eq(2)
-        expect(res[0]).to match(/This Type-Name has no effect/)
-        expect(res[1]).to match(/Syntax error at '}' \(file: \S*\/fail_error_also.epp, line: 2, column: 4\)/)
-      end
+    expect(res.size).to eq(2)
+    expect(res[0]).to match(/This Type-Name has no effect/)
+    expect(res[1]).to match(/Syntax error at '}' \(file: \S*\/fail_error_also.epp, line: 2, column: 4\)/)
+  end
 
-      context "when the 'epp_only' options is set" do
-        before(:each) {
-          PuppetSyntax.epp_only = true
-        }
+  context "when the 'epp_only' options is set" do
+    before(:each) {
+      PuppetSyntax.epp_only = true
+    }
 
-        it 'should process an ERB as EPP and find an error' do
-          files = fixture_templates('pass.erb')
-          res = subject.check(files)
+    it 'should process an ERB as EPP and find an error' do
+      files = fixture_templates('pass.erb')
+      res = subject.check(files)
 
-          expect(res.size).to eq(1)
-        end
-      end
+      expect(res.size).to eq(1)
     end
   end
 end

data/spec/puppet-syntax_spec.rb

@@ -3,7 +3,6 @@ require 'spec_helper'
 describe PuppetSyntax do
   after do
     PuppetSyntax.exclude_paths = []
-    PuppetSyntax.app_management = false if Puppet.version.to_i < 5
   end
 
   it 'should default exclude_paths to include the pkg directory' do
@@ -20,20 +19,6 @@ describe PuppetSyntax do
     expect(PuppetSyntax.exclude_paths).to eq(["foo"])
   end
 
-  it 'should support future parser setting' do
-    PuppetSyntax.future_parser = true
-    expect(PuppetSyntax.future_parser).to eq(true)
-  end
-
-  it 'should support app_management setting setting' do
-    PuppetSyntax.app_management = true
-    expect(PuppetSyntax.app_management).to eq(true)
-  end
-
-  it 'should raise error when app_management is disabled on 5.x', :if => (Puppet.version.to_i >= 5) do
-    expect { PuppetSyntax.app_management = false }.to raise_error(/app_management cannot be disabled on Puppet 5 or higher/)
-  end
-
   it 'should support a fail_on_deprecation_notices setting' do
     PuppetSyntax.fail_on_deprecation_notices = false
     expect(PuppetSyntax.fail_on_deprecation_notices).to eq(false)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-syntax
 version: !ruby/object:Gem::Version
-  version: 2.6.1
+  version: 3.0.0
 platform: ruby
 authors:
 - Vox Pupuli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-11 00:00:00.000000000 Z
+date: 2020-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -95,7 +95,6 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
-- jenkins.sh
 - lib/puppet-syntax.rb
 - lib/puppet-syntax/hiera.rb
 - lib/puppet-syntax/manifests.rb
@@ -149,7 +148,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/jenkins.sh

@@ -1,10 +0,0 @@
-#!/usr/bin/env bash
-set -eu
-
-# Avoid Psych bug in Ruby 1.9.3p0 on Ubuntu 12.04
-export RBENV_VERSION="1.9.3"
-
-rm -f Gemfile.lock
-bundle install --path "${HOME}/bundles/${JOB_NAME}" --shebang ruby
-bundle exec rake
-bundle exec rake publish_gem