aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--.gitignore1
-rw-r--r--.gitlab-ci.yml75
-rw-r--r--.pdkignore1
-rw-r--r--.rubocop.yml12
-rw-r--r--.sync.yml3
-rw-r--r--.travis.yml16
-rw-r--r--Gemfile62
-rw-r--r--Rakefile5
-rw-r--r--appveyor.yml17
-rw-r--r--metadata.json6
-rw-r--r--spec/spec_helper.rb6
11 files changed, 67 insertions, 137 deletions
diff --git a/.gitignore b/.gitignore
index 56efb9c..49bc2a4 100644
--- a/.gitignore
+++ b/.gitignore
@@ -19,4 +19,5 @@
/tmp/
/vendor/
/convert_report.txt
+/update_report.txt
.DS_Store
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 35e4209..d651650 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1,70 +1,41 @@
---
stages:
- - test_2.4.1
- - test_2.1.9
+ - syntax
+ - unit
+
+cache:
+ paths:
+ - vendor/bundle
before_script:
- bundle -v
- rm Gemfile.lock || true
- gem update --system
- - gem update bundler
- gem --version
- bundle -v
- - bundle install --without system_tests
-
-rubocop-2.4.1:
- stage: test_2.4.1
- image: ruby:2.4.1
- script:
- - bundle exec rake rubocop
-
-syntax-2.4.1:
- stage: test_2.4.1
- image: ruby:2.4.1
- script:
- - bundle exec rake syntax lint
+ - bundle install --without system_tests --path vendor/bundle --jobs $(nproc)
-metadata-2.4.1:
- stage: test_2.4.1
- image: ruby:2.4.1
- script:
- - bundle exec rake metadata_lint
-
-rspec-puppet-2.4.1:
- stage: test_2.4.1
- image: ruby:2.4.1
- variables:
- PUPPET_GEM_VERSION: ~> 4.0
- CHECK: spec
- script:
- - bundle update
- - bundle exec rake $CHECK
-
-rubocop-2.1.9:
- stage: test_2.1.9
+parallel_spec-Ruby 2.1.9-Puppet ~> 4.0:
+ stage: syntax
image: ruby:2.1.9
script:
- - bundle exec rake rubocop
+ - bundle exec rake parallel_spec
+ variables:
+ PUPPET_GEM_VERSION: '~> 4.0'
-syntax-2.1.9:
- stage: test_2.1.9
- image: ruby:2.1.9
+syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop-Ruby 2.4.4-Puppet ~> 5.5:
+ stage: syntax
+ image: ruby:2.4.4
script:
- - bundle exec rake syntax lint
+ - bundle exec rake syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop
+ variables:
+ PUPPET_GEM_VERSION: '~> 5.5'
-metadata-2.1.9:
- stage: test_2.1.9
- image: ruby:2.1.9
+parallel_spec-Ruby 2.4.4-Puppet ~> 5.5:
+ stage: syntax
+ image: ruby:2.4.4
script:
- - bundle exec rake metadata_lint
-
-rspec-puppet-2.1.9:
- stage: test_2.1.9
- image: ruby:2.1.9
+ - bundle exec rake parallel_spec
variables:
- PUPPET_GEM_VERSION: ~> 4.0
- CHECK: spec
- script:
- - bundle update
- - bundle exec rake $CHECK
+ PUPPET_GEM_VERSION: '~> 5.5'
diff --git a/.pdkignore b/.pdkignore
index 56efb9c..49bc2a4 100644
--- a/.pdkignore
+++ b/.pdkignore
@@ -19,4 +19,5 @@
/tmp/
/vendor/
/convert_report.txt
+/update_report.txt
.DS_Store
diff --git a/.rubocop.yml b/.rubocop.yml
index 40a58e0..7ed6225 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -8,11 +8,14 @@ AllCops:
Exclude:
- bin/*
- ".vendor/**/*"
- - Gemfile
- - Rakefile
+ - "**/Gemfile"
+ - "**/Rakefile"
- pkg/**/*
- spec/fixtures/**/*
- vendor/**/*
+ - "**/Puppetfile"
+ - "**/Vagrantfile"
+ - "**/Guardfile"
Metrics/LineLength:
Description: People have wide screens, use them.
Max: 200
@@ -65,6 +68,11 @@ Style/SymbolArray:
EnforcedStyle: brackets
RSpec/MessageSpies:
EnforcedStyle: receive
+Style/Documentation:
+ Exclude:
+ - lib/puppet/parser/functions/**/*
+Style/WordArray:
+ EnforcedStyle: brackets
Style/CollectionMethods:
Enabled: true
Style/MethodCalledOnDoEndBlock:
diff --git a/.sync.yml b/.sync.yml
index 00a3fb4..367f666 100644
--- a/.sync.yml
+++ b/.sync.yml
@@ -21,11 +21,8 @@ Gemfile:
- gem: beaker-rspec
from_env: BEAKER_RSPEC_VERSION
':development':
- - gem: puppet-blacksmith
- version: '~> 3.4'
- gem: puppet-strings
Rakefile:
requires:
- - puppet_blacksmith/rake_tasks
- puppet-lint/tasks/puppet-lint
diff --git a/.travis.yml b/.travis.yml
index 6b1f5fb..81f77dd 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -5,9 +5,8 @@ language: ruby
cache: bundler
before_install:
- bundle -v
- - rm Gemfile.lock || true
+ - rm -f Gemfile.lock
- gem update --system
- - gem update bundler
- gem --version
- bundle -v
script:
@@ -16,20 +15,17 @@ bundler_args: --without system_tests
rvm:
- 2.4.1
env:
- - PUPPET_GEM_VERSION="~> 5.0" CHECK=spec
+ global:
+ - BEAKER_PUPPET_COLLECTION=puppet5 PUPPET_GEM_VERSION="~> 5.0"
matrix:
fast_finish: true
include:
-
- env: CHECK=rubocop
+ env: CHECK="syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop"
-
- env: CHECK="syntax lint"
+ env: CHECK=parallel_spec
-
- env: CHECK=metadata_lint
- -
- env: CHECK=spec
- -
- env: PUPPET_GEM_VERSION="~> 4.0" CHECK=spec
+ env: PUPPET_GEM_VERSION="~> 4.0" CHECK=parallel_spec
rvm: 2.1.9
branches:
only:
diff --git a/Gemfile b/Gemfile
index bb4eff8..86e8d7f 100644
--- a/Gemfile
+++ b/Gemfile
@@ -28,11 +28,12 @@ group :development do
gem "fast_gettext", require: false if Gem::Version.new(RUBY_VERSION.dup) >= Gem::Version.new('2.1.0')
gem "json_pure", '<= 2.0.1', require: false if Gem::Version.new(RUBY_VERSION.dup) < Gem::Version.new('2.0.0')
gem "json", '= 1.8.1', require: false if Gem::Version.new(RUBY_VERSION.dup) == Gem::Version.new('2.1.9')
+ gem "json", '<= 2.0.4', require: false if Gem::Version.new(RUBY_VERSION.dup) == Gem::Version.new('2.4.4')
gem "puppet-module-posix-default-r#{minor_version}", require: false, platforms: [:ruby]
gem "puppet-module-posix-dev-r#{minor_version}", require: false, platforms: [:ruby]
gem "puppet-module-win-default-r#{minor_version}", require: false, platforms: [:mswin, :mingw, :x64_mingw]
gem "puppet-module-win-dev-r#{minor_version}", require: false, platforms: [:mswin, :mingw, :x64_mingw]
- gem "puppet-blacksmith", '~> 3.4', require: false
+ gem "puppet-blacksmith", '~> 3.4', require: false, platforms: [:ruby]
gem "puppet-strings", require: false
end
group :system_tests do
@@ -50,73 +51,24 @@ puppet_type = gem_type(puppet_version)
facter_version = ENV['FACTER_GEM_VERSION']
hiera_version = ENV['HIERA_GEM_VERSION']
-def puppet_older_than?(version)
- puppet_version = ENV['PUPPET_GEM_VERSION']
- !puppet_version.nil? &&
- Gem::Version.correct?(puppet_version) &&
- Gem::Requirement.new("< #{version}").satisfied_by?(Gem::Version.new(puppet_version.dup))
-end
-
gems = {}
gems['puppet'] = location_for(puppet_version)
# If facter or hiera versions have been specified via the environment
-# variables, use those versions. If not, and if the puppet version is < 3.5.0,
-# use known good versions of both for puppet < 3.5.0.
-if facter_version
- gems['facter'] = location_for(facter_version)
-elsif puppet_type == :gem && puppet_older_than?('3.5.0')
- gems['facter'] = ['>= 1.6.11', '<= 1.7.5', require: false]
-end
-
-if hiera_version
- gems['hiera'] = location_for(ENV['HIERA_GEM_VERSION'])
-elsif puppet_type == :gem && puppet_older_than?('3.5.0')
- gems['hiera'] = ['>= 1.0.0', '<= 1.3.0', require: false]
-end
+# variables
-if Gem.win_platform? && (puppet_type != :gem || puppet_older_than?('3.5.0'))
- # For Puppet gems < 3.5.0 (tested as far back as 3.0.0) on Windows
- if puppet_type == :gem
- gems['ffi'] = ['1.9.0', require: false]
- gems['minitar'] = ['0.5.4', require: false]
- gems['win32-eventlog'] = ['0.5.3', '<= 0.6.5', require: false]
- gems['win32-process'] = ['0.6.5', '<= 0.7.5', require: false]
- gems['win32-security'] = ['~> 0.1.2', '<= 0.2.5', require: false]
- gems['win32-service'] = ['0.7.2', '<= 0.8.8', require: false]
- else
- gems['ffi'] = ['~> 1.9.0', require: false]
- gems['minitar'] = ['~> 0.5.4', require: false]
- gems['win32-eventlog'] = ['~> 0.5', '<= 0.6.5', require: false]
- gems['win32-process'] = ['~> 0.6', '<= 0.7.5', require: false]
- gems['win32-security'] = ['~> 0.1', '<= 0.2.5', require: false]
- gems['win32-service'] = ['~> 0.7', '<= 0.8.8', require: false]
- end
-
- gems['win32-dir'] = ['~> 0.3', '<= 0.4.9', require: false]
-
- if RUBY_VERSION.start_with?('1.')
- gems['win32console'] = ['1.3.2', require: false]
- # sys-admin was removed in Puppet 3.7.0 and doesn't compile under Ruby 2.x
- gems['sys-admin'] = ['1.5.6', require: false]
- end
+gems['facter'] = location_for(facter_version) if facter_version
+gems['hiera'] = location_for(hiera_version) if hiera_version
- # Puppet < 3.7.0 requires these.
- # Puppet >= 3.5.0 gem includes these as requirements.
- # The following versions are tested to work with 3.0.0 <= puppet < 3.7.0.
- gems['win32-api'] = ['1.4.8', require: false]
- gems['win32-taskscheduler'] = ['0.2.2', require: false]
- gems['windows-api'] = ['0.4.3', require: false]
- gems['windows-pr'] = ['1.2.3', require: false]
-elsif Gem.win_platform?
+if Gem.win_platform? && puppet_version =~ %r{^(file:///|git://)}
# If we're using a Puppet gem on Windows which handles its own win32-xxx gem
# dependencies (>= 3.5.0), set the maximum versions (see PUP-6445).
gems['win32-dir'] = ['<= 0.4.9', require: false]
gems['win32-eventlog'] = ['<= 0.6.5', require: false]
gems['win32-process'] = ['<= 0.7.5', require: false]
gems['win32-security'] = ['<= 0.2.5', require: false]
- gems['win32-service'] = ['<= 0.8.8', require: false]
+ gems['win32-service'] = ['0.8.8', require: false]
end
gems.each do |gem_name, gem_params|
diff --git a/Rakefile b/Rakefile
index a39cae2..ef5f698 100644
--- a/Rakefile
+++ b/Rakefile
@@ -1,4 +1,7 @@
require 'puppetlabs_spec_helper/rake_tasks'
require 'puppet-syntax/tasks/puppet-syntax'
-require 'puppet_blacksmith/rake_tasks'
+require 'puppet_blacksmith/rake_tasks' if Bundler.rubygems.find_name('puppet-blacksmith').any?
require 'puppet-lint/tasks/puppet-lint'
+
+PuppetLint.configuration.send('disable_relative')
+
diff --git a/appveyor.yml b/appveyor.yml
index 5fd5e89..4a5b227 100644
--- a/appveyor.yml
+++ b/appveyor.yml
@@ -1,3 +1,4 @@
+---
version: 1.1.x.{build}
skip_commits:
message: /^\(?doc\)?.*/
@@ -12,29 +13,23 @@ environment:
matrix:
-
RUBY_VERSION: 24-x64
- CHECK: syntax lint
- -
- RUBY_VERSION: 24-x64
- CHECK: metadata_lint
- -
- RUBY_VERSION: 24-x64
- CHECK: rubocop
+ CHECK: syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop
-
PUPPET_GEM_VERSION: ~> 4.0
RUBY_VERSION: 21
- CHECK: spec
+ CHECK: parallel_spec
-
PUPPET_GEM_VERSION: ~> 4.0
RUBY_VERSION: 21-x64
- CHECK: spec
+ CHECK: parallel_spec
-
PUPPET_GEM_VERSION: ~> 5.0
RUBY_VERSION: 24
- CHECK: spec
+ CHECK: parallel_spec
-
PUPPET_GEM_VERSION: ~> 5.0
RUBY_VERSION: 24-x64
- CHECK: spec
+ CHECK: parallel_spec
matrix:
fast_finish: true
install:
diff --git a/metadata.json b/metadata.json
index 409ae01..3c4850f 100644
--- a/metadata.json
+++ b/metadata.json
@@ -78,7 +78,7 @@
"version_requirement": ">= 4.7.0 < 6.0.0"
}
],
- "pdk-version": "1.4.1",
- "template-url": "file:///opt/puppetlabs/pdk/share/cache/pdk-templates.git",
- "template-ref": "1.4.1-0-g52adbbb"
+ "pdk-version": "1.5.0",
+ "template-url": "https://github.com/puppetlabs/pdk-templates",
+ "template-ref": "heads/master-0-g34e3266"
}
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index efd225b..e117192 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -1,3 +1,4 @@
+
require 'puppetlabs_spec_helper/module_spec_helper'
require 'rspec-puppet-facts'
@@ -27,4 +28,9 @@ end
RSpec.configure do |c|
c.default_facts = default_facts
+ c.before :each do
+ # set to strictest setting for testing
+ # by default Puppet runs at warning level
+ Puppet.settings[:strict] = :warning
+ end
end
generated by cgit v1.2.3 (git 2.39.1) at 2025-09-23 20:09:26 +0000