Skip to content

Commit

Permalink
Merge pull request #159 from voxpupuli/dependabot/bundler/voxpupuli-r…
Browse files Browse the repository at this point in the history
…ubocop-tw-2.4.0

Update voxpupuli-rubocop requirement from ~> 2.0.0 to ~> 2.4.0
  • Loading branch information
bastelfreak authored Feb 7, 2024
2 parents 0854df1 + 55d1d99 commit 2e13c53
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 6 deletions.
15 changes: 14 additions & 1 deletion .rubocop_todo.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# This configuration was generated by
# `rubocop --auto-gen-config`
# on 2023-07-07 07:29:59 UTC using RuboCop version 1.50.2.
# on 2024-02-07 19:46:27 UTC using RuboCop version 1.60.2.
# The point is for the user to remove these configuration records
# one by one as the offenses are removed from the code base.
# Note that changes in the inspected code, or installation of new
Expand All @@ -13,6 +13,7 @@ Lint/DuplicateBranch:
- 'lib/puppet-syntax/templates.rb'

# Offense count: 1
# Configuration parameters: AllowedParentClasses.
Lint/MissingSuper:
Exclude:
- 'lib/puppet-syntax/tasks/puppet-syntax.rb'
Expand All @@ -29,6 +30,7 @@ Lint/SuppressedException:
- 'Rakefile'

# Offense count: 1
# This cop supports unsafe autocorrection (--autocorrect-all).
Lint/UselessAssignment:
Exclude:
- 'lib/puppet-syntax/templates.rb'
Expand Down Expand Up @@ -113,6 +115,17 @@ RSpec/RepeatedDescription:
Exclude:
- 'spec/puppet-syntax/templates_spec.rb'

# Offense count: 4
# Configuration parameters: Include, CustomTransform, IgnoreMethods, IgnoreMetadata.
# Include: **/*_spec.rb
RSpec/SpecFilePathFormat:
Exclude:
- '**/spec/routing/**/*'
- 'spec/puppet-syntax/hiera_spec.rb'
- 'spec/puppet-syntax/manifests_spec.rb'
- 'spec/puppet-syntax/templates_spec.rb'
- 'spec/puppet-syntax_spec.rb'

# Offense count: 3
RSpec/SubjectDeclaration:
Exclude:
Expand Down
2 changes: 1 addition & 1 deletion puppet-syntax.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@ Gem::Specification.new do |spec|
spec.add_development_dependency 'pry'
spec.add_development_dependency 'rb-readline'

spec.add_development_dependency 'voxpupuli-rubocop', '~> 2.0.0'
spec.add_development_dependency 'voxpupuli-rubocop', '~> 2.4.0'
end
8 changes: 4 additions & 4 deletions spec/puppet-syntax/hiera_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@
it 'returns nothing from valid YAML' do
files = fixture_hiera('hiera_good.yaml')
res = subject.check(files)
expect(res).to be == []
expect(res).to eq []
end

it 'returns an error from invalid YAML' do
files = fixture_hiera('hiera_bad.yaml')
expected = /ERROR: Failed to parse #{files[0]}:/
res = subject.check(files)
expect(res.size).to be == 1
expect(res.size).to eq 1
expect(res.first).to match(expected)
end

Expand All @@ -34,7 +34,7 @@
(1..examples).each do |n|
expect(res).to include(/::warning#{n}/)
end
expect(res.size).to be == examples
expect(res.size).to eq examples
expect(res[0]).to match('Key :typical:typo::warning1: Looks like a missing colon')
expect(res[1]).to match('Key ::notsotypical::warning2: Puppet automatic lookup will not use leading \'::\'')
expect(res[2]).to match('Key :noCamelCase::warning3: Not a valid Puppet variable name for automatic lookup')
Expand All @@ -50,7 +50,7 @@
(1..examples).each do |n|
expect(res).to include(/::warning#{n}/)
end
expect(res.size).to be == examples
expect(res.size).to eq examples
expect(res[0]).to match('Key acme::warning1 has unknown eyaml method unknown-method')
expect(res[1]).to match('Key acme::warning2 has unterminated eyaml value')
expect(res[2]).to match('Key acme::warning3 has unpadded or truncated base64 data')
Expand Down

0 comments on commit 2e13c53

Please sign in to comment.