puppet-lint-manifest_whitespace-check 0.0.1 → 0.1.6
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 +5 -5
- data/README.md +25 -3
- data/lib/puppet-lint/plugins/check_manifest_whitespace_class_inherits_single_space.rb +28 -0
- data/lib/puppet-lint/plugins/check_manifest_whitespace_class_name_single_space.rb +10 -10
- data/lib/puppet-lint/plugins/check_manifest_whitespace_closing_brace.rb +96 -0
- data/lib/puppet-lint/plugins/check_manifest_whitespace_closing_bracket.rb +86 -0
- data/lib/puppet-lint/plugins/check_manifest_whitespace_empty_lines.rb +35 -0
- data/lib/puppet-lint/plugins/check_manifest_whitespace_opening_brace.rb +83 -0
- data/lib/puppet-lint/plugins/check_manifest_whitespace_opening_bracket.rb +79 -0
- data/lib/puppet-lint/plugins/tools.rb +16 -0
- data/spec/puppet-lint/plugins/manifest_whitespace_class_header_spec.rb +35 -158
- data/spec/puppet-lint/plugins/manifest_whitespace_class_inherits_spec.rb +69 -0
- data/spec/puppet-lint/plugins/manifest_whitespace_closing_brace_spec.rb +460 -0
- data/spec/puppet-lint/plugins/manifest_whitespace_closing_bracket_spec.rb +277 -0
- data/spec/puppet-lint/plugins/manifest_whitespace_double_newline_spec.rb +60 -0
- data/spec/puppet-lint/plugins/manifest_whitespace_opening_brace_spec.rb +792 -0
- data/spec/puppet-lint/plugins/manifest_whitespace_opening_bracket_spec.rb +470 -0
- metadata +54 -37
- data/lib/puppet-lint/plugins/check_manifest_whitespace_class_opening_curly_brace.rb +0 -40
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: puppet-lint-manifest_whitespace-check
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.
|
4
|
+
version: 0.1.6
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Jo Vandeginste
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2020-06-
|
11
|
+
date: 2020-06-23 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: puppet-lint
|
@@ -31,89 +31,89 @@ dependencies:
|
|
31
31
|
- !ruby/object:Gem::Version
|
32
32
|
version: '3.0'
|
33
33
|
- !ruby/object:Gem::Dependency
|
34
|
-
name:
|
34
|
+
name: coveralls
|
35
35
|
requirement: !ruby/object:Gem::Requirement
|
36
36
|
requirements:
|
37
|
-
- - "
|
37
|
+
- - ">="
|
38
38
|
- !ruby/object:Gem::Version
|
39
|
-
version: '
|
39
|
+
version: '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
|
-
version: '
|
46
|
+
version: '0'
|
47
47
|
- !ruby/object:Gem::Dependency
|
48
|
-
name:
|
48
|
+
name: mime-types
|
49
49
|
requirement: !ruby/object:Gem::Requirement
|
50
50
|
requirements:
|
51
|
-
- - "
|
51
|
+
- - ">="
|
52
52
|
- !ruby/object:Gem::Version
|
53
|
-
version: '
|
53
|
+
version: '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
|
-
version: '
|
60
|
+
version: '0'
|
61
61
|
- !ruby/object:Gem::Dependency
|
62
|
-
name:
|
62
|
+
name: rake
|
63
63
|
requirement: !ruby/object:Gem::Requirement
|
64
64
|
requirements:
|
65
|
-
- - "
|
65
|
+
- - ">="
|
66
66
|
- !ruby/object:Gem::Version
|
67
|
-
version: '
|
67
|
+
version: '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
|
-
version: '
|
74
|
+
version: '0'
|
75
75
|
- !ruby/object:Gem::Dependency
|
76
|
-
name:
|
76
|
+
name: rspec
|
77
77
|
requirement: !ruby/object:Gem::Requirement
|
78
78
|
requirements:
|
79
|
-
- - "
|
79
|
+
- - "~>"
|
80
80
|
- !ruby/object:Gem::Version
|
81
|
-
version: '0'
|
81
|
+
version: '3.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
|
-
version: '0'
|
88
|
+
version: '3.0'
|
89
89
|
- !ruby/object:Gem::Dependency
|
90
|
-
name:
|
90
|
+
name: rspec-collection_matchers
|
91
91
|
requirement: !ruby/object:Gem::Requirement
|
92
92
|
requirements:
|
93
|
-
- - "
|
93
|
+
- - "~>"
|
94
94
|
- !ruby/object:Gem::Version
|
95
|
-
version: '0'
|
95
|
+
version: '1.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
|
-
version: '0'
|
102
|
+
version: '1.0'
|
103
103
|
- !ruby/object:Gem::Dependency
|
104
|
-
name:
|
104
|
+
name: rspec-its
|
105
105
|
requirement: !ruby/object:Gem::Requirement
|
106
106
|
requirements:
|
107
|
-
- - "
|
107
|
+
- - "~>"
|
108
108
|
- !ruby/object:Gem::Version
|
109
|
-
version: '0'
|
109
|
+
version: '1.0'
|
110
110
|
type: :development
|
111
111
|
prerelease: false
|
112
112
|
version_requirements: !ruby/object:Gem::Requirement
|
113
113
|
requirements:
|
114
|
-
- - "
|
114
|
+
- - "~>"
|
115
115
|
- !ruby/object:Gem::Version
|
116
|
-
version: '0'
|
116
|
+
version: '1.0'
|
117
117
|
description: " A new check for puppet-lint that validates generic whitespace issues
|
118
118
|
in manifests.\n"
|
119
119
|
email: jo.vandeginste@kuleuven.be
|
@@ -124,16 +124,27 @@ files:
|
|
124
124
|
- LICENSE
|
125
125
|
- README.md
|
126
126
|
- lib/puppet-lint/plugins/check_manifest_whitespace_arrow_spaces.rb
|
127
|
+
- lib/puppet-lint/plugins/check_manifest_whitespace_class_inherits_single_space.rb
|
127
128
|
- lib/puppet-lint/plugins/check_manifest_whitespace_class_name_single_space.rb
|
128
|
-
- lib/puppet-lint/plugins/
|
129
|
+
- lib/puppet-lint/plugins/check_manifest_whitespace_closing_brace.rb
|
130
|
+
- lib/puppet-lint/plugins/check_manifest_whitespace_closing_bracket.rb
|
131
|
+
- lib/puppet-lint/plugins/check_manifest_whitespace_empty_lines.rb
|
129
132
|
- lib/puppet-lint/plugins/check_manifest_whitespace_newline_beginning_of_file.rb
|
130
133
|
- lib/puppet-lint/plugins/check_manifest_whitespace_newline_end_of_file.rb
|
134
|
+
- lib/puppet-lint/plugins/check_manifest_whitespace_opening_brace.rb
|
135
|
+
- lib/puppet-lint/plugins/check_manifest_whitespace_opening_bracket.rb
|
131
136
|
- lib/puppet-lint/plugins/tools.rb
|
132
137
|
- spec/puppet-lint/plugins/manifest_whitespace_arrow_spaces_spec.rb
|
133
138
|
- spec/puppet-lint/plugins/manifest_whitespace_class_header_spec.rb
|
139
|
+
- spec/puppet-lint/plugins/manifest_whitespace_class_inherits_spec.rb
|
140
|
+
- spec/puppet-lint/plugins/manifest_whitespace_closing_brace_spec.rb
|
141
|
+
- spec/puppet-lint/plugins/manifest_whitespace_closing_bracket_spec.rb
|
134
142
|
- spec/puppet-lint/plugins/manifest_whitespace_double_newline_end_of_file_spec.rb
|
143
|
+
- spec/puppet-lint/plugins/manifest_whitespace_double_newline_spec.rb
|
135
144
|
- spec/puppet-lint/plugins/manifest_whitespace_missing_newline_end_of_file_spec.rb
|
136
145
|
- spec/puppet-lint/plugins/manifest_whitespace_newline_begin_of_file_spec.rb
|
146
|
+
- spec/puppet-lint/plugins/manifest_whitespace_opening_brace_spec.rb
|
147
|
+
- spec/puppet-lint/plugins/manifest_whitespace_opening_bracket_spec.rb
|
137
148
|
- spec/spec_helper.rb
|
138
149
|
homepage: https://github.com/kuleuven/puppet-lint-manifest_whitespace-check
|
139
150
|
licenses:
|
@@ -155,14 +166,20 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
155
166
|
version: '0'
|
156
167
|
requirements: []
|
157
168
|
rubyforge_project:
|
158
|
-
rubygems_version: 2.
|
169
|
+
rubygems_version: 2.7.7
|
159
170
|
signing_key:
|
160
171
|
specification_version: 4
|
161
172
|
summary: A puppet-lint check to validate whitespace in manifests
|
162
173
|
test_files:
|
174
|
+
- spec/spec_helper.rb
|
175
|
+
- spec/puppet-lint/plugins/manifest_whitespace_opening_bracket_spec.rb
|
176
|
+
- spec/puppet-lint/plugins/manifest_whitespace_class_inherits_spec.rb
|
177
|
+
- spec/puppet-lint/plugins/manifest_whitespace_closing_bracket_spec.rb
|
178
|
+
- spec/puppet-lint/plugins/manifest_whitespace_missing_newline_end_of_file_spec.rb
|
179
|
+
- spec/puppet-lint/plugins/manifest_whitespace_newline_begin_of_file_spec.rb
|
180
|
+
- spec/puppet-lint/plugins/manifest_whitespace_closing_brace_spec.rb
|
163
181
|
- spec/puppet-lint/plugins/manifest_whitespace_class_header_spec.rb
|
164
182
|
- spec/puppet-lint/plugins/manifest_whitespace_double_newline_end_of_file_spec.rb
|
165
|
-
- spec/puppet-lint/plugins/
|
183
|
+
- spec/puppet-lint/plugins/manifest_whitespace_double_newline_spec.rb
|
166
184
|
- spec/puppet-lint/plugins/manifest_whitespace_arrow_spaces_spec.rb
|
167
|
-
- spec/puppet-lint/plugins/
|
168
|
-
- spec/spec_helper.rb
|
185
|
+
- spec/puppet-lint/plugins/manifest_whitespace_opening_brace_spec.rb
|
@@ -1,40 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
PuppetLint.new_check(:manifest_whitespace_class_opening_curly_brace) do
|
4
|
-
def check
|
5
|
-
(class_indexes + defined_type_indexes).each do |class_idx|
|
6
|
-
class_token = class_idx[:tokens].first
|
7
|
-
bracket_token = class_token.next_token_of(:LBRACE)
|
8
|
-
prev_token = bracket_token.prev_token
|
9
|
-
prev_code_token = bracket_token.prev_token_of(:RPAREN)
|
10
|
-
|
11
|
-
next unless prev_code_token
|
12
|
-
next unless tokens.index(prev_code_token) != tokens.index(bracket_token) - 2 ||
|
13
|
-
!is_single_space(prev_token)
|
14
|
-
|
15
|
-
notify(
|
16
|
-
:error,
|
17
|
-
message: 'there should be a single space before the opening curly bracket of a class body',
|
18
|
-
line: bracket_token.line,
|
19
|
-
column: bracket_token.column,
|
20
|
-
token: bracket_token,
|
21
|
-
)
|
22
|
-
end
|
23
|
-
end
|
24
|
-
|
25
|
-
def fix(problem)
|
26
|
-
token = problem[:token]
|
27
|
-
prev_code_token = token.prev_token_of(:RPAREN).next_token
|
28
|
-
|
29
|
-
while token != prev_code_token
|
30
|
-
unless %i[WHITESPACE INDENT NEWLINE].include?(prev_code_token.type)
|
31
|
-
raise PuppetLint::NoFix
|
32
|
-
end
|
33
|
-
|
34
|
-
remove_token(prev_code_token)
|
35
|
-
prev_code_token = prev_code_token.next_token
|
36
|
-
end
|
37
|
-
|
38
|
-
add_token(tokens.index(token), new_single_space)
|
39
|
-
end
|
40
|
-
end
|