puppet-lint-manifest_whitespace-check 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73d348cb4f0a000a02c631f97562527a401f7e61f2954e31e8841c9095b32a8f
-  data.tar.gz: 7fbdd7c452078510f2ae545a93f492f3effd982775cfce706ed7d41219eaa282
+  metadata.gz: 2b3bc8cb98f4459ad929e160c0759c73e76fc6253c54061b4edea16370f8b38d
+  data.tar.gz: ddc62ccbc2c3537f6057c73d498a035dc4007f416a8b435140a207aa8c0380f5
 SHA512:
-  metadata.gz: 1e5df8707c5cbbe5a05bac3b1f16542e1df5c02cc41f4ca5a4d2e97eda2508bb5aa25601bbce7e04a3f06c6803a96dbd69752cd43be33d6f7dd69f23ff1c54dc
-  data.tar.gz: c0c670b007c1456d68a59635e3e29da5ffbc9bdc0e3ff3c0cc1623766180665ef2180b3fff0b584b7ae9bee29243e618b92a7b9030880c3d965b090ac802a790
+  metadata.gz: 59189f5b0b1922bfd04ef96b64f53662b5740ef3a20d8a34f44045fa90d194a4c94e41ca6440ada5b0ecb8b8ec3bb27631b85c02e61f59b4844a708665e417a2
+  data.tar.gz: 3547dfb1ab41289ef4ad92168bbc02a12b0ae717b9e97146c0fc2cb13314a972cc1d1c191c0d4feaf73ac7604deb7b272ac90616693f980d1be224a2d498ec26

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

@@ -9,19 +9,13 @@ PuppetLint.new_check(:manifest_whitespace_closing_brace_before) do
       next unless prev_token && prev_code_token
       next if %i[LBRACE].include?(prev_token.type)
 
-      unless %i[LBRACE].include?(prev_code_token.type)
-        if is_single_space(prev_token) && tokens.index(prev_code_token) == tokens.index(brace_token) - 2
-          next
-        end
+      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
 
-      if prev_token.type == :INDENT
-        next if tokens.index(prev_code_token) == tokens.index(brace_token) - 3
-      end
+      next if prev_token.type == :INDENT && (tokens.index(prev_code_token) == tokens.index(brace_token) - 3)
 
-      if prev_token.type == :NEWLINE
-        next if tokens.index(prev_code_token) == tokens.index(brace_token) - 2
-      end
+      next if prev_token.type == :NEWLINE && (tokens.index(prev_code_token) == tokens.index(brace_token) - 2)
 
       notify(
         :error,
@@ -39,13 +33,9 @@ PuppetLint.new_check(:manifest_whitespace_closing_brace_before) do
     next_token = token
 
     until next_token.type == :RBRACE
-      if tokens[tokens.index(next_token)..-1].first(2).collect(&:type) == %i[NEWLINE RBRACE]
-        break
-      end
+      break if tokens[tokens.index(next_token)..-1].first(2).collect(&:type) == %i[NEWLINE RBRACE]
 
-      if tokens[tokens.index(next_token)..-1].first(3).collect(&:type) == %i[NEWLINE INDENT RBRACE]
-        break
-      end
+      break if tokens[tokens.index(next_token)..-1].first(3).collect(&:type) == %i[NEWLINE INDENT RBRACE]
 
       remove_token(next_token)
       next_token = next_token.next_token

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

@@ -11,12 +11,8 @@ PuppetLint.new_check(:manifest_whitespace_closing_bracket_before) do
 
       next unless %i[NEWLINE INDENT WHITESPACE].include?(prev_token.type)
 
-      if prev_token.type == :INDENT
-        next if tokens.index(prev_code_token) == tokens.index(bracket_token) - 3
-      end
-      if prev_token.type == :NEWLINE
-        next if tokens.index(prev_code_token) == tokens.index(bracket_token) - 2
-      end
+      next if prev_token.type == :INDENT && (tokens.index(prev_code_token) == tokens.index(bracket_token) - 3)
+      next if prev_token.type == :NEWLINE && (tokens.index(prev_code_token) == tokens.index(bracket_token) - 2)
 
       notify(
         :error,
@@ -37,9 +33,7 @@ PuppetLint.new_check(:manifest_whitespace_closing_bracket_before) do
 
     next_token = token.next_token
     until next_token.type == :RBRACK
-      if next_token.type == :INDENT && next_token.next_token.type == :RBRACK
-        break
-      end
+      break if next_token.type == :INDENT && next_token.next_token.type == :RBRACK
 
       remove_token(next_token)
       next_token = next_token.next_token

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

@@ -7,11 +7,17 @@ PuppetLint.new_check(:manifest_whitespace_opening_brace_before) do
       prev_code_token = prev_non_space_token(brace_token)
 
       next unless prev_token && prev_code_token
-      if %i[LPAREN LBRACK LBRACE COLON COMMA COMMENT].include?(prev_code_token.type)
-        next
+
+      next if %i[COLON].include?(prev_code_token.type)
+
+      if %i[COMMENT COMMA 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)
       end
-      next unless tokens.index(prev_code_token) != tokens.index(brace_token) - 2 ||
-                  !is_single_space(prev_token)
 
       notify(
         :error,
@@ -29,9 +35,7 @@ PuppetLint.new_check(:manifest_whitespace_opening_brace_before) do
     prev_code_token = prev_non_space_token(token)
 
     while prev_code_token != prev_token
-      unless %i[WHITESPACE INDENT NEWLINE].include?(prev_token.type)
-        raise PuppetLint::NoFix
-      end
+      raise PuppetLint::NoFix unless %i[WHITESPACE INDENT NEWLINE].include?(prev_token.type)
 
       remove_token(prev_token)
       prev_token = prev_token.prev_token

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

@@ -7,14 +7,10 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
       prev_code_token = prev_non_space_token(bracket_token)
 
       next unless prev_token && prev_code_token
-      if %i[LBRACK LBRACE COMMA SEMIC COMMENT].include?(prev_code_token.type)
-        next
-      end
+      next if %i[LBRACK LBRACE COMMA SEMIC COMMENT NAME].include?(prev_code_token.type)
       next unless %i[WHITESPACE NEWLINE INDENT].include?(prev_token.type)
 
-      if %i[INDENT NEWLINE].include?(prev_token.type) && %i[RBRACK RBRACE].include?(prev_code_token.type)
-        next
-      end
+      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)
 
@@ -34,9 +30,7 @@ PuppetLint.new_check(:manifest_whitespace_opening_bracket_before) do
     prev_code_token = prev_non_space_token(token)
 
     while prev_code_token != prev_token
-      unless %i[WHITESPACE INDENT NEWLINE].include?(prev_token.type)
-        raise PuppetLint::NoFix
-      end
+      raise PuppetLint::NoFix unless %i[WHITESPACE INDENT NEWLINE].include?(prev_token.type)
 
       remove_token(prev_token)
       prev_token = prev_token.prev_token

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

@@ -3,7 +3,9 @@
 require 'spec_helper'
 
 describe 'manifest_whitespace_class_name_single_space_before' do
-  let(:single_space_msg) { 'there should be a single space between the class or defined resource statement and the name' }
+  let(:single_space_msg) {
+    'there should be a single space between the class or defined resource statement and the name'
+  }
 
   context 'with two spaces' do
     let(:code) do

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

@@ -402,7 +402,9 @@ describe 'manifest_whitespace_closing_brace_before' do
 end
 
 describe 'manifest_whitespace_closing_brace_after' do
-  let(:closing_brace_msg) { 'there should be either a bracket, punctuation mark, closing quote or a newline after a closing brace, or whitespace and none of the aforementioned' }
+  let(:closing_brace_msg) do
+    'there should be either a bracket, punctuation mark, closing quote or a newline after a closing brace, or whitespace and none of the aforementioned'
+  end
 
   context 'with iterator' do
     let(:code) do
@@ -417,6 +419,18 @@ describe 'manifest_whitespace_closing_brace_after' do
     end
   end
 
+  context 'inline with a function after' do
+    let(:code) do
+      <<~EOF
+        Hash({ $key => $return_value } )
+      EOF
+    end
+
+    it 'should detect 1 problem' do
+      expect(problems).to have(1).problem
+    end
+  end
+
   context 'with spaces' do
     let(:code) do
       <<~EOF

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

@@ -153,7 +153,9 @@ describe 'manifest_whitespace_closing_bracket_before' do
 end
 
 describe 'manifest_whitespace_closing_bracket_after' do
-  let(:closing_bracket_msg) { 'there should be either a bracket, punctuation mark, closing quote or a newline after a closing bracket, or whitespace and none of the aforementioned' }
+  let(:closing_bracket_msg) {
+    'there should be either a bracket, punctuation mark, closing quote or a newline after a closing bracket, or whitespace and none of the aforementioned'
+  }
 
   context 'with many brackets' do
     let(:code) do

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

@@ -28,8 +28,8 @@ describe 'manifest_whitespace_double_newline_end_of_file' do
     context 'with 1 empty line at the end of a manifest' do
       let(:code) do
         <<~EOF
-        class example {
-        }
+          class example {
+          }
 
         EOF
       end
@@ -46,8 +46,8 @@ describe 'manifest_whitespace_double_newline_end_of_file' do
     context 'with 3 empty lines at the end of a manifest' do
       let(:code) do
         <<~EOF
-        class example {
-        }
+          class example {
+          }
 
 
 
@@ -78,8 +78,8 @@ describe 'manifest_whitespace_double_newline_end_of_file' do
     context 'with 1 empty line at the end of a manifest' do
       let(:code) do
         <<~EOF
-        class example {
-        }
+          class example {
+          }
 
         EOF
       end
@@ -95,8 +95,8 @@ describe 'manifest_whitespace_double_newline_end_of_file' do
       it 'should add the final newline' do
         expect(manifest).to eq(
           <<~EOF
-          class example {
-          }
+            class example {
+            }
           EOF
         )
       end
@@ -105,8 +105,8 @@ describe 'manifest_whitespace_double_newline_end_of_file' do
     context 'with 3 empty lines at the end of a manifest' do
       let(:code) do
         <<~EOF
-        class example {
-        }
+          class example {
+          }
 
 
 
@@ -126,8 +126,8 @@ describe 'manifest_whitespace_double_newline_end_of_file' do
       it 'should add the final newline' do
         expect(manifest).to eq(
           <<~EOF
-          class example {
-          }
+            class example {
+            }
           EOF
         )
       end

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

@@ -19,6 +19,42 @@ describe 'manifest_whitespace_opening_brace_before' do
     end
   end
 
+  context 'inside interpolation' do
+    let(:code) do
+      <<~EOF
+        my_define { "foo-${myvar}": }
+      EOF
+    end
+
+    it 'should detect no problems' do
+      expect(problems).to be_empty
+    end
+  end
+
+  context 'inline with a function before' do
+    let(:code) do
+      <<~EOF
+        Hash( { $key => $return_value })
+      EOF
+    end
+
+    it 'should detect 1 problem' do
+      expect(problems).to have(1).problem
+    end
+  end
+
+  context 'inline with a function' do
+    let(:code) do
+      <<~EOF
+        Hash({ $key => $return_value })
+      EOF
+    end
+
+    it 'should detect no problems' do
+      expect(problems).to be_empty
+    end
+  end
+
   context 'inside a function' do
     let(:code) do
       <<~EOF

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

@@ -51,7 +51,6 @@ describe 'manifest_whitespace_opening_bracket_before' do
     end
 
     it 'should detect no problems' do
-      pending 'Fix the linter'
       expect(problems).to be_empty
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-manifest_whitespace-check
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Jo Vandeginste
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-06 00:00:00.000000000 Z
+date: 2022-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: puppet-lint
@@ -151,7 +151,7 @@ licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -167,7 +167,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.3.7
-signing_key: 
+signing_key:
 specification_version: 4
 summary: A puppet-lint check to validate whitespace in manifests
 test_files: []