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
|
-
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
2
|
+
SHA256:
|
3
|
+
metadata.gz: bf3b9e48706995481fdef10c0869330e6345e3defdc29f105e908ec902b16c13
|
4
|
+
data.tar.gz: cc96c29ac6c9bf139b334b1dea6d658f9dbe4bf5f9ffe8ecf2a07ac186e50c78
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
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
|
[](https://travis-ci.org/fiddyspence/puppetlint-variablecase)
|
4
|
-
[](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.
|
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:
|
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: '
|
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: '
|
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:
|
104
|
-
|
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
|
-
|
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
|