Skip to content

Commit 26c18c7

Browse files
author
Helen Campbell
committed
pdksync_heads/master-0-gfe44b98
1 parent c8593fb commit 26c18c7

9 files changed

+65
-67
lines changed

.gitignore

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,11 @@
1313
/Gemfile.lock
1414
/junit/
1515
/log/
16-
/log/
1716
/pkg/
1817
/spec/fixtures/manifests/
1918
/spec/fixtures/modules/
2019
/tmp/
2120
/vendor/
2221
/convert_report.txt
22+
/update_report.txt
2323
.DS_Store

.pdkignore

+23
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
.*.sw[op]
2+
.metadata
3+
.yardoc
4+
.yardwarns
5+
*.iml
6+
/.bundle/
7+
/.idea/
8+
/.vagrant/
9+
/coverage/
10+
/bin/
11+
/doc/
12+
/Gemfile.local
13+
/Gemfile.lock
14+
/junit/
15+
/log/
16+
/pkg/
17+
/spec/fixtures/manifests/
18+
/spec/fixtures/modules/
19+
/tmp/
20+
/vendor/
21+
/convert_report.txt
22+
/update_report.txt
23+
.DS_Store

.rubocop.yml

+7-2
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,14 @@ AllCops:
1010
Exclude:
1111
- bin/*
1212
- ".vendor/**/*"
13-
- Gemfile
14-
- Rakefile
13+
- "**/Gemfile"
14+
- "**/Rakefile"
1515
- pkg/**/*
1616
- spec/fixtures/**/*
1717
- vendor/**/*
18+
- "**/Puppetfile"
19+
- "**/Vagrantfile"
20+
- "**/Guardfile"
1821
Metrics/LineLength:
1922
Description: People have wide screens, use them.
2023
Max: 200
@@ -66,6 +69,8 @@ Style/SymbolArray:
6669
Description: Using percent style obscures symbolic intent of array's contents.
6770
EnforcedStyle: brackets
6871
inherit_from: ".rubocop_todo.yml"
72+
RSpec/MessageSpies:
73+
EnforcedStyle: receive
6974
Style/CollectionMethods:
7075
Enabled: true
7176
Style/MethodCalledOnDoEndBlock:

.travis.yml

+7-6
Original file line numberDiff line numberDiff line change
@@ -16,22 +16,23 @@ bundler_args: --without system_tests
1616
rvm:
1717
- 2.4.1
1818
env:
19-
- PUPPET_GEM_VERSION="~> 5.0" CHECK=spec
19+
global:
20+
- BEAKER_PUPPET_COLLECTION=puppet5 PUPPET_GEM_VERSION="~> 5.0"
2021
matrix:
2122
fast_finish: true
2223
include:
2324
-
2425
bundler_args:
2526
dist: trusty
26-
env: PUPPET_INSTALL_TYPE=agent BEAKER_debug=true BEAKER_set=docker/centos-7
27+
env: PUPPET_INSTALL_TYPE=agent BEAKER_debug=true BEAKER_PUPPET_COLLECTION=puppet5 BEAKER_set=docker/centos-7
2728
rvm: 2.4.1
2829
script: bundle exec rake beaker
2930
services: docker
3031
sudo: required
3132
-
3233
bundler_args:
3334
dist: trusty
34-
env: PUPPET_INSTALL_TYPE=agent BEAKER_debug=true BEAKER_set=docker/ubuntu-14.04
35+
env: PUPPET_INSTALL_TYPE=agent BEAKER_debug=true BEAKER_PUPPET_COLLECTION=puppet5 BEAKER_set=docker/ubuntu-14.04
3536
rvm: 2.4.1
3637
script: bundle exec rake beaker
3738
services: docker
@@ -43,11 +44,11 @@ matrix:
4344
-
4445
env: CHECK=metadata_lint
4546
-
46-
env: CHECK=release_checks
47+
env: CHECK="check:symlinks check:git_ignore check:dot_underscore check:test_file"
4748
-
48-
env: CHECK=spec
49+
env: CHECK=parallel_spec
4950
-
50-
env: PUPPET_GEM_VERSION="~> 4.0" CHECK=spec
51+
env: PUPPET_GEM_VERSION="~> 4.0" CHECK=parallel_spec
5152
rvm: 2.1.9
5253
branches:
5354
only:

.yardopts

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
--markup markdown

Gemfile

+7-55
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,12 @@ group :development do
2828
gem "fast_gettext", require: false if Gem::Version.new(RUBY_VERSION.dup) >= Gem::Version.new('2.1.0')
2929
gem "json_pure", '<= 2.0.1', require: false if Gem::Version.new(RUBY_VERSION.dup) < Gem::Version.new('2.0.0')
3030
gem "json", '= 1.8.1', require: false if Gem::Version.new(RUBY_VERSION.dup) == Gem::Version.new('2.1.9')
31+
gem "json", '<= 2.0.4', require: false if Gem::Version.new(RUBY_VERSION.dup) == Gem::Version.new('2.4.4')
3132
gem "puppet-module-posix-default-r#{minor_version}", require: false, platforms: [:ruby]
3233
gem "puppet-module-posix-dev-r#{minor_version}", require: false, platforms: [:ruby]
3334
gem "puppet-module-win-default-r#{minor_version}", require: false, platforms: [:mswin, :mingw, :x64_mingw]
3435
gem "puppet-module-win-dev-r#{minor_version}", require: false, platforms: [:mswin, :mingw, :x64_mingw]
35-
gem "puppet-blacksmith", '~> 3.4', require: false
36+
gem "puppet-blacksmith", '~> 3.4', require: false, platforms: [:ruby]
3637
gem "puppet-lint-i18n", require: false
3738
end
3839
group :system_tests do
@@ -50,73 +51,24 @@ puppet_type = gem_type(puppet_version)
5051
facter_version = ENV['FACTER_GEM_VERSION']
5152
hiera_version = ENV['HIERA_GEM_VERSION']
5253

53-
def puppet_older_than?(version)
54-
puppet_version = ENV['PUPPET_GEM_VERSION']
55-
!puppet_version.nil? &&
56-
Gem::Version.correct?(puppet_version) &&
57-
Gem::Requirement.new("< #{version}").satisfied_by?(Gem::Version.new(puppet_version.dup))
58-
end
59-
6054
gems = {}
6155

6256
gems['puppet'] = location_for(puppet_version)
6357

6458
# If facter or hiera versions have been specified via the environment
65-
# variables, use those versions. If not, and if the puppet version is < 3.5.0,
66-
# use known good versions of both for puppet < 3.5.0.
67-
if facter_version
68-
gems['facter'] = location_for(facter_version)
69-
elsif puppet_type == :gem && puppet_older_than?('3.5.0')
70-
gems['facter'] = ['>= 1.6.11', '<= 1.7.5', require: false]
71-
end
72-
73-
if hiera_version
74-
gems['hiera'] = location_for(ENV['HIERA_GEM_VERSION'])
75-
elsif puppet_type == :gem && puppet_older_than?('3.5.0')
76-
gems['hiera'] = ['>= 1.0.0', '<= 1.3.0', require: false]
77-
end
59+
# variables
7860

79-
if Gem.win_platform? && (puppet_type != :gem || puppet_older_than?('3.5.0'))
80-
# For Puppet gems < 3.5.0 (tested as far back as 3.0.0) on Windows
81-
if puppet_type == :gem
82-
gems['ffi'] = ['1.9.0', require: false]
83-
gems['minitar'] = ['0.5.4', require: false]
84-
gems['win32-eventlog'] = ['0.5.3', '<= 0.6.5', require: false]
85-
gems['win32-process'] = ['0.6.5', '<= 0.7.5', require: false]
86-
gems['win32-security'] = ['~> 0.1.2', '<= 0.2.5', require: false]
87-
gems['win32-service'] = ['0.7.2', '<= 0.8.8', require: false]
88-
else
89-
gems['ffi'] = ['~> 1.9.0', require: false]
90-
gems['minitar'] = ['~> 0.5.4', require: false]
91-
gems['win32-eventlog'] = ['~> 0.5', '<= 0.6.5', require: false]
92-
gems['win32-process'] = ['~> 0.6', '<= 0.7.5', require: false]
93-
gems['win32-security'] = ['~> 0.1', '<= 0.2.5', require: false]
94-
gems['win32-service'] = ['~> 0.7', '<= 0.8.8', require: false]
95-
end
96-
97-
gems['win32-dir'] = ['~> 0.3', '<= 0.4.9', require: false]
98-
99-
if RUBY_VERSION.start_with?('1.')
100-
gems['win32console'] = ['1.3.2', require: false]
101-
# sys-admin was removed in Puppet 3.7.0 and doesn't compile under Ruby 2.x
102-
gems['sys-admin'] = ['1.5.6', require: false]
103-
end
61+
gems['facter'] = location_for(facter_version) if facter_version
62+
gems['hiera'] = location_for(hiera_version) if hiera_version
10463

105-
# Puppet < 3.7.0 requires these.
106-
# Puppet >= 3.5.0 gem includes these as requirements.
107-
# The following versions are tested to work with 3.0.0 <= puppet < 3.7.0.
108-
gems['win32-api'] = ['1.4.8', require: false]
109-
gems['win32-taskscheduler'] = ['0.2.2', require: false]
110-
gems['windows-api'] = ['0.4.3', require: false]
111-
gems['windows-pr'] = ['1.2.3', require: false]
112-
elsif Gem.win_platform?
64+
if Gem.win_platform? && puppet_version =~ %r{^(file:///|git://)}
11365
# If we're using a Puppet gem on Windows which handles its own win32-xxx gem
11466
# dependencies (>= 3.5.0), set the maximum versions (see PUP-6445).
11567
gems['win32-dir'] = ['<= 0.4.9', require: false]
11668
gems['win32-eventlog'] = ['<= 0.6.5', require: false]
11769
gems['win32-process'] = ['<= 0.7.5', require: false]
11870
gems['win32-security'] = ['<= 0.2.5', require: false]
119-
gems['win32-service'] = ['<= 0.8.8', require: false]
71+
gems['win32-service'] = ['0.8.8', require: false]
12072
end
12173

12274
gems.each do |gem_name, gem_params|

Rakefile

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
require 'puppetlabs_spec_helper/rake_tasks'
22
require 'puppet-syntax/tasks/puppet-syntax'
3-
require 'puppet_blacksmith/rake_tasks'
3+
require 'puppet_blacksmith/rake_tasks' if Bundler.rubygems.find_name('puppet-blacksmith').any?
44
require 'puppet_pot_generator/rake_tasks'
5+
6+
PuppetLint.configuration.send('relative')

metadata.json

+3-2
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@
8484
}
8585
],
8686
"description": "MySQL module",
87-
"template-url": "file:///opt/puppetlabs/pdk/share/cache/pdk-templates.git",
88-
"template-ref": "1.3.2-0-g07678c8"
87+
"template-url": "https://github.com/puppetlabs/pdk-templates",
88+
"template-ref": "heads/master-0-gfe44b98",
89+
"pdk-version": "1.5.0"
8990
}

spec/spec_helper.rb

+13
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
1+
12
require 'puppetlabs_spec_helper/module_spec_helper'
23
require 'rspec-puppet-facts'
4+
5+
begin
6+
require 'spec_helper_local' if File.file?(File.join(File.dirname(__FILE__), 'spec_helper_local.rb'))
7+
rescue LoadError => loaderror
8+
warn "Could not require spec_helper_local: #{loaderror.message}"
9+
end
10+
311
include RspecPuppetFacts
412

513
default_facts = {
@@ -20,5 +28,10 @@
2028

2129
RSpec.configure do |c|
2230
c.default_facts = default_facts
31+
c.before :each do
32+
# set to strictest setting for testing
33+
# by default Puppet runs at warning level
34+
Puppet.settings[:strict] = :warning
35+
end
2336
end
2437
require 'spec_helper_local'

0 commit comments

Comments
 (0)