puppet-lint-manifest_whitespace-check 0.2.7 → 0.2.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f296fd04d6453e0737d586c3c64bcd76c1d92cbf906a4789c1b9a6797d032d82
-  data.tar.gz: f8c3e2ce53f3654bb1746e5132b049111ba3d1cfabc11174fb121c312ff403ab
+  metadata.gz: 3fc583d9842a6466e61e1f6f2f9403f8fc5cb8e0958fafa9f5bd5f5fc80696ca
+  data.tar.gz: d4ab05139da0cc924d211d1f1b48fe78126bf44d554f8fa43ec01c13d1025daf
 SHA512:
-  metadata.gz: f4a888c0b801d677132b618d21719ab02e93a446f5e473c3dc1a2bdc70c539396155e50ea312e8fa5de21affd8e3b1ead8521a884aa697978cedd30c820206d3
-  data.tar.gz: 6b7183ef9094d3f0c42ee80e34fd6f7d54a571a510431291d9025aa5b896bb0f98fbed5d262c0ab0fd53949b13c279baa7988af0159658fd8230b6dbe77f665d
+  metadata.gz: ed6e8917d9cb0cb98fc3f4d1be0ff82c15f7328b8dbd0fda51a156eb6ca9546dbb5af389321da9a3b16308fdc3870280c5dd48457344b90d09fb3ce177938e17
+  data.tar.gz: f9fe4bf529999d65acf13efaa4cea04410714cf6f122e2986b82c3eca6c9ad28dc8369b2cf937547e3e464a6bccf5544e90500b0e1528e16aad57ec6a111e929

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

@@ -9,9 +9,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_brace_before) do
       next unless prev_token && prev_code_token
       next if %i[LBRACE].include?(prev_token.type)
 
-      if !%i[LBRACE].include?(prev_code_token.type) && (is_single_space(prev_token) && tokens.index(prev_code_token) == tokens.index(brace_token) - 2)
-        next
-      end
+      next if !%i[LBRACE].include?(prev_code_token.type) && (is_single_space(prev_token) && tokens.index(prev_code_token) == tokens.index(brace_token) - 2)
 
       next if prev_token.type == :INDENT && (tokens.index(prev_code_token) == tokens.index(brace_token) - 3)
 
@@ -41,9 +39,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_brace_before) do
       next_token = next_token.next_token
     end
 
-    if next_token.type == :RBRACE && !%i[LBRACE NEWLINE INDENT].include?(next_token.prev_token.type)
-      add_token(tokens.index(next_token), new_single_space)
-    end
+    add_token(tokens.index(next_token), new_single_space) if next_token.type == :RBRACE && !%i[LBRACE NEWLINE INDENT].include?(next_token.prev_token.type)
   end
 end
 

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

@@ -25,8 +25,9 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
 
       if prev_code_token.type == :LBRACE
         ppct = prev_non_space_token(prev_code_token)
+        next unless ppct
 
-        if ppct && ppct.type == :NAME
+        if %i[NAME PIPE].include?(ppct.type)
           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

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

@@ -7,19 +7,19 @@ describe 'manifest_whitespace_arrows_single_space_after' do
 
   context 'with two spaces' do
     let(:code) do
-      <<~EOF
+      <<~CODE
         class { 'example2':
           param1 =>  'value1',
         }
-      EOF
+      CODE
     end
 
     context 'with fix disabled' do
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(1).problem
       end
 
-      it 'should create a error' do
+      it 'creates a error' do
         expect(problems).to contain_error(single_space_msg).on_line(2).in_column(12)
       end
     end
@@ -33,21 +33,21 @@ describe 'manifest_whitespace_arrows_single_space_after' do
         PuppetLint.configuration.fix = false
       end
 
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(1).problem
       end
 
-      it 'should fix the manifest' do
+      it 'fixes the manifest' do
         expect(problems).to contain_fixed(single_space_msg)
       end
 
-      it 'should fix the space' do
+      it 'fixes the space' do
         expect(manifest).to eq(
-          <<~EOF,
+          <<~CODE,
             class { 'example2':
               param1 => 'value1',
             }
-          EOF
+          CODE
         )
       end
     end
@@ -55,19 +55,19 @@ describe 'manifest_whitespace_arrows_single_space_after' do
 
   context 'with no spaces' do
     let(:code) do
-      <<~EOF
+      <<~CODE
         class { 'example2':
           param1 =>'value1',
         }
-      EOF
+      CODE
     end
 
     context 'with fix disabled' do
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(1).problem
       end
 
-      it 'should create a error' do
+      it 'creates a error' do
         expect(problems).to contain_error(single_space_msg).on_line(2).in_column(12)
       end
     end
@@ -81,21 +81,21 @@ describe 'manifest_whitespace_arrows_single_space_after' do
         PuppetLint.configuration.fix = false
       end
 
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(1).problem
       end
 
-      it 'should fix the manifest' do
+      it 'fixes the manifest' do
         expect(problems).to contain_fixed(single_space_msg)
       end
 
-      it 'should fix the space' do
+      it 'fixes the space' do
         expect(manifest).to eq(
-          <<~EOF,
+          <<~CODE,
             class { 'example2':
               param1 => 'value1',
             }
-          EOF
+          CODE
         )
       end
     end
@@ -103,19 +103,19 @@ describe 'manifest_whitespace_arrows_single_space_after' do
 
   context 'with string as space' do
     let(:code) do
-      <<~EOF
+      <<~CODE
         class { 'example2':
           param1 =>' ',
         }
-      EOF
+      CODE
     end
 
     context 'with fix disabled' do
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(1).problem
       end
 
-      it 'should create a error' do
+      it 'creates a error' do
         expect(problems).to contain_error(single_space_msg).on_line(2).in_column(12)
       end
     end
@@ -129,21 +129,21 @@ describe 'manifest_whitespace_arrows_single_space_after' do
         PuppetLint.configuration.fix = false
       end
 
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(1).problem
       end
 
-      it 'should fix the manifest' do
+      it 'fixes the manifest' do
         expect(problems).to contain_fixed(single_space_msg)
       end
 
-      it 'should fix the space' do
+      it 'fixes the space' do
         expect(manifest).to eq(
-          <<~EOF,
+          <<~CODE,
             class { 'example2':
               param1 => ' ',
             }
-          EOF
+          CODE
         )
       end
     end
@@ -151,22 +151,22 @@ describe 'manifest_whitespace_arrows_single_space_after' do
 
   context 'many resources' do
     let(:code) do
-      <<~EOF
+      <<~CODE
         class { 'example2':
           param1 =>'value1',
           param2 => 'value2',
           param3 =>   'value3',
           param4 =>  'value4',
         }
-      EOF
+      CODE
     end
 
     context 'with fix disabled' do
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(3).problems
       end
 
-      it 'should create a error' do
+      it 'creates a error' do
         expect(problems).to contain_error(single_space_msg).on_line(2).in_column(12)
         expect(problems).to contain_error(single_space_msg).on_line(4).in_column(12)
         expect(problems).to contain_error(single_space_msg).on_line(5).in_column(12)
@@ -182,24 +182,24 @@ describe 'manifest_whitespace_arrows_single_space_after' do
         PuppetLint.configuration.fix = false
       end
 
-      it 'should detect a single problem' do
+      it 'detects a single problem' do
         expect(problems).to have(3).problem
       end
 
-      it 'should fix the manifest' do
+      it 'fixes the manifest' do
         expect(problems).to contain_fixed(single_space_msg)
       end
 
-      it 'should fix the space' do
+      it 'fixes the space' do
         expect(manifest).to eq(
-          <<~EOF,
+          <<~CODE,
             class { 'example2':
               param1 => 'value1',
               param2 => 'value2',
               param3 => 'value3',
               param4 => 'value4',
             }
-          EOF
+          CODE
         )
       end
     end
@@ -208,40 +208,40 @@ describe 'manifest_whitespace_arrows_single_space_after' do
   context 'valid cases' do
     context 'correct arrow' do
       let(:code) do
-        <<~EOF
+        <<~CODE
           class { 'example2':
             param1 => 'value1',
           }
-        EOF
+        CODE
       end
 
-      it 'should detect a no problem' do
+      it 'detects a no problem' do
         expect(problems).to be_empty
       end
     end
 
     context 'in a string' do
       let(:code) do
-        <<~EOF
+        <<~CODE
           "param1 =>    'value1'",
-        EOF
+        CODE
       end
 
-      it 'should detect a no problem' do
+      it 'detects a no problem' do
         expect(problems).to be_empty
       end
     end
 
     context 'in a heredoc' do
       let(:code) do
-        <<~EOF
+        <<~CODE
           $data = @("DATA"/L)
             param1 =>    'value1',
             | DATA
-        EOF
+        CODE
       end
 
-      it 'should detect a no problem' do
+      it 'detects a no problem' do
         expect(problems).to be_empty
       end
     end