Skip to content

Commit

Permalink
(CONT-666) Skip CLASSREF types
Browse files Browse the repository at this point in the history
This commit fixes voxpupuli/puppet-lint-trailing_comma-check#31.

It appears that the `resource_indexes` method cannot differentiate between a
resource declaration and a defaults declaration.

If we compare the two, you can see that a defaults declaration type starts with a
capital letter whereas a resource declaration does not.

```puppet
Service { 'foo':
  ensure => running,
}
```

```puppet
service { 'foo':
  ensure => running,
}
```

When the `resource_indexes` method runs, it initially selects tokens by the
presence of a colon then works to the right to decide if there are any violations.

That means we are actually only starting to collect information about the
declaration after the name.. so we never know the actual type of the declaration.

```puppet
Service { 'foo':
               ↑ (here)
  ensure => running,
}
```

https://github.com/puppetlabs/puppet-lint/blob/main/lib/puppet-lint/data.rb#L167

In contrast, `defaults_indexes` will skip any tokens that do not have a type of
`:CLASSREF` because we know that a defaults declaration will always start
with a capital letter (See [puppet-lint.com](http://puppet-lint.com/developer/tokens/)
for more information on token types).

This commit fixes the above by ensuring that `resource_indexes` checks
the type of resource that it is analysing and skips the itteration if it
detects a `:CLASSREF`.
  • Loading branch information
chelnak committed Feb 27, 2023
1 parent 8a6442d commit cb76ffc
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions lib/puppet-lint/data.rb
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,19 @@ def title_tokens
end
end

def is_classref?(token)
current_token = token
while true
current_token = current_token.prev_code_token

break if current_token.nil?

return true if current_token.type == :CLASSREF
return false if current_token.type == :NAME
end
end


# Internal: Calculate the positions of all resource declarations within the
# tokenised manifest. These positions only point to the content of the
# resource declarations, they do not include resource types or titles.
Expand All @@ -169,6 +182,7 @@ def resource_indexes
marker = 0
result = []
tokens.select { |t| t.type == :COLON }.each do |colon_token|
next if is_classref?(colon_token)
next unless colon_token.next_code_token && colon_token.next_code_token.type != :LBRACE

rel_start_idx = tokens[marker..-1].index(colon_token)
Expand Down

0 comments on commit cb76ffc

Please sign in to comment.