puppet-lint-manifest_whitespace-check 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68ebf647e803e56c8a3fdb45055b756a31a8aad16ba5e2626d5f9c37b2cd3edf
-  data.tar.gz: 9e982b6494eb4a85e9853374c65ac21dc68431d506254d3975244f2ca9b1a902
+  metadata.gz: 1a7733009c47320dbae135031b97cd389cd2edab33095ef5a9c64d2b61375d8c
+  data.tar.gz: 5e356c323847a31282c948131d41cd6c9b4f40338818bf42f46b65d3320694b1
 SHA512:
-  metadata.gz: 7af1583cd0809ca34d8e6aa4fd63353873a326c1372cc757fa625c20a478cf6f0bf6a72af032b38038c6fe51f18f01035862d9f3c5b0e28518db503869758b58
-  data.tar.gz: 9699d991b726203e67c2aac3c9c69aaa5a94888fb6196b2dfbdac402ae91db990a86b07ad16a83da2ec499aacbb1c3958be63d047afd1a86e679d571e9fa5980
+  metadata.gz: a8621832c8cf644f7ad467a1215d944ce2427ad97c127f2b60521bd6e6f743f9d37265a030dc11e9491455ab4ce50bab6db6cef1de0417d058c28fa8731fb110
+  data.tar.gz: 7bd64637bf4bde4a1e71c9dd50534a69a63bb698bda7097f68de2759da4bcd8e0b8f7146686c5dc6e38bd539a8e31c1c67708229127e3f4ea448dbbc80bc198e

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

@@ -6,13 +6,29 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
       prev_token = bracket_token.prev_token
       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)
+      next if %i[CLASSREF VARIABLE].include?(prev_code_token.type) && (prev_code_token == prev_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 %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)
+
+        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 +52,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,8 @@ describe 'manifest_whitespace_opening_bracket_before' do
 
         class example (
           String $content,
-        ){
+          Optional[String] $some_other_content,
+        ) {
           $value = [{ 'key' => 'value' }]
           $value2 = [
             {
@@ -80,11 +93,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 +119,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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-manifest_whitespace-check
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Jo Vandeginste