forked from markt-de/puppet-galera
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
convert to PDK, make test suite not failing
work on markt-de#108 and markt-de#114 Make the testsuite run again and not fail. For this a lot of test code was disabled. Since the failing tests were ignored for such a long time, there is a lot to do here. Disabled test code is marked with `# FIXME`. I removed a bunch of supported OSes from metadata.json. One reason being missing facst in facterdb, another reason is that those OS-releases might simply not be be really supported. Thats at least the impression I get from the tests. Also i don't think one needs to list RedHat and CentOS and Scintific. The latter two should be included in simply Redhat.
- Loading branch information
Showing
31 changed files
with
1,002 additions
and
423 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
*.rb eol=lf | ||
*.erb eol=lf | ||
*.pp eol=lf | ||
*.sh eol=lf | ||
*.epp eol=lf |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,27 @@ | ||
vendor | ||
Gemfile.lock | ||
spec/fixtures | ||
log | ||
.vagrant | ||
|
||
.idea/ | ||
.git/ | ||
.*.sw[op] | ||
.metadata | ||
.yardoc | ||
.yardwarns | ||
*.iml | ||
/.bundle/ | ||
/.idea/ | ||
/.vagrant/ | ||
/coverage/ | ||
/bin/ | ||
/doc/ | ||
/Gemfile.local | ||
/Gemfile.lock | ||
/junit/ | ||
/log/ | ||
/pkg/ | ||
/spec/fixtures/manifests/ | ||
/spec/fixtures/modules/ | ||
/tmp/ | ||
/vendor/ | ||
/convert_report.txt | ||
/update_report.txt | ||
.DS_Store | ||
.project | ||
.envrc | ||
/inventory.yaml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
--- | ||
stages: | ||
- syntax | ||
- unit | ||
|
||
cache: | ||
paths: | ||
- vendor/bundle | ||
|
||
before_script: | ||
- bundle -v | ||
- rm Gemfile.lock || true | ||
- gem update --system $RUBYGEMS_VERSION | ||
- gem --version | ||
- bundle -v | ||
- bundle install --without system_tests --path vendor/bundle --jobs $(nproc) | ||
|
||
syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop-Ruby 2.5.3-Puppet ~> 6: | ||
stage: syntax | ||
image: ruby:2.5.3 | ||
script: | ||
- bundle exec rake syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop | ||
variables: | ||
PUPPET_GEM_VERSION: '~> 6' | ||
|
||
parallel_spec-Ruby 2.5.3-Puppet ~> 6: | ||
stage: unit | ||
image: ruby:2.5.3 | ||
script: | ||
- bundle exec rake parallel_spec | ||
variables: | ||
PUPPET_GEM_VERSION: '~> 6' | ||
|
||
parallel_spec-Ruby 2.4.5-Puppet ~> 5: | ||
stage: unit | ||
image: ruby:2.4.5 | ||
script: | ||
- bundle exec rake parallel_spec | ||
variables: | ||
PUPPET_GEM_VERSION: '~> 5' | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
.git/ | ||
.*.sw[op] | ||
.metadata | ||
.yardoc | ||
.yardwarns | ||
*.iml | ||
/.bundle/ | ||
/.idea/ | ||
/.vagrant/ | ||
/coverage/ | ||
/bin/ | ||
/doc/ | ||
/Gemfile.local | ||
/Gemfile.lock | ||
/junit/ | ||
/log/ | ||
/pkg/ | ||
/spec/fixtures/manifests/ | ||
/spec/fixtures/modules/ | ||
/tmp/ | ||
/vendor/ | ||
/convert_report.txt | ||
/update_report.txt | ||
.DS_Store | ||
.project | ||
.envrc | ||
/inventory.yaml | ||
/appveyor.yml | ||
/.fixtures.yml | ||
/Gemfile | ||
/.gitattributes | ||
/.gitignore | ||
/.gitlab-ci.yml | ||
/.pdkignore | ||
/Rakefile | ||
/rakelib/ | ||
/.rspec | ||
/.rubocop.yml | ||
/.travis.yml | ||
/.yardopts | ||
/spec/ | ||
/.vscode/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
--relative |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
--color | ||
--format documentation | ||
--fail-fast |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,133 @@ | ||
--- | ||
require: | ||
- rubocop-rspec | ||
- rubocop-i18n | ||
AllCops: | ||
DisplayCopNames: true | ||
TargetRubyVersion: '2.1' | ||
Include: | ||
- "./**/*.rb" | ||
Exclude: | ||
- bin/* | ||
- ".vendor/**/*" | ||
- "**/Gemfile" | ||
- "**/Rakefile" | ||
- pkg/**/* | ||
- spec/fixtures/**/* | ||
- vendor/**/* | ||
- "**/Puppetfile" | ||
- "**/Vagrantfile" | ||
- "**/Guardfile" | ||
Metrics/LineLength: | ||
Description: People have wide screens, use them. | ||
Max: 200 | ||
GetText: | ||
Enabled: false | ||
GetText/DecorateString: | ||
Description: We don't want to decorate test output. | ||
Exclude: | ||
- spec/**/* | ||
Enabled: false | ||
RSpec/BeforeAfterAll: | ||
Description: Beware of using after(:all) as it may cause state to leak between tests. | ||
A necessary evil in acceptance testing. | ||
Exclude: | ||
- spec/acceptance/**/*.rb | ||
RSpec/HookArgument: | ||
Description: Prefer explicit :each argument, matching existing module's style | ||
EnforcedStyle: each | ||
Style/BlockDelimiters: | ||
Description: Prefer braces for chaining. Mostly an aesthetical choice. Better to | ||
be consistent then. | ||
EnforcedStyle: braces_for_chaining | ||
Style/ClassAndModuleChildren: | ||
Description: Compact style reduces the required amount of indentation. | ||
EnforcedStyle: compact | ||
Style/EmptyElse: | ||
Description: Enforce against empty else clauses, but allow `nil` for clarity. | ||
EnforcedStyle: empty | ||
Style/FormatString: | ||
Description: Following the main puppet project's style, prefer the % format format. | ||
EnforcedStyle: percent | ||
Style/FormatStringToken: | ||
Description: Following the main puppet project's style, prefer the simpler template | ||
tokens over annotated ones. | ||
EnforcedStyle: template | ||
Style/Lambda: | ||
Description: Prefer the keyword for easier discoverability. | ||
EnforcedStyle: literal | ||
Style/RegexpLiteral: | ||
Description: Community preference. See https://github.com/voxpupuli/modulesync_config/issues/168 | ||
EnforcedStyle: percent_r | ||
Style/TernaryParentheses: | ||
Description: Checks for use of parentheses around ternary conditions. Enforce parentheses | ||
on complex expressions for better readability, but seriously consider breaking | ||
it up. | ||
EnforcedStyle: require_parentheses_when_complex | ||
Style/TrailingCommaInArguments: | ||
Description: Prefer always trailing comma on multiline argument lists. This makes | ||
diffs, and re-ordering nicer. | ||
EnforcedStyleForMultiline: comma | ||
Style/TrailingCommaInLiteral: | ||
Description: Prefer always trailing comma on multiline literals. This makes diffs, | ||
and re-ordering nicer. | ||
EnforcedStyleForMultiline: comma | ||
Style/SymbolArray: | ||
Description: Using percent style obscures symbolic intent of array's contents. | ||
EnforcedStyle: brackets | ||
RSpec/MessageSpies: | ||
EnforcedStyle: receive | ||
Style/Documentation: | ||
Exclude: | ||
- lib/puppet/parser/functions/**/* | ||
- spec/**/* | ||
Style/WordArray: | ||
EnforcedStyle: brackets | ||
Style/CollectionMethods: | ||
Enabled: true | ||
Style/MethodCalledOnDoEndBlock: | ||
Enabled: true | ||
Style/StringMethods: | ||
Enabled: true | ||
GetText/DecorateFunctionMessage: | ||
Enabled: false | ||
GetText/DecorateStringFormattingUsingInterpolation: | ||
Enabled: false | ||
GetText/DecorateStringFormattingUsingPercent: | ||
Enabled: false | ||
Layout/EndOfLine: | ||
Enabled: false | ||
Layout/IndentHeredoc: | ||
Enabled: false | ||
Metrics/AbcSize: | ||
Enabled: false | ||
Metrics/BlockLength: | ||
Enabled: false | ||
Metrics/ClassLength: | ||
Enabled: false | ||
Metrics/CyclomaticComplexity: | ||
Enabled: false | ||
Metrics/MethodLength: | ||
Enabled: false | ||
Metrics/ModuleLength: | ||
Enabled: false | ||
Metrics/ParameterLists: | ||
Enabled: false | ||
Metrics/PerceivedComplexity: | ||
Enabled: false | ||
RSpec/DescribeClass: | ||
Enabled: false | ||
RSpec/ExampleLength: | ||
Enabled: false | ||
RSpec/MessageExpectation: | ||
Enabled: false | ||
RSpec/MultipleExpectations: | ||
Enabled: false | ||
RSpec/NestedGroups: | ||
Enabled: false | ||
Style/AsciiComments: | ||
Enabled: false | ||
Style/IfUnlessModifier: | ||
Enabled: false | ||
Style/SymbolProc: | ||
Enabled: false |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
|
||
spec/spec_helper.rb: | ||
hiera_config: spec/fixtures/hiera.yaml | ||
coverage_report: true | ||
|
||
spec/default_facts.yml: | ||
extra_facts: | ||
root_home: /root | ||
fqdn: localhost |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,41 @@ | ||
--- | ||
dist: xenial | ||
language: ruby | ||
cache: bundler | ||
before_install: | ||
- bundle -v | ||
- rm -f Gemfile.lock | ||
- gem update --system $RUBYGEMS_VERSION | ||
- gem --version | ||
- bundle -v | ||
script: | ||
- 'bundle exec rake $CHECK' | ||
bundler_args: --without system_tests | ||
rvm: | ||
- 2.2.5 | ||
- 2.5.3 | ||
stages: | ||
- static | ||
- spec | ||
- acceptance | ||
- | ||
if: tag =~ ^v\d | ||
name: deploy | ||
matrix: | ||
fast_finish: true | ||
include: | ||
- | ||
env: CHECK="check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop syntax lint metadata_lint" | ||
stage: static | ||
- | ||
env: PUPPET_GEM_VERSION="~> 5.0" CHECK=parallel_spec | ||
rvm: 2.4.5 | ||
stage: spec | ||
- | ||
env: PUPPET_GEM_VERSION="~> 6.0" CHECK=parallel_spec | ||
rvm: 2.5.3 | ||
stage: spec | ||
- | ||
env: DEPLOY_TO_FORGE=yes | ||
stage: deploy | ||
notifications: | ||
email: | ||
- woppin@gmail.com | ||
env: | ||
- PUPPET_VERSION="~> 2.7.0" STRICT_VARIABLES=no | ||
- PUPPET_VERSION="~> 3.2.0" STRICT_VARIABLES=no | ||
- PUPPET_VERSION="~> 3.4.0" STRICT_VARIABLES=no | ||
email: false |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
{ | ||
"recommendations": [ | ||
"jpogran.puppet-vscode", | ||
"rebornix.Ruby" | ||
] | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
--markup markdown |
Oops, something went wrong.