puppet-lint-variable_contains_upcase 1.1.0 → 1.3.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
- SHA1:
3
- metadata.gz: c47187b5699ffc368d42cd63ca37cbc4cc7c8d7c
4
- data.tar.gz: 4895198bfe98f7b40a88961220916b9dcb14eb7a
2
+ SHA256:
3
+ metadata.gz: bf3b9e48706995481fdef10c0869330e6345e3defdc29f105e908ec902b16c13
4
+ data.tar.gz: cc96c29ac6c9bf139b334b1dea6d658f9dbe4bf5f9ffe8ecf2a07ac186e50c78
5
5
  SHA512:
6
- metadata.gz: a74bb8eed94f347455b6fda32eb76fb9346e32f415c386db4f68c87c2abf0c838d3338cce3635932196b8d8bbb8137af73c86fe0e3c9d3e66095fde0e193dba2
7
- data.tar.gz: dd3d9997e01d2cf51cb99883c4b3ab2b2bf3c1590cbbcc86fd3c48b1dd531fa5b3e8079696a93a42fc45316b84d88d90b19fe4a86f0f364704a562fcee0440be
6
+ metadata.gz: bb2c28e9c276f5a3938af2a4b8d463e6530c783a21581bf5adffc8b1968d7fc9e73d188f47217fc9b9cbcd20ce02ba06f86c5a9944b586369c707225cec46d09
7
+ data.tar.gz: 242b0dac3f613e5ce9153d644464c853fbae0770411110ada8057e710b4bd9418dd386d9cfd6bc1a34c7815a635d2ca2e3aef47165229e03c5636545e49c5607
data/README.md CHANGED
@@ -1,7 +1,6 @@
1
1
  # Puppet lint check for upcase variables
2
2
 
3
3
  [![Build Status](https://travis-ci.org/fiddyspence/puppetlint-variablecase.png?branch=master)](https://travis-ci.org/fiddyspence/puppetlint-variablecase)
4
- [![puppet-lint-variable_contains_upcase API Documentation](https://www.omniref.com/ruby/gems/puppet-lint-variable_contains_upcase.png)](https://www.omniref.com/ruby/gems/puppet-lint-variable_contains_upcase)
5
4
 
6
5
  The future parser disallows mixed case variables, so this check implements
7
6
  warnings for them
@@ -3,7 +3,7 @@ PuppetLint.new_check(:variable_contains_upcase) do
3
3
  tokens.select { |r|
4
4
  Set[:VARIABLE, :UNENC_VARIABLE].include? r.type
5
5
  }.each do |token|
6
- if token.value.match(/[A-Z]/)
6
+ if token.value.gsub(/\[.+?\]/, '').match(/[A-Z]/)
7
7
  notify :warning, {
8
8
  :message => 'variable contains a capital letter',
9
9
  :line => token.line,
@@ -14,6 +14,6 @@ PuppetLint.new_check(:variable_contains_upcase) do
14
14
  end
15
15
  end
16
16
  def fix(problem)
17
- problem[:token].value.downcase
17
+ problem[:token].value.downcase!
18
18
  end
19
19
  end
@@ -52,5 +52,26 @@ describe 'variable_contains_upcase' do
52
52
  end
53
53
  end
54
54
 
55
+ context 'a hash with an upcase key should not fail' do
56
+ let(:code) { ' $myhash["Foo"]' }
57
+
58
+ it 'should not detect a single problem' do
59
+ expect(problems).to have(0).problem
60
+ end
61
+ end
62
+ context 'a hash with an upcase key should not fail' do
63
+ let(:code) { ' $myhash["Foo"]["bAr"]' }
64
+
65
+ it 'should not detect a single problem' do
66
+ expect(problems).to have(0).problem
67
+ end
68
+ end
69
+ context 'a hash variable with an upcase key should fail' do
70
+ let(:code) { ' $myHash["foo"]["bar"]' }
71
+
72
+ it 'should detect a single problem' do
73
+ expect(problems).to have(1).problem
74
+ end
75
+ end
55
76
  end
56
77
  end
metadata CHANGED
@@ -1,114 +1,114 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: puppet-lint-variable_contains_upcase
3
3
  version: !ruby/object:Gem::Version
4
- version: 1.1.0
4
+ version: 1.3.0
5
5
  platform: ruby
6
6
  authors:
7
7
  - Chris Spence
8
8
  autorequire:
9
9
  bindir: bin
10
10
  cert_chain: []
11
- date: 2017-01-19 00:00:00.000000000 Z
11
+ date: 2022-11-29 00:00:00.000000000 Z
12
12
  dependencies:
13
13
  - !ruby/object:Gem::Dependency
14
14
  name: puppet-lint
15
15
  requirement: !ruby/object:Gem::Requirement
16
16
  requirements:
17
- - - '>='
17
+ - - ">="
18
18
  - !ruby/object:Gem::Version
19
19
  version: '1.0'
20
- - - <
20
+ - - "<"
21
21
  - !ruby/object:Gem::Version
22
- version: '3.0'
22
+ version: '4'
23
23
  type: :runtime
24
24
  prerelease: false
25
25
  version_requirements: !ruby/object:Gem::Requirement
26
26
  requirements:
27
- - - '>='
27
+ - - ">="
28
28
  - !ruby/object:Gem::Version
29
29
  version: '1.0'
30
- - - <
30
+ - - "<"
31
31
  - !ruby/object:Gem::Version
32
- version: '3.0'
32
+ version: '4'
33
33
  - !ruby/object:Gem::Dependency
34
34
  name: rspec
35
35
  requirement: !ruby/object:Gem::Requirement
36
36
  requirements:
37
- - - ~>
37
+ - - "~>"
38
38
  - !ruby/object:Gem::Version
39
39
  version: '3.0'
40
40
  type: :development
41
41
  prerelease: false
42
42
  version_requirements: !ruby/object:Gem::Requirement
43
43
  requirements:
44
- - - ~>
44
+ - - "~>"
45
45
  - !ruby/object:Gem::Version
46
46
  version: '3.0'
47
47
  - !ruby/object:Gem::Dependency
48
48
  name: rspec-its
49
49
  requirement: !ruby/object:Gem::Requirement
50
50
  requirements:
51
- - - ~>
51
+ - - "~>"
52
52
  - !ruby/object:Gem::Version
53
53
  version: '1.0'
54
54
  type: :development
55
55
  prerelease: false
56
56
  version_requirements: !ruby/object:Gem::Requirement
57
57
  requirements:
58
- - - ~>
58
+ - - "~>"
59
59
  - !ruby/object:Gem::Version
60
60
  version: '1.0'
61
61
  - !ruby/object:Gem::Dependency
62
62
  name: rspec-collection_matchers
63
63
  requirement: !ruby/object:Gem::Requirement
64
64
  requirements:
65
- - - ~>
65
+ - - "~>"
66
66
  - !ruby/object:Gem::Version
67
67
  version: '1.0'
68
68
  type: :development
69
69
  prerelease: false
70
70
  version_requirements: !ruby/object:Gem::Requirement
71
71
  requirements:
72
- - - ~>
72
+ - - "~>"
73
73
  - !ruby/object:Gem::Version
74
74
  version: '1.0'
75
75
  - !ruby/object:Gem::Dependency
76
76
  name: rake
77
77
  requirement: !ruby/object:Gem::Requirement
78
78
  requirements:
79
- - - '>='
79
+ - - ">="
80
80
  - !ruby/object:Gem::Version
81
81
  version: '0'
82
82
  type: :development
83
83
  prerelease: false
84
84
  version_requirements: !ruby/object:Gem::Requirement
85
85
  requirements:
86
- - - '>='
86
+ - - ">="
87
87
  - !ruby/object:Gem::Version
88
88
  version: '0'
89
89
  - !ruby/object:Gem::Dependency
90
90
  name: rspec-json_expectations
91
91
  requirement: !ruby/object:Gem::Requirement
92
92
  requirements:
93
- - - '>='
93
+ - - ">="
94
94
  - !ruby/object:Gem::Version
95
95
  version: '0'
96
96
  type: :development
97
97
  prerelease: false
98
98
  version_requirements: !ruby/object:Gem::Requirement
99
99
  requirements:
100
- - - '>='
100
+ - - ">="
101
101
  - !ruby/object:Gem::Version
102
102
  version: '0'
103
- description: |2
104
- Extends puppet-lint to ensure that your variables are all lower case
103
+ description: " Extends puppet-lint to ensure that your variables are all lower
104
+ case\n"
105
105
  email: chris@spence.org.uk
106
106
  executables: []
107
107
  extensions: []
108
108
  extra_rdoc_files: []
109
109
  files:
110
- - README.md
111
110
  - LICENSE
111
+ - README.md
112
112
  - lib/puppet-lint/plugins/variable_contains_upcase.rb
113
113
  - spec/puppet-lint/plugins/variable_contains_upcase_spec.rb
114
114
  - spec/spec_helper.rb
@@ -122,20 +122,19 @@ require_paths:
122
122
  - lib
123
123
  required_ruby_version: !ruby/object:Gem::Requirement
124
124
  requirements:
125
- - - '>='
125
+ - - ">="
126
126
  - !ruby/object:Gem::Version
127
127
  version: '0'
128
128
  required_rubygems_version: !ruby/object:Gem::Requirement
129
129
  requirements:
130
- - - '>='
130
+ - - ">="
131
131
  - !ruby/object:Gem::Version
132
132
  version: '0'
133
133
  requirements: []
134
- rubyforge_project:
135
- rubygems_version: 2.0.14.1
134
+ rubygems_version: 3.1.2
136
135
  signing_key:
137
136
  specification_version: 4
138
137
  summary: puppet-lint variable_case check
139
138
  test_files:
140
- - spec/puppet-lint/plugins/variable_contains_upcase_spec.rb
141
139
  - spec/spec_helper.rb
140
+ - spec/puppet-lint/plugins/variable_contains_upcase_spec.rb