puppet-lint-variable_contains_upcase 1.0.2 → 1.1.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.
Files changed (4) hide show
  1. checksums.yaml +4 -4
  2. data/lib/puppet-lint/plugins/variable_contains_upcase.rb +4 -0
  3. data/spec/puppet-lint/plugins/variable_contains_upcase_spec.rb +27 -0
  4. metadata +32 -18
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA1:
3
- metadata.gz: 39554932c521c1f42fc75d6da016a85386221eea
4
- data.tar.gz: 4bcff0bd60853ecc26115cd4ac57c00bf0b907a5
3
+ metadata.gz: c47187b5699ffc368d42cd63ca37cbc4cc7c8d7c
4
+ data.tar.gz: 4895198bfe98f7b40a88961220916b9dcb14eb7a
5
5
  SHA512:
6
- metadata.gz: bb55819703cce72b77b63ee168b30d0dfcfe3459dd0dce86d54db538cffba0cf78c37d011bf7e4924de77f438c6c3206bc0954420deafc4bf19b958c7db22d9e
7
- data.tar.gz: 3b962fdd599a186590602dc200c169a66068c7ee322a4904825a10fe49d722f08aab8f079efd00716b67a758e8115e2a190086afbb563d353acdbc292a7df483
6
+ metadata.gz: a74bb8eed94f347455b6fda32eb76fb9346e32f415c386db4f68c87c2abf0c838d3338cce3635932196b8d8bbb8137af73c86fe0e3c9d3e66095fde0e193dba2
7
+ data.tar.gz: dd3d9997e01d2cf51cb99883c4b3ab2b2bf3c1590cbbcc86fd3c48b1dd531fa5b3e8079696a93a42fc45316b84d88d90b19fe4a86f0f364704a562fcee0440be
data/lib/puppet-lint/plugins/variable_contains_upcase.rb CHANGED
@@ -8,8 +8,12 @@ PuppetLint.new_check(:variable_contains_upcase) do
8
8
  :message => 'variable contains a capital letter',
9
9
  :line => token.line,
10
10
  :column => token.column,
11
+ :token => token,
11
12
  }
12
13
  end
13
14
  end
14
15
  end
16
+ def fix(problem)
17
+ problem[:token].value.downcase
18
+ end
15
19
  end
data/spec/puppet-lint/plugins/variable_contains_upcase_spec.rb CHANGED
@@ -26,4 +26,31 @@ describe 'variable_contains_upcase' do
26
26
  expect(problems).to contain_warning(msg).on_line(1).in_column(3)
27
27
  end
28
28
  end
29
+
30
+ context 'with fix enabled' do
31
+ before do
32
+ PuppetLint.configuration.fix = true
33
+ end
34
+
35
+ after do
36
+ PuppetLint.configuration.fix = false
37
+ end
38
+
39
+ context 'fix variable containing a capital' do
40
+ let(:code) { '" $fOobar"' }
41
+
42
+ it 'should only detect a single problem' do
43
+ expect(problems).to have(1).problem
44
+ end
45
+
46
+ it 'should fix the manifest' do
47
+ expect(problems).to contain_fixed(msg).on_line(1).in_column(3)
48
+ end
49
+
50
+ it 'should downcase the variable name' do
51
+ expect(manifest).to eq('" $foobar"')
52
+ end
53
+ end
54
+
55
+ end
29
56
  end
metadata CHANGED
@@ -1,89 +1,103 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: puppet-lint-variable_contains_upcase
3
3
  version: !ruby/object:Gem::Version
4
- version: 1.0.2
4
+ version: 1.1.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: 2016-08-24 00:00:00.000000000 Z
11
+ date: 2017-01-19 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
22
  version: '3.0'
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
32
  version: '3.0'
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
+ - !ruby/object:Gem::Version
88
+ version: '0'
89
+ - !ruby/object:Gem::Dependency
90
+ name: rspec-json_expectations
91
+ requirement: !ruby/object:Gem::Requirement
92
+ requirements:
93
+ - - '>='
94
+ - !ruby/object:Gem::Version
95
+ version: '0'
96
+ type: :development
97
+ prerelease: false
98
+ version_requirements: !ruby/object:Gem::Requirement
99
+ requirements:
100
+ - - '>='
87
101
  - !ruby/object:Gem::Version
88
102
  version: '0'
89
103
  description: |2
@@ -93,8 +107,8 @@ executables: []
93
107
  extensions: []
94
108
  extra_rdoc_files: []
95
109
  files:
96
- - LICENSE
97
110
  - README.md
111
+ - LICENSE
98
112
  - lib/puppet-lint/plugins/variable_contains_upcase.rb
99
113
  - spec/puppet-lint/plugins/variable_contains_upcase_spec.rb
100
114
  - spec/spec_helper.rb
@@ -108,17 +122,17 @@ require_paths:
108
122
  - lib
109
123
  required_ruby_version: !ruby/object:Gem::Requirement
110
124
  requirements:
111
- - - ">="
125
+ - - '>='
112
126
  - !ruby/object:Gem::Version
113
127
  version: '0'
114
128
  required_rubygems_version: !ruby/object:Gem::Requirement
115
129
  requirements:
116
- - - ">="
130
+ - - '>='
117
131
  - !ruby/object:Gem::Version
118
132
  version: '0'
119
133
  requirements: []
120
134
  rubyforge_project:
121
- rubygems_version: 2.2.3
135
+ rubygems_version: 2.0.14.1
122
136
  signing_key:
123
137
  specification_version: 4
124
138
  summary: puppet-lint variable_case check