puppet-lint-manifest_whitespace-check 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82fd48a0b533bc6357b7e250f945f8ab3f0ba80625a19fa060fde543bf854dae
-  data.tar.gz: 5289f75f1e16b08fb0b18c6798326017625d6a9b71cb061e9f020e4267630a45
+  metadata.gz: e80c80c6f7e442284b4e19c5ba0dc556dedaca26bbdb2ab51cca142cea6000b9
+  data.tar.gz: ba85e3831db963d8a5dac5f423377bd1bc8cb780a0e780fc3eb892d854d7419a
 SHA512:
-  metadata.gz: 4ff960cc339885dfbfa62823058381bb609ca8de39f1047837f2882efb325e176251dcec6f9786907b5fa14145e4f8ad0dfea52e942eadc6909ebc1f8f8f3dd1
-  data.tar.gz: 9a4f2cd1929e5ca731efb25894730e0ec793d4f2ebc9f3a46baa045deb9d6891e092402afe42acdd8cb83e6cfa808d7033fdb515d60360a4be1a7452e83f54dd
+  metadata.gz: d6bc3ffe70c55c49dd71149291fae2be47b25fd9f4409026f69330be7c2368a5ea97f259f2e4899844e4bfeeff7311b492a80ad0882340e8398709b06be13887
+  data.tar.gz: d4d4c42cfa65144d4d5ce71edf7d46ce278a1e2a1efcaff866c73f73b3bbe9c573500592b82c0689314c77dd665f4eca0c325f12552b9b5e5b95239d57dfb1c2

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

@@ -4,7 +4,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_brace_before) do
   def check
     tokens.select { |token| token.type == :RBRACE }.each do |brace_token|
       prev_token = brace_token.prev_token
-      prev_code_token = brace_token.prev_code_token
+      prev_code_token = prev_non_space_token(brace_token)
 
       next unless prev_token && prev_code_token
       next if %i[LBRACE RBRACK RBRACE].include?(prev_token.type)
@@ -66,7 +66,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_brace_after) do
       next if after_bracket_tokens.include?(next_token.type)
 
       if next_token.type == :WHITESPACE
-        next_code_token = brace_token.next_code_token
+        next_code_token = next_non_space_token(brace_token)
         next unless next_code_token
         next unless after_bracket_tokens.include?(next_code_token.type)
       end

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

@@ -6,7 +6,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_bracket_before) do
       prev_token = bracket_token.prev_token
       next unless prev_token
 
-      prev_code_token = bracket_token.prev_code_token
+      prev_code_token = prev_non_space_token(bracket_token)
       next unless prev_code_token
 
       next unless %i[NEWLINE INDENT WHITESPACE].include?(prev_token.type)
@@ -56,7 +56,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_bracket_after) do
       next if after_bracket_tokens.include?(next_token.type)
 
       if next_token.type == :WHITESPACE
-        next_code_token = bracket_token.next_code_token
+        next_code_token = next_non_space_token(bracket_token)
         next unless next_code_token
         next unless after_bracket_tokens.include?(next_code_token.type)
       end

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

@@ -4,10 +4,10 @@ PuppetLint.new_check(:manifest_whitespace_opening_brace_before) do
   def check
     tokens.select { |token| token.type == :LBRACE }.each do |brace_token|
       prev_token = brace_token.prev_token
-      prev_code_token = brace_token.prev_code_token
+      prev_code_token = prev_non_space_token(brace_token)
 
       next unless prev_token && prev_code_token
-      next if %i[LBRACK LBRACE COMMA].include?(prev_code_token.type)
+      next if %i[LBRACK LBRACE COMMA COMMENT].include?(prev_code_token.type)
       next unless tokens.index(prev_code_token) != tokens.index(brace_token) - 2 ||
                   !is_single_space(prev_token)
 
@@ -24,7 +24,7 @@ PuppetLint.new_check(:manifest_whitespace_opening_brace_before) do
   def fix(problem)
     token = problem[:token]
     prev_token = token.prev_token
-    prev_code_token = token.prev_code_token
+    prev_code_token = prev_non_space_token(token)
 
     while prev_code_token != prev_token
       unless %i[WHITESPACE INDENT NEWLINE].include?(prev_token.type)

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

@@ -4,7 +4,7 @@ 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
-      prev_code_token = bracket_token.prev_code_token
+      prev_code_token = prev_non_space_token(bracket_token)
 
       next unless prev_token && prev_code_token
       next if %i[LBRACK LBRACE COMMA SEMIC].include?(prev_code_token.type)
@@ -25,7 +25,7 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
   def fix(problem)
     token = problem[:token]
     prev_token = token.prev_token
-    prev_code_token = token.prev_code_token
+    prev_code_token = prev_non_space_token(token)
 
     while prev_code_token != prev_token
       unless %i[WHITESPACE INDENT NEWLINE].include?(prev_token.type)

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

@@ -11,3 +11,15 @@ end
 def after_bracket_tokens
   %i[RBRACE RBRACK RPAREN SEMIC COMMA COLON NEWLINE DQPOST]
 end
+
+def prev_non_space_token(token)
+  while token = token.prev_token
+    return token unless %i[WHITESPACE INDENT NEWLINE].include?(token.type)
+  end
+end
+
+def next_non_space_token(token)
+  while token = token.next_token
+    return token unless %i[WHITESPACE INDENT NEWLINE].include?(token.type)
+  end
+end

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

@@ -5,6 +5,20 @@ require 'spec_helper'
 describe 'manifest_whitespace_closing_brace_before' do
   let(:closing_brace_msg) { 'there should be a bracket or a single newline before a closing brace' }
 
+  context 'with comment only' do
+    let(:code) do
+      <<~EOF
+        $value7 = {
+          # nothing
+        }
+      EOF
+    end
+
+    it 'should detect no problems' do
+      expect(problems).to have(0).problem
+    end
+  end
+
   context 'with no spaces' do
     let(:code) do
       <<~EOF
@@ -30,6 +44,9 @@ describe 'manifest_whitespace_closing_brace_before' do
           $value5 = []
           $value6 = {}
           $value7 = "x${server_facts['environment']}y"
+          $value8 = {
+            # nothing
+          }
 
           if someothercondition { include ::otherclass}
           if somecondition {
@@ -91,6 +108,9 @@ describe 'manifest_whitespace_closing_brace_before' do
               $value5 = []
               $value6 = {}
               $value7 = "x${server_facts['environment']}y"
+              $value8 = {
+                # nothing
+              }
 
               if someothercondition { include ::otherclass }
               if somecondition {

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

@@ -5,6 +5,20 @@ require 'spec_helper'
 describe 'manifest_whitespace_opening_brace_before' do
   let(:opening_brace_msg) { 'there should be a single space before an opening brace' }
 
+  context 'with comment only' do
+    let(:code) do
+      <<~EOF
+        $value7 = {
+          # nothing
+        }
+      EOF
+    end
+
+    it 'should detect no problems' do
+      expect(problems).to have(0).problem
+    end
+  end
+
   context 'with no spaces' do
     let(:code) do
       <<~EOF
@@ -29,6 +43,9 @@ describe 'manifest_whitespace_opening_brace_before' do
           $value4 = ['somekey']
           $value5 = []
           $value6 = {}
+          $value7 = {
+            # nothing
+          }
 
           if somecondition{
             class{ 'example2':
@@ -91,6 +108,9 @@ describe 'manifest_whitespace_opening_brace_before' do
               $value4 = ['somekey']
               $value5 = []
               $value6 = {}
+              $value7 = {
+                # nothing
+              }
 
               if somecondition {
                 class { 'example2':
@@ -332,29 +352,7 @@ describe 'manifest_whitespace_opening_brace_before' do
 
     context 'with fix disabled' do
       it 'should detect a single problem' do
-        expect(problems).to have(2).problem
-      end
-
-      it 'should create a error' do
-        expect(problems).to contain_error(opening_brace_msg).on_line(9).in_column(1)
-      end
-    end
-
-    context 'with fix enabled' do
-      before do
-        PuppetLint.configuration.fix = true
-      end
-
-      after do
-        PuppetLint.configuration.fix = false
-      end
-
-      it 'should detect a single problem' do
-        expect(problems).to have(2).problem
-      end
-
-      it 'should not fix the manifest' do
-        expect(problems).to contain_error(opening_brace_msg).on_line(9).in_column(1)
+        expect(problems).to have(0).problem
       end
     end
   end
@@ -516,6 +514,9 @@ describe 'manifest_whitespace_opening_brace_after' do
           $value4 = ['somekey']
           $value5 = []
           $value6 = {}
+          $value7 = {
+            # nothing
+          }
 
           if somecondition {
             class {  'example2':
@@ -578,6 +579,9 @@ describe 'manifest_whitespace_opening_brace_after' do
               $value4 = ['somekey']
               $value5 = []
               $value6 = {}
+              $value7 = {
+                # nothing
+              }
 
               if somecondition {
                 class { 'example2':

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-manifest_whitespace-check
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - Jo Vandeginste