From efe03e6af4e63b1c6a98c22cfdbfee040b99c44b Mon Sep 17 00:00:00 2001 From: Tim Meusel Date: Mon, 11 Jul 2016 16:26:37 +0200 Subject: modulesync 0.9.1 --- .github/CONTRIBUTING.md | 95 ++++++++ .github/ISSUE_TEMPLATE.md | 23 ++ .github/PULL_REQUEST_TEMPLATE.md | 5 + .msync.yml | 1 + .rubocop.yml | 480 ++++++++++++++++++++++++++++++++++++--- .travis.yml | 4 +- Gemfile | 2 - spec/spec_helper.rb | 13 ++ 8 files changed, 593 insertions(+), 30 deletions(-) create mode 100644 .github/CONTRIBUTING.md create mode 100644 .github/ISSUE_TEMPLATE.md create mode 100644 .github/PULL_REQUEST_TEMPLATE.md create mode 100644 .msync.yml diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md new file mode 100644 index 0000000..5b09ea6 --- /dev/null +++ b/.github/CONTRIBUTING.md @@ -0,0 +1,95 @@ +This module has grown over time based on a range of contributions from +people using it. If you follow these contributing guidelines your patch +will likely make it into a release a little quicker. + + +## Contributing + +Please note that this project is released with a Contributor Code of Conduct. By participating in this project you agree to abide by its terms. [Contributor Code of Conduct](https://voxpupuli.org/coc/). + +1. Fork the repo. + +1. Create a separate branch for your change. + +1. Run the tests. We only take pull requests with passing tests, and + documentation. + +1. Add a test for your change. Only refactoring and documentation + changes require no new tests. If you are adding functionality + or fixing a bug, please add a test. + +1. Squash your commits down into logical components. Make sure to rebase + against the current master. + +1. Push the branch to your fork and submit a pull request. + +Please be prepared to repeat some of these steps as our contributors review +your code. + +## Dependencies + +The testing and development tools have a bunch of dependencies, +all managed by [bundler](http://bundler.io/) according to the +[Puppet support matrix](http://docs.puppetlabs.com/guides/platforms.html#ruby-versions). + +By default the tests use a baseline version of Puppet. + +If you have Ruby 2.x or want a specific version of Puppet, +you must set an environment variable such as: + + export PUPPET_VERSION="~> 4.2.0" + +Install the dependencies like so... + + bundle install + +## Syntax and style + +The test suite will run [Puppet Lint](http://puppet-lint.com/) and +[Puppet Syntax](https://github.com/gds-operations/puppet-syntax) to +check various syntax and style things. You can run these locally with: + + bundle exec rake lint + bundle exec rake validate + +## Running the unit tests + +The unit test suite covers most of the code, as mentioned above please +add tests if you're adding new functionality. If you've not used +[rspec-puppet](http://rspec-puppet.com/) before then feel free to ask +about how best to test your new feature. + +To run your all the unit tests + + bundle exec rake spec SPEC_OPTS='--format documentation' + +To run a specific spec test set the `SPEC` variable: + + bundle exec rake spec SPEC=spec/foo_spec.rb + +To run the linter, the syntax checker and the unit tests: + + bundle exec rake test + + +## Integration tests + +The unit tests just check the code runs, not that it does exactly what +we want on a real machine. For that we're using +[beaker](https://github.com/puppetlabs/beaker). + +This fires up a new virtual machine (using vagrant) and runs a series of +simple tests against it after applying the module. You can run this +with: + + bundle exec rake acceptance + +This will run the tests on an Ubuntu 12.04 virtual machine. You can also +run the integration tests against Centos 6.5 with. + + BEAKER_set=centos-64-x64 bundle exec rake acceptances + +If you don't want to have to recreate the virtual machine every time you +can use `BEAKER_DESTROY=no` and `BEAKER_PROVISION=no`. On the first run you will +at least need `BEAKER_PROVISION` set to yes (the default). The Vagrantfile +for the created virtual machines will be in `.vagrant/beaker_vagrant_fies`. diff --git a/.github/ISSUE_TEMPLATE.md b/.github/ISSUE_TEMPLATE.md new file mode 100644 index 0000000..9ac4a2b --- /dev/null +++ b/.github/ISSUE_TEMPLATE.md @@ -0,0 +1,23 @@ + + +### Affected Puppet, Ruby, OS and module versions/distributions + +- Puppet: +- Ruby: +- Distribution: +- Module version: + +### How to reproduce (e.g Puppet code you use) + +### What are you seeing + +### What behaviour did you expect instead + +### Output log + +### Any additional information you'd like to impart diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md new file mode 100644 index 0000000..79272bf --- /dev/null +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -0,0 +1,5 @@ + diff --git a/.msync.yml b/.msync.yml new file mode 100644 index 0000000..efd68b6 --- /dev/null +++ b/.msync.yml @@ -0,0 +1 @@ +modulesync_config_version: '0.9.1' diff --git a/.rubocop.yml b/.rubocop.yml index 364b022..c834cd2 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -8,22 +8,53 @@ AllCops: - .vendor/**/* - pkg/**/* - spec/fixtures/**/* +Lint/ConditionPosition: + Enabled: true -# Configuration parameters: AllowURI, URISchemes. -Metrics/LineLength: - Max: 328 +Lint/ElseLayout: + Enabled: true -# 'Complexity' is very relative -Metrics/PerceivedComplexity: - Enabled: false +Lint/UnreachableCode: + Enabled: true -# 'Complexity' is very relative -Metrics/CyclomaticComplexity: - Enabled: false +Lint/UselessComparison: + Enabled: true -# 'Complexity' is very relative -Metrics/AbcSize: - Enabled: false +Lint/EnsureReturn: + Enabled: true + +Lint/HandleExceptions: + Enabled: true + +Lint/LiteralInCondition: + Enabled: true + +Lint/ShadowingOuterLocalVariable: + Enabled: true + +Lint/LiteralInInterpolation: + Enabled: true + +Style/HashSyntax: + Enabled: true + +Style/RedundantReturn: + Enabled: true + +Lint/AmbiguousOperator: + Enabled: true + +Lint/AssignmentInCondition: + Enabled: true + +Style/SpaceBeforeComment: + Enabled: true + +Style/AndOr: + Enabled: true + +Style/RedundantSelf: + Enabled: true # Method length is not necessarily an indicator of code quality Metrics/MethodLength: @@ -33,42 +64,437 @@ Metrics/MethodLength: Metrics/ModuleLength: Enabled: false +Style/WhileUntilModifier: + Enabled: true + +Lint/AmbiguousRegexpLiteral: + Enabled: true + +Lint/Eval: + Enabled: true + +Lint/BlockAlignment: + Enabled: true + +Lint/DefEndAlignment: + Enabled: true + +Lint/EndAlignment: + Enabled: true + +Lint/DeprecatedClassMethods: + Enabled: true + +Lint/Loop: + Enabled: true + +Lint/ParenthesesAsGroupedExpression: + Enabled: true + +Lint/RescueException: + Enabled: true + +Lint/StringConversionInInterpolation: + Enabled: true + +Lint/UnusedBlockArgument: + Enabled: true + +Lint/UnusedMethodArgument: + Enabled: true + +Lint/UselessAccessModifier: + Enabled: true + +Lint/UselessAssignment: + Enabled: true + +Lint/Void: + Enabled: true + +Style/AccessModifierIndentation: + Enabled: true + +Style/AccessorMethodName: + Enabled: true + +Style/Alias: + Enabled: true + +Style/AlignArray: + Enabled: true + +Style/AlignHash: + Enabled: true + +Style/AlignParameters: + Enabled: true + +Metrics/BlockNesting: + Enabled: true + +Style/AsciiComments: + Enabled: true + +Style/Attr: + Enabled: true + +Style/BracesAroundHashParameters: + Enabled: true + +Style/CaseEquality: + Enabled: true + +Style/CaseIndentation: + Enabled: true + +Style/CharacterLiteral: + Enabled: true + +Style/ClassAndModuleCamelCase: + Enabled: true + +Style/ClassAndModuleChildren: + Enabled: false + +Style/ClassCheck: + Enabled: true + # Class length is not necessarily an indicator of code quality Metrics/ClassLength: Enabled: false -# dealbreaker: -Style/TrailingCommaInArguments: +Style/ClassMethods: + Enabled: true + +Style/ClassVars: + Enabled: true + +Style/WhenThen: + Enabled: true + +Style/WordArray: + Enabled: true + +Style/UnneededPercentQ: + Enabled: true + +Style/Tab: + Enabled: true + +Style/SpaceBeforeSemicolon: + Enabled: true + +Style/TrailingBlankLines: + Enabled: true + +Style/SpaceInsideBlockBraces: + Enabled: true + +Style/SpaceInsideBrackets: + Enabled: true + +Style/SpaceInsideHashLiteralBraces: + Enabled: true + +Style/SpaceInsideParens: + Enabled: true + +Style/LeadingCommentSpace: + Enabled: true + +Style/SpaceBeforeFirstArg: + Enabled: true + +Style/SpaceAfterColon: + Enabled: true + +Style/SpaceAfterComma: + Enabled: true + +Style/SpaceAfterMethodName: + Enabled: true + +Style/SpaceAfterNot: + Enabled: true + +Style/SpaceAfterSemicolon: + Enabled: true + +Style/SpaceAroundEqualsInParameterDefault: + Enabled: true + +Style/SpaceAroundOperators: + Enabled: true + +Style/SpaceBeforeBlockBraces: + Enabled: true + +Style/SpaceBeforeComma: + Enabled: true + +Style/CollectionMethods: + Enabled: true + +Style/CommentIndentation: + Enabled: true + +Style/ColonMethodCall: + Enabled: true + +Style/CommentAnnotation: + Enabled: true + +# 'Complexity' is very relative +Metrics/CyclomaticComplexity: Enabled: false -Style/TrailingCommaInLiteral: + +Style/ConstantName: + Enabled: true + +Style/Documentation: Enabled: false -Style/ClosingParenthesisIndentation: + +Style/DefWithParentheses: + Enabled: true + +Style/PreferredHashMethods: + Enabled: true + +Style/DotPosition: + EnforcedStyle: trailing + +Style/DoubleNegation: + Enabled: true + +Style/EachWithObject: + Enabled: true + +Style/EmptyLineBetweenDefs: + Enabled: true + +Style/IndentArray: + Enabled: true + +Style/IndentHash: + Enabled: true + +Style/IndentationConsistency: + Enabled: true + +Style/IndentationWidth: + Enabled: true + +Style/EmptyLines: + Enabled: true + +Style/EmptyLinesAroundAccessModifier: + Enabled: true + +Style/EmptyLiteral: + Enabled: true + +# Configuration parameters: AllowURI, URISchemes. +Metrics/LineLength: Enabled: false -Lint/AmbiguousRegexpLiteral: +Style/MethodCallParentheses: + Enabled: true + +Style/MethodDefParentheses: + Enabled: true + +Style/LineEndConcatenation: + Enabled: true + +Style/TrailingWhitespace: + Enabled: true + +Style/StringLiterals: Enabled: true + +Style/TrailingCommaInArguments: + Enabled: true + +Style/TrailingCommaInLiteral: + Enabled: true + +Style/GlobalVars: + Enabled: true + +Style/GuardClause: + Enabled: true + +Style/IfUnlessModifier: + Enabled: true + +Style/MultilineIfThen: + Enabled: true + +Style/NegatedIf: + Enabled: true + +Style/NegatedWhile: + Enabled: true + +Style/Next: + Enabled: true + +Style/SingleLineBlockParams: + Enabled: true + +Style/SingleLineMethods: + Enabled: true + +Style/SpecialGlobalVars: + Enabled: true + +Style/TrivialAccessors: + Enabled: true + +Style/UnlessElse: + Enabled: true + +Style/VariableInterpolation: + Enabled: true + +Style/VariableName: + Enabled: true + +Style/WhileUntilDo: + Enabled: true + +Style/EvenOdd: + Enabled: true + +Style/FileName: + Enabled: true + +Style/For: + Enabled: true + +Style/Lambda: + Enabled: true + +Style/MethodName: + Enabled: true + +Style/MultilineTernaryOperator: + Enabled: true + +Style/NestedTernaryOperator: + Enabled: true + +Style/NilComparison: + Enabled: true + +Style/FormatString: + Enabled: true + +Style/MultilineBlockChain: + Enabled: true + +Style/Semicolon: + Enabled: true + +Style/SignalException: + Enabled: true + +Style/NonNilCheck: + Enabled: true + +Style/Not: + Enabled: true + +Style/NumericLiterals: + Enabled: true + +Style/OneLineConditional: + Enabled: true + +Style/OpMethod: + Enabled: true + +Style/ParenthesesAroundCondition: + Enabled: true + +Style/PercentLiteralDelimiters: + Enabled: true + +Style/PerlBackrefs: + Enabled: true + +Style/PredicateName: + Enabled: true + +Style/RedundantException: + Enabled: true + +Style/SelfAssignment: + Enabled: true + +Style/Proc: + Enabled: true + +Style/RaiseArgs: + Enabled: true + +Style/RedundantBegin: + Enabled: true + +Style/RescueModifier: + Enabled: true + +# based on https://github.com/voxpupuli/modulesync_config/issues/168 Style/RegexpLiteral: + EnforcedStyle: percent_r Enabled: true -Style/WordArray: + +Lint/UnderscorePrefixedVariableName: Enabled: true -# this catches the cases of using `module` for parser functions, types, or -# providers -Style/ClassAndModuleChildren: +Metrics/ParameterLists: Enabled: false -Style/Documentation: - Description: 'Document classes and non-namespace modules.' - Enabled: false +Lint/RequireParentheses: + Enabled: true + +Style/SpaceBeforeFirstArg: + Enabled: true + +Style/ModuleFunction: + Enabled: true + +Lint/Debugger: + Enabled: true + +Style/IfWithSemicolon: + Enabled: true + +Style/Encoding: + Enabled: true -# More comfortable block layouts Style/BlockDelimiters: - Enabled: False + Enabled: true Style/MultilineBlockLayout: + Enabled: true + +# 'Complexity' is very relative +Metrics/AbcSize: + Enabled: False + +# 'Complexity' is very relative +Metrics/PerceivedComplexity: Enabled: False -# Rspec +Lint/UselessAssignment: + Enabled: true + +Style/ClosingParenthesisIndentation: + Enabled: false + +# RSpec # We don't use rspec in this way RSpec/DescribeClass: diff --git a/.travis.yml b/.travis.yml index cc86b12..5eb58f0 100644 --- a/.travis.yml +++ b/.travis.yml @@ -25,6 +25,8 @@ matrix: env: PUPPET_VERSION="~> 4.0" STRICT_VARIABLES="yes" CHECK=test - rvm: 2.2 env: PUPPET_VERSION="~> 4.0" STRICT_VARIABLES="yes" CHECK=test + - rvm: 2.3.1 + env: PUPPET_VERSION="~> 4.0" STRICT_VARIABLES="yes" CHECK=build - rvm: 2.3.1 env: PUPPET_VERSION="~> 4.0" STRICT_VARIABLES="yes" CHECK=rubocop - rvm: 2.3.1 @@ -41,4 +43,4 @@ deploy: # all_branches is required to use tags all_branches: true # Only publish if our main Ruby target builds - rvm: 2.2 + rvm: 2.3.1 diff --git a/Gemfile b/Gemfile index f9ac864..90188fe 100644 --- a/Gemfile +++ b/Gemfile @@ -15,7 +15,6 @@ group :test do gem 'rspec-puppet', :require => false, :git => 'https://github.com/rodjek/rspec-puppet.git' gem 'rspec-puppet-facts', :require => false gem 'rspec-puppet-utils', :require => false - gem 'puppet-lint', :require => false, :git => 'https://github.com/rodjek/puppet-lint.git' gem 'puppet-lint-absolute_classname-check', :require => false gem 'puppet-lint-leading_zero-check', :require => false gem 'puppet-lint-trailing_comma-check', :require => false @@ -37,7 +36,6 @@ group :development do end group :system_tests do - gem 'beaker', :require => false if beaker_version = ENV['BEAKER_VERSION'] gem 'beaker', *location_for(beaker_version) end diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index 0f53c4e..047b126 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -1,4 +1,17 @@ require 'puppetlabs_spec_helper/module_spec_helper' require 'rspec-puppet-facts' include RspecPuppetFacts + +RSpec.configure do |c| + c.default_facts = { + concat_basedir: '/tmp', + is_pe: false, + selinux_config_mode: 'disabled', + puppetversion: Puppet.version, + facterversion: Facter.version, + ipaddress: '172.16.254.254', + macaddress: 'AA:AA:AA:AA:AA:AA' + } +end + # vim: syntax=ruby -- cgit v1.2.3 From cf9154f3cb3074da3520dd21c3f0304545c04876 Mon Sep 17 00:00:00 2001 From: Tim Meusel Date: Mon, 11 Jul 2016 16:32:54 +0200 Subject: improve spec tests --- spec/classes/unattended_upgrades_spec.rb | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/spec/classes/unattended_upgrades_spec.rb b/spec/classes/unattended_upgrades_spec.rb index f6e74af..6141f63 100644 --- a/spec/classes/unattended_upgrades_spec.rb +++ b/spec/classes/unattended_upgrades_spec.rb @@ -15,7 +15,12 @@ describe 'unattended_upgrades' do } context 'with defaults on Debian' do - it { should contain_package('unattended-upgrades') } + it do + should contain_package('unattended-upgrades') + should compile.with_all_deps + should contain_class('unattended_upgrades::params') + should contain_class('unattended_upgrades') + end it { should contain_apt__conf('unattended-upgrades').with( -- cgit v1.2.3 From 6c01fe76275f0fdfedb56e30a60e9f72e6984960 Mon Sep 17 00:00:00 2001 From: Tim Meusel Date: Mon, 11 Jul 2016 16:33:25 +0200 Subject: rubocop: fix Style/TrailingCommaInLiteral --- spec/classes/unattended_upgrades_spec.rb | 66 ++++++++++++++++---------------- 1 file changed, 33 insertions(+), 33 deletions(-) diff --git a/spec/classes/unattended_upgrades_spec.rb b/spec/classes/unattended_upgrades_spec.rb index 6141f63..2d2bf79 100644 --- a/spec/classes/unattended_upgrades_spec.rb +++ b/spec/classes/unattended_upgrades_spec.rb @@ -8,7 +8,7 @@ describe 'unattended_upgrades' do osfamily: 'Debian', lsbdistid: 'Debian', lsbdistcodename: 'wheezy', - lsbrelease: '7.0.3', + lsbrelease: '7.0.3' } } let(:pre_condition) { 'include ::apt' @@ -133,7 +133,7 @@ describe 'unattended_upgrades' do osfamily: 'Debian', lsbdistid: 'Debian', lsbdistcodename: 'squeeze', - lsbdistrelease: '6.0.10', + lsbdistrelease: '6.0.10' } } it { should create_file(file_unattended).with( @@ -156,7 +156,7 @@ describe 'unattended_upgrades' do osfamily: 'Debian', lsbdistid: 'Debian', lsbdistcodename: 'wheezy', - lsbdistrelease: '7.1', + lsbdistrelease: '7.1' } } it { should create_file(file_unattended).with( @@ -178,7 +178,7 @@ describe 'unattended_upgrades' do osfamily: 'Debian', lsbdistid: 'Debian', lsbdistcodename: 'jessie', - lsbdistrelease: '8.2', + lsbdistrelease: '8.2' } } it { should create_file(file_unattended).with( @@ -200,7 +200,7 @@ describe 'unattended_upgrades' do lsbdistid: 'Ubuntu', lsbdistcodename: 'precise', lsbrelease: '12.04', - lsbdistrelease: '12.04', + lsbdistrelease: '12.04' } } it { should create_file(file_unattended).with( @@ -222,7 +222,7 @@ describe 'unattended_upgrades' do lsbdistid: 'Ubuntu', lsbdistcodename: 'trusty', lsbrelease: '14.04', - lsbdistrelease: '14.04', + lsbdistrelease: '14.04' } } it { should create_file(file_unattended).with( @@ -244,7 +244,7 @@ describe 'unattended_upgrades' do lsbdistid: 'Ubuntu', lsbdistcodename: 'vivid', lsbrelease: '15.04', - lsbdistrelease: '15.04', + lsbdistrelease: '15.04' } } it { should create_file(file_unattended).with( @@ -266,7 +266,7 @@ describe 'unattended_upgrades' do lsbdistid: 'Ubuntu', lsbdistcodename: 'wily', lsbrelease: '15.10', - lsbdistrelease: '15.10', + lsbdistrelease: '15.10' } } it { should create_file(file_unattended).with( @@ -287,7 +287,7 @@ describe 'unattended_upgrades' do osfamily: 'Debian', lsbdistid: 'Raspbian', lsbdistcodename: 'jessie', - lsbrelease: '8.0', + lsbrelease: '8.0' } } it { should create_file(file_unattended).with( @@ -304,7 +304,7 @@ describe 'unattended_upgrades' do lsbdistid: 'LinuxMint', lsbdistcodename: 'maya', lsbdistrelease: '13', - lsbmajdistrelease: '13', + lsbmajdistrelease: '13' } } it { should create_file(file_unattended).with( @@ -326,7 +326,7 @@ describe 'unattended_upgrades' do lsbdistid: 'LinuxMint', lsbdistcodename: 'rosa', lsbdistrelease: '17.3', - lsbmajdistrelease: '17', + lsbmajdistrelease: '17' } } it { should create_file(file_unattended).with( @@ -348,7 +348,7 @@ describe 'unattended_upgrades' do lsbdistid: 'LinuxMint', lsbdistcodename: 'sarah', lsbdistrelease: '18', - lsbmajdistrelease: '18', + lsbmajdistrelease: '18' } } it { should create_file(file_unattended).with( @@ -372,7 +372,7 @@ describe 'unattended_upgrades' do update: 5, upgradeable_packages: { 'download_only' => 5, - 'debdelta' => 5, + 'debdelta' => 5 }, upgrade: 5, auto: { @@ -380,7 +380,7 @@ describe 'unattended_upgrades' do 'fix_interrupted_dpkg' => false, 'remove' => false, 'reboot' => true, - 'reboot_time' => '03:00', + 'reboot_time' => '03:00' }, verbose: 1, legacy_origin: true, @@ -390,7 +390,7 @@ describe 'unattended_upgrades' do install_on_shutdown: true, mail: { 'to' => 'root@localhost', - 'only_on_error' => true, + 'only_on_error' => true }, dl_limit: 70, random_sleep: 300, @@ -399,7 +399,7 @@ describe 'unattended_upgrades' do 'force_confdef' => false, 'force_confold' => false, 'force_confnew' => true, - 'force_confmiss' => true, + 'force_confmiss' => true } } end @@ -516,7 +516,7 @@ describe 'unattended_upgrades' do context 'bad install_on_shutdown' do let :params do { - install_on_shutdown: 'foo', + install_on_shutdown: 'foo' } end it do @@ -528,7 +528,7 @@ describe 'unattended_upgrades' do context 'bad legacy_origin' do let :params do { - legacy_origin: 'foo', + legacy_origin: 'foo' } end it do @@ -540,7 +540,7 @@ describe 'unattended_upgrades' do context 'bad minimal_steps' do let :params do { - minimal_steps: 'foo', + minimal_steps: 'foo' } end it do @@ -552,7 +552,7 @@ describe 'unattended_upgrades' do context 'bad blacklist' do let :params do { - blacklist: 'foo', + blacklist: 'foo' } end it do @@ -564,7 +564,7 @@ describe 'unattended_upgrades' do context 'bad origins' do let :params do { - origins: 'foo', + origins: 'foo' } end it do @@ -576,7 +576,7 @@ describe 'unattended_upgrades' do context 'bad auto' do let :params do { - auto: 'foo', + auto: 'foo' } end it do @@ -588,7 +588,7 @@ describe 'unattended_upgrades' do context 'bad mail' do let :params do { - mail: 'foo', + mail: 'foo' } end it do @@ -600,7 +600,7 @@ describe 'unattended_upgrades' do context 'bad backup' do let :params do { - backup: 'foo', + backup: 'foo' } end it do @@ -612,7 +612,7 @@ describe 'unattended_upgrades' do context 'bad age' do let :params do { - age: 'foo', + age: 'foo' } end it do @@ -624,7 +624,7 @@ describe 'unattended_upgrades' do context 'bad size' do let :params do { - size: 'foo', + size: 'foo' } end it do @@ -636,7 +636,7 @@ describe 'unattended_upgrades' do context 'bad upgradeable_packages' do let :params do { - upgradeable_packages: 'foo', + upgradeable_packages: 'foo' } end it do @@ -648,7 +648,7 @@ describe 'unattended_upgrades' do context 'bad mail[\'only_on_error\']' do let :params do { - mail: { 'only_on_error' => 'foo' }, + mail: { 'only_on_error' => 'foo' } } end it do @@ -660,7 +660,7 @@ describe 'unattended_upgrades' do context 'bad options[\'force_confdef\']' do let :params do { - options: { 'force_confdef' => 'foo' }, + options: { 'force_confdef' => 'foo' } } end it do @@ -672,7 +672,7 @@ describe 'unattended_upgrades' do context 'bad options[\'force_confold\']' do let :params do { - options: { 'force_confold' => 'foo' }, + options: { 'force_confold' => 'foo' } } end it do @@ -684,7 +684,7 @@ describe 'unattended_upgrades' do context 'bad options[\'force_confnew\']' do let :params do { - options: { 'force_confnew' => 'foo' }, + options: { 'force_confnew' => 'foo' } } end it do @@ -696,7 +696,7 @@ describe 'unattended_upgrades' do context 'bad options[\'force_confmiss\']' do let :params do { - options: { 'force_confmiss' => 'foo' }, + options: { 'force_confmiss' => 'foo' } } end it do @@ -708,7 +708,7 @@ describe 'unattended_upgrades' do context 'bad options[\'invalid_key\']' do let :params do { - options: { 'invalid_key' => true }, + options: { 'invalid_key' => true } } end it do -- cgit v1.2.3 From 9b9a53778eb2ab2d131d9975216812263ee66582 Mon Sep 17 00:00:00 2001 From: Tim Meusel Date: Mon, 11 Jul 2016 16:36:37 +0200 Subject: rubocop: fix Style/TrailingCommaInArguments --- spec/classes/unattended_upgrades_spec.rb | 50 ++++++++++++++++---------------- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/spec/classes/unattended_upgrades_spec.rb b/spec/classes/unattended_upgrades_spec.rb index 2d2bf79..f13f78a 100644 --- a/spec/classes/unattended_upgrades_spec.rb +++ b/spec/classes/unattended_upgrades_spec.rb @@ -25,21 +25,21 @@ describe 'unattended_upgrades' do it { should contain_apt__conf('unattended-upgrades').with( require: 'Package[unattended-upgrades]', - notify_update: false, + notify_update: false ) } it { should contain_apt__conf('periodic').with( require: 'Package[unattended-upgrades]', - notify_update: false, + notify_update: false ) } it { should contain_apt__conf('options').with( require: 'Package[unattended-upgrades]', - notify_update: false, + notify_update: false ) } @@ -47,7 +47,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( /Unattended-Upgrade::Origins-Pattern {/ ).with_content( @@ -77,7 +77,7 @@ describe 'unattended_upgrades' do should create_file(file_periodic).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( /APT::Periodic::Enable "1";/ ).with_content( @@ -106,14 +106,14 @@ describe 'unattended_upgrades' do } it { should contain_apt__conf('auto-upgrades').with( - ensure: 'absent', + ensure: 'absent' ) } it { should create_file(file_options).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( /^Dpkg::Options\s{/ ).with_content( @@ -139,7 +139,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This section varies for different releases /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -162,7 +162,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This section varies for different releases /\Unattended-Upgrade::Origins-Pattern\ {\n @@ -184,7 +184,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This section varies for different releases /\Unattended-Upgrade::Origins-Pattern\ {\n @@ -206,7 +206,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -228,7 +228,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -250,7 +250,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -272,7 +272,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -293,7 +293,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ) } end @@ -310,7 +310,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( 'owner' => 'root', 'group' => 'root', - 'mode' => '0644', + 'mode' => '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -332,7 +332,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( 'owner' => 'root', 'group' => 'root', - 'mode' => '0644', + 'mode' => '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -354,7 +354,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( 'owner' => 'root', 'group' => 'root', - 'mode' => '0644', + 'mode' => '0644' ).with_content( # This is the only section that's different for Ubuntu compared to Debian /\Unattended-Upgrade::Allowed-Origins\ {\n @@ -407,20 +407,20 @@ describe 'unattended_upgrades' do it { should contain_apt__conf('unattended-upgrades').with( require: 'Package[unattended-upgrades]', - notify_update: true, + notify_update: true ) } it { should contain_apt__conf('periodic').with( require: 'Package[unattended-upgrades]', - notify_update: true, + notify_update: true ) } it { should contain_apt__conf('options').with( require: 'Package[unattended-upgrades]', - notify_update: true, + notify_update: true ) } @@ -428,7 +428,7 @@ describe 'unattended_upgrades' do should create_file(file_unattended).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( /Unattended-Upgrade::Allowed-Origins {\n\t"bananas";\n};/ ).with_content( @@ -458,7 +458,7 @@ describe 'unattended_upgrades' do should create_file(file_periodic).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( /APT::Periodic::Enable "1";/ ).with_content( @@ -492,7 +492,7 @@ describe 'unattended_upgrades' do should create_file(file_options).with( owner: 'root', group: 'root', - mode: '0644', + mode: '0644' ).with_content( /^Dpkg::Options\s{/ ).without_content( @@ -507,7 +507,7 @@ describe 'unattended_upgrades' do } it { should contain_apt__conf('auto-upgrades').with( - ensure: 'absent', + ensure: 'absent' ) } end -- cgit v1.2.3 From d9ee278b47a450f125bde2eb3eae5d33c4995236 Mon Sep 17 00:00:00 2001 From: Tim Meusel Date: Mon, 11 Jul 2016 16:43:00 +0200 Subject: rubocop: fix Style/MultilineBlockLayout --- spec/classes/unattended_upgrades_spec.rb | 369 +++++++++++++++++-------------- 1 file changed, 198 insertions(+), 171 deletions(-) diff --git a/spec/classes/unattended_upgrades_spec.rb b/spec/classes/unattended_upgrades_spec.rb index f13f78a..2262fe8 100644 --- a/spec/classes/unattended_upgrades_spec.rb +++ b/spec/classes/unattended_upgrades_spec.rb @@ -4,15 +4,17 @@ describe 'unattended_upgrades' do let(:file_unattended) { '/etc/apt/apt.conf.d/50unattended-upgrades' } let(:file_periodic) { '/etc/apt/apt.conf.d/10periodic' } let(:file_options) { '/etc/apt/apt.conf.d/10options' } - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Debian', - lsbdistcodename: 'wheezy', - lsbrelease: '7.0.3' - } } - let(:pre_condition) { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Debian', + lsbdistcodename: 'wheezy', + lsbrelease: '7.0.3' + } + end + let(:pre_condition) do 'include ::apt' - } + end context 'with defaults on Debian' do it do @@ -22,28 +24,28 @@ describe 'unattended_upgrades' do should contain_class('unattended_upgrades') end - it { + it do should contain_apt__conf('unattended-upgrades').with( require: 'Package[unattended-upgrades]', notify_update: false ) - } + end - it { + it do should contain_apt__conf('periodic').with( require: 'Package[unattended-upgrades]', notify_update: false ) - } + end - it { + it do should contain_apt__conf('options').with( require: 'Package[unattended-upgrades]', notify_update: false ) - } + end - it { + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -71,9 +73,9 @@ describe 'unattended_upgrades' do ).without_content( /Acquire::http::Dl-Limit/ ) - } + end - it { + it do should create_file(file_periodic).with( owner: 'root', group: 'root', @@ -103,13 +105,14 @@ describe 'unattended_upgrades' do ).with_content( /APT::Periodic::Verbose "0";/ ) - } + end - it { should contain_apt__conf('auto-upgrades').with( - ensure: 'absent' - ) - } - it { + it do + should contain_apt__conf('auto-upgrades').with( + ensure: 'absent' + ) + end + it do should create_file(file_options).with( owner: 'root', group: 'root', @@ -125,17 +128,19 @@ describe 'unattended_upgrades' do ).without_content( /\"--force-confmiss\";/ ) - } + end end context 'with defaults on Debian 6 Squeeze' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Debian', - lsbdistcodename: 'squeeze', - lsbdistrelease: '6.0.10' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Debian', + lsbdistcodename: 'squeeze', + lsbdistrelease: '6.0.10' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -148,17 +153,19 @@ describe 'unattended_upgrades' do \t"\${distro_id}\ \${distro_codename}-lts";\n };/x ) - } + end end context 'with defaults on Debian 7 Wheezy' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Debian', - lsbdistcodename: 'wheezy', - lsbdistrelease: '7.1' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Debian', + lsbdistcodename: 'wheezy', + lsbdistrelease: '7.1' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -170,17 +177,19 @@ describe 'unattended_upgrades' do \t"origin=Debian,archive=oldstable,label=Debian-Security";\n };/x ) - } + end end context 'with defaults on Debian 8 Jessie' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Debian', - lsbdistcodename: 'jessie', - lsbdistrelease: '8.2' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Debian', + lsbdistcodename: 'jessie', + lsbdistrelease: '8.2' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -191,18 +200,20 @@ describe 'unattended_upgrades' do \t"origin=Debian,codename=\${distro_codename},label=Debian-Security";\n };/x ) - } + end end context 'with defaults on Ubuntu 12.04LTS Precise Pangolin' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Ubuntu', - lsbdistcodename: 'precise', - lsbrelease: '12.04', - lsbdistrelease: '12.04' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Ubuntu', + lsbdistcodename: 'precise', + lsbrelease: '12.04', + lsbdistrelease: '12.04' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -213,18 +224,20 @@ describe 'unattended_upgrades' do \t"\${distro_id}\:\${distro_codename}-security";\n };/x ) - } + end end context 'with defaults on Ubuntu 14.04LTS Trusty Tahr' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Ubuntu', - lsbdistcodename: 'trusty', - lsbrelease: '14.04', - lsbdistrelease: '14.04' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Ubuntu', + lsbdistcodename: 'trusty', + lsbrelease: '14.04', + lsbdistrelease: '14.04' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -235,18 +248,20 @@ describe 'unattended_upgrades' do \t"\${distro_id}\:\${distro_codename}-security";\n };/x ) - } + end end context 'with defaults on Ubuntu 15.04 Vivid Vervet' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Ubuntu', - lsbdistcodename: 'vivid', - lsbrelease: '15.04', - lsbdistrelease: '15.04' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Ubuntu', + lsbdistcodename: 'vivid', + lsbrelease: '15.04', + lsbdistrelease: '15.04' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -257,18 +272,20 @@ describe 'unattended_upgrades' do \t"\${distro_id}\:\${distro_codename}-security";\n };/x ) - } + end end context 'with defaults on Ubuntu 15.10 Wily Werewolf' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Ubuntu', - lsbdistcodename: 'wily', - lsbrelease: '15.10', - lsbdistrelease: '15.10' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Ubuntu', + lsbdistcodename: 'wily', + lsbrelease: '15.10', + lsbdistrelease: '15.10' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -279,34 +296,38 @@ describe 'unattended_upgrades' do \t"\${distro_id}\:\${distro_codename}-security";\n };/x ) - } + end end context 'with defaults on Raspbian' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'Raspbian', - lsbdistcodename: 'jessie', - lsbrelease: '8.0' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'Raspbian', + lsbdistcodename: 'jessie', + lsbrelease: '8.0' + } + end + it do should create_file(file_unattended).with( owner: 'root', group: 'root', mode: '0644' ) - } + end end context 'with defaults on Linux Mint 13 Maya' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'LinuxMint', - lsbdistcodename: 'maya', - lsbdistrelease: '13', - lsbmajdistrelease: '13' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'LinuxMint', + lsbdistcodename: 'maya', + lsbdistrelease: '13', + lsbmajdistrelease: '13' + } + end + it do should create_file(file_unattended).with( 'owner' => 'root', 'group' => 'root', @@ -317,18 +338,20 @@ describe 'unattended_upgrades' do \t"Ubuntu\:precise-security";\n };/x ) - } + end end context 'with defaults on Linux Mint 17.3 Rosa' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'LinuxMint', - lsbdistcodename: 'rosa', - lsbdistrelease: '17.3', - lsbmajdistrelease: '17' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'LinuxMint', + lsbdistcodename: 'rosa', + lsbdistrelease: '17.3', + lsbmajdistrelease: '17' + } + end + it do should create_file(file_unattended).with( 'owner' => 'root', 'group' => 'root', @@ -339,18 +362,20 @@ describe 'unattended_upgrades' do \t"Ubuntu\:trusty-security";\n };/x ) - } + end end context 'with defaults on Linux Mint 18 Sarah' do - let(:facts) { { - osfamily: 'Debian', - lsbdistid: 'LinuxMint', - lsbdistcodename: 'sarah', - lsbdistrelease: '18', - lsbmajdistrelease: '18' - } } - it { + let(:facts) do + { + osfamily: 'Debian', + lsbdistid: 'LinuxMint', + lsbdistcodename: 'sarah', + lsbdistrelease: '18', + lsbmajdistrelease: '18' + } + end + it do should create_file(file_unattended).with( 'owner' => 'root', 'group' => 'root', @@ -361,7 +386,7 @@ describe 'unattended_upgrades' do \t"Ubuntu\:xenial-security";\n };/x ) - } + end end context 'set all the things' do @@ -405,26 +430,28 @@ describe 'unattended_upgrades' do end it { should contain_package('unattended-upgrades') } - it { should contain_apt__conf('unattended-upgrades').with( - require: 'Package[unattended-upgrades]', - notify_update: true + it do + should contain_apt__conf('unattended-upgrades').with( + require: 'Package[unattended-upgrades]', + notify_update: true ) - } + end - it { should contain_apt__conf('periodic').with( - require: 'Package[unattended-upgrades]', - notify_update: true + it do + should contain_apt__conf('periodic').with( + require: 'Package[unattended-upgrades]', + notify_update: true ) - } + end - it { + it do should contain_apt__conf('options').with( require: 'Package[unattended-upgrades]', notify_update: true ) - } + end - it { + it do should create_file(file_unattended).with( owner: 'root', group: 'root', @@ -452,9 +479,9 @@ describe 'unattended_upgrades' do ).with_content( /Acquire::http::Dl-Limit "70";/ ) - } + end - it { + it do should create_file(file_periodic).with( owner: 'root', group: 'root', @@ -486,9 +513,9 @@ describe 'unattended_upgrades' do ).with_content( /APT::Periodic::RandomSleep "300";/ ) - } + end - it { + it do should create_file(file_options).with( owner: 'root', group: 'root', @@ -504,12 +531,12 @@ describe 'unattended_upgrades' do ).with_content( /^\s+"--force-confmiss";/ ) - } - it { + end + it do should contain_apt__conf('auto-upgrades').with( ensure: 'absent' ) - } + end end describe 'validation tests' do @@ -520,9 +547,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad legacy_origin' do @@ -532,9 +559,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad minimal_steps' do @@ -544,9 +571,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad blacklist' do @@ -556,9 +583,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not an Array/) + end.to raise_error(Puppet::Error, /not an Array/) end end context 'bad origins' do @@ -568,9 +595,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not an Array/) + end.to raise_error(Puppet::Error, /not an Array/) end end context 'bad auto' do @@ -580,9 +607,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a Hash/) + end.to raise_error(Puppet::Error, /not a Hash/) end end context 'bad mail' do @@ -592,9 +619,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a Hash/) + end.to raise_error(Puppet::Error, /not a Hash/) end end context 'bad backup' do @@ -604,9 +631,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a Hash/) + end.to raise_error(Puppet::Error, /not a Hash/) end end context 'bad age' do @@ -616,9 +643,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a Hash/) + end.to raise_error(Puppet::Error, /not a Hash/) end end context 'bad size' do @@ -628,9 +655,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /to be an Integer/) + end.to raise_error(Puppet::Error, /to be an Integer/) end end context 'bad upgradeable_packages' do @@ -640,9 +667,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a Hash/) + end.to raise_error(Puppet::Error, /not a Hash/) end end context 'bad mail[\'only_on_error\']' do @@ -652,9 +679,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad options[\'force_confdef\']' do @@ -664,9 +691,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad options[\'force_confold\']' do @@ -676,9 +703,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad options[\'force_confnew\']' do @@ -688,9 +715,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad options[\'force_confmiss\']' do @@ -700,9 +727,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /not a boolean/) + end.to raise_error(Puppet::Error, /not a boolean/) end end context 'bad options[\'invalid_key\']' do @@ -712,9 +739,9 @@ describe 'unattended_upgrades' do } end it do - expect { + expect do subject.call - }.to raise_error(Puppet::Error, /invalid_key not a valid key/) + end.to raise_error(Puppet::Error, /invalid_key not a valid key/) end end end -- cgit v1.2.3 From c9a7131191c41a5154b92dc5a18d347524d7dd36 Mon Sep 17 00:00:00 2001 From: Tim Meusel Date: Mon, 11 Jul 2016 16:55:52 +0200 Subject: rubocop: disable Style/RegexpLiteral this cop doesn't make much sense in our spec file. the regex delimiter would be {}, but we often have to test for these braces. --- spec/classes/unattended_upgrades_spec.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/spec/classes/unattended_upgrades_spec.rb b/spec/classes/unattended_upgrades_spec.rb index 2262fe8..c3b9673 100644 --- a/spec/classes/unattended_upgrades_spec.rb +++ b/spec/classes/unattended_upgrades_spec.rb @@ -1,5 +1,6 @@ require 'spec_helper' +# rubocop:disable Style/RegexpLiteral describe 'unattended_upgrades' do let(:file_unattended) { '/etc/apt/apt.conf.d/50unattended-upgrades' } let(:file_periodic) { '/etc/apt/apt.conf.d/10periodic' } -- cgit v1.2.3