puppet-lint-manifest_whitespace-check 0.2.3 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68ebf647e803e56c8a3fdb45055b756a31a8aad16ba5e2626d5f9c37b2cd3edf
-  data.tar.gz: 9e982b6494eb4a85e9853374c65ac21dc68431d506254d3975244f2ca9b1a902
+  metadata.gz: c3fe61412435fe6e35447e02c54fe9b71937f5b1ceb30c65308eb8e1b61adaf0
+  data.tar.gz: d50c16634bb1f6609b5029b163a615d21e26e49df289be506634ee2f5240c205
 SHA512:
-  metadata.gz: 7af1583cd0809ca34d8e6aa4fd63353873a326c1372cc757fa625c20a478cf6f0bf6a72af032b38038c6fe51f18f01035862d9f3c5b0e28518db503869758b58
-  data.tar.gz: 9699d991b726203e67c2aac3c9c69aaa5a94888fb6196b2dfbdac402ae91db990a86b07ad16a83da2ec499aacbb1c3958be63d047afd1a86e679d571e9fa5980
+  metadata.gz: db462d214af83d3f3f32ae1d4aa91aad45c32d7c1ead916bc459cc56e1d345c258d4e28c3a5d48b2c13b890f7fa4a929177c3b91c3ce08d15a3618467c0d4a10
+  data.tar.gz: 2d57e6f697d41bf685bbc8b403fa11a2408c0aae3995b462f3f7f700c29ebc840a236767e841dce0c37233bd124227e3eec2f047c95ca1150caf7603d1072c79

data/lib/puppet-lint/plugins/check_manifest_whitespace_opening_brace.rb

@@ -14,7 +14,6 @@ PuppetLint.new_check(:manifest_whitespace_opening_brace_before) do
       if %i[LPAREN LBRACK LBRACE].include?(prev_code_token.type)
         next if tokens.index(prev_code_token) == tokens.index(brace_token) - 1
         next if tokens[tokens.index(prev_code_token)..tokens.index(brace_token)].collect(&:type).include?(:NEWLINE)
-
       else
         next unless tokens.index(prev_code_token) != tokens.index(brace_token) - 2 ||
                     !is_single_space(prev_token)

data/lib/puppet-lint/plugins/check_manifest_whitespace_opening_bracket.rb

@@ -4,15 +4,33 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
   def check
     tokens.select { |token| token.type == :LBRACK }.each do |bracket_token|
       prev_token = bracket_token.prev_token
+
+      next if %i[CLASSREF VARIABLE NAME].include?(prev_token.type)
+
       prev_code_token = prev_non_space_token(bracket_token)
 
-      next unless prev_token && prev_code_token
-      next if %i[LBRACK LBRACE COMMA SEMIC COMMENT NAME].include?(prev_code_token.type)
-      next unless %i[WHITESPACE NEWLINE INDENT].include?(prev_token.type)
+      next if %i[SEMIC COMMA COLON].include?(prev_code_token.type) && %i[INDENT NEWLINE].include?(prev_token.type)
+      next if %i[COMMENT].include?(prev_code_token.type)
+      next if %i[INDENT NEWLINE].include?(prev_token.type) && %i[NAME RBRACK RBRACE].include?(prev_code_token.type)
+
+      if %i[RBRACK RBRACE RPAREN LPAREN LBRACK LBRACE TYPE].include?(prev_code_token.type)
+        next if prev_code_token == prev_token
+        next if %i[INDENT NEWLINE].include?(prev_token.type)
+      elsif %i[CLASSREF VARIABLE].include?(prev_code_token.type)
+        # not good
+      elsif tokens.index(prev_code_token) == tokens.index(bracket_token) - 2 &&
+            is_single_space(prev_token)
+        next
+      end
+
+      if prev_code_token.type == :LBRACE
+        ppct = prev_non_space_token(prev_code_token)
 
-      next if %i[INDENT NEWLINE].include?(prev_token.type) && %i[RBRACK RBRACE].include?(prev_code_token.type)
-      next unless tokens.index(prev_code_token) != tokens.index(bracket_token) - 2 ||
-                  !is_single_space(prev_token)
+        if ppct && ppct.type == :NAME
+          next if %i[INDENT NEWLINE].include?(prev_token.type)
+          next if tokens.index(prev_code_token) == tokens.index(bracket_token) - 2 && is_single_space(prev_token)
+        end
+      end
 
       notify(
         :error,
@@ -36,7 +54,7 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
       prev_token = prev_token.prev_token
     end
 
-    add_token(tokens.index(token), new_single_space)
+    add_token(tokens.index(token), new_single_space) unless %i[LPAREN LBRACK LBRACE].include?(prev_code_token.type)
   end
 end
 

data/spec/puppet-lint/plugins/manifest_whitespace_opening_bracket_spec.rb

@@ -38,6 +38,18 @@ describe 'manifest_whitespace_opening_bracket_before' do
     end
   end
 
+  context 'with array key in interpolation' do
+    let(:code) do
+      <<~EOF
+        "${my_array['keyname']}"
+      EOF
+    end
+
+    it 'should detect no problems' do
+      expect(problems).to be_empty
+    end
+  end
+
   context 'with multiline iterator' do
     let(:code) do
       <<~EOF
@@ -65,7 +77,17 @@ describe 'manifest_whitespace_opening_bracket_before' do
 
         class example (
           String $content,
-        ){
+          Optional[String] $some_other_content,
+          Array $var = lookup('foo::bar', Array, undef, [])
+        ) {
+          if $fact["${var}"] != $var2.keys[0] {
+            # noop
+          }
+
+          if fact["${var}"] != $var2.keys[0] {
+            # noop
+          }
+
           $value = [{ 'key' => 'value' }]
           $value2 = [
             {
@@ -80,11 +102,20 @@ describe 'manifest_whitespace_opening_bracket_before' do
           $value5 = []
           $value6 = {}
 
-          if somecondition{
-            class{ 'example2':
+          if somecondition {
+            class { 'example2':
               param1  => 'value1',
               require => File['somefile'],
             }
+            package { ['pack1', 'pack2']:
+              ensure => present,
+            }
+            package {
+              ['pack3', 'pack4']:
+                ensure => present;
+              ['pack5', 'pack6']:
+                ensure => present;
+            }
           }
         }
       EOF
@@ -97,6 +128,41 @@ describe 'manifest_whitespace_opening_bracket_before' do
     end
   end
 
+  context 'with resource inline' do
+    let(:code) do
+      <<~EOF
+        package { ['pack1', 'pack2']:
+          ensure => present,
+        }
+      EOF
+    end
+
+    context 'with fix disabled' do
+      it 'should detect 0 problems' do
+        expect(problems).to be_empty
+      end
+    end
+  end
+
+  context 'with resource next line' do
+    let(:code) do
+      <<~EOF
+        package {
+          ['pack3', 'pack4']:
+            ensure => present;
+          ['pack5', 'pack6']:
+            ensure => present;
+        }
+      EOF
+    end
+
+    context 'with fix disabled' do
+      it 'should detect 0 problems' do
+        expect(problems).to be_empty
+      end
+    end
+  end
+
   context 'with two spaces' do
     let(:code) do
       <<~EOF

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-manifest_whitespace-check
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.5
 platform: ruby
 authors:
 - Jo Vandeginste
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-10-10 00:00:00.000000000 Z
+date: 2022-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: puppet-lint