puppet-lint-variable_contains_upcase 1.1.0 → 1.3.0

Sign up to get free protection for your applications and to get access to all the features.
Files changed (5) hide show
  1. checksums.yaml +5 -5
  2. data/README.md +0 -1
  3. data/lib/puppet-lint/plugins/variable_contains_upcase.rb +2 -2
  4. data/spec/puppet-lint/plugins/variable_contains_upcase_spec.rb +21 -0
  5. metadata +25 -26
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
data/lib/puppet-lint/plugins/variable_contains_upcase.rb CHANGED
@@ -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
data/spec/puppet-lint/plugins/variable_contains_upcase_spec.rb CHANGED
@@ -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