puppet-lint-param_comment-check 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc1677e0675072c3a48173655360a1daca9c0539ff2a58bd4f71a7c05f38e37b
-  data.tar.gz: af924c9a20e1957f117538c80cc9a7fa3aa89dcdfafbb2d9b85b26990855e851
+  metadata.gz: 760c2ecbb41952244ece53d7fddafcc91f322300b512a65dfc4d5ab7b5622636
+  data.tar.gz: 187d476c9e3418c03935d00ddf84fd9c9618cd8ac4a14a391e0046f68f92221f
 SHA512:
-  metadata.gz: 68b99e9b40064843a160dfd7e44d81962ed2ee4fd0246b5a623cedc1cb632b9cbbb97645ae3c8ac4caf2d9236fd694dbbccabf56d49a1b12074e8618e69f7e76
-  data.tar.gz: f466b51ef27e5bcd8edff95fb434706c04b00351b3d6616913d9a5ee3d91004fc8b15614c50a8deb1e9cbeb14b3c67d45acb4d6b061bd7e80bc556e2d4996adf
+  metadata.gz: 8396f55b48c7cd1e82f66855eb950c2c1fd4e908423625a262e6ba11496d2cd3b9c90931c66deb97efba5d706502416e71ef4ae47f7d459b5c0fcf24d7b86ef4
+  data.tar.gz: '08c1d478a70c0a204805844aa0ad1724ee7a371852c3f7059596204d3c62f89d9044fa2cf44b26b6df2731331f371765b3cdf86de148ecf5cab88216336dea93'

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

@@ -51,9 +51,9 @@ def analyze_params(param_tokens) # rubocop:disable Metrics/AbcSize,Metrics/Cyclo
   params = []
   current_param = EMPTY_PARAM.dup
   brackets = 0
-  param_tokens.reject { |token| %i[WHITESPACE NEWLINE].include? token.type }.each do |token|
-    brackets += 1 if token.type == :LBRACK
-    brackets -= 1 if token.type == :RBRACK
+  param_tokens.reject { |token| %i[WHITESPACE NEWLINE INDENT].include? token.type }.each do |token|
+    brackets += 1 if %i[LBRACK LBRACE].include? token.type
+    brackets -= 1 if %i[RBRACK RBRACE].include? token.type
     next unless brackets.zero?
 
     current_param = analyze_param_token(token, current_param) unless token.type == :COMMA
@@ -62,7 +62,7 @@ def analyze_params(param_tokens) # rubocop:disable Metrics/AbcSize,Metrics/Cyclo
       current_param = EMPTY_PARAM.dup
     end
   end
-  params.append(current_param) unless current_param == EMPTY_PARAM
+  params.append(current_param) unless current_param[:name] == ''
   params
 end
 
@@ -135,9 +135,9 @@ PuppetLint.new_check(:param_comment) do # rubocop:disable Metrics/BlockLength
   def check_comment_order(params)
     param_comments = @comment_engine.params
     param_comments.each_with_index do |param, index|
-      return false unless param[:name] == params[index][:name]
+      return param[:line] unless param[:name] == params[index][:name]
     end
-    true
+    -1
   end
 
   # Check class or defined type indexes
@@ -147,7 +147,9 @@ PuppetLint.new_check(:param_comment) do # rubocop:disable Metrics/BlockLength
       params = analyze_params(index[:param_tokens])
       return false unless check_comments(comments)
       return false unless check_parameters_count(params)
-      return warn('Parameters sorted wrong') unless check_comment_order(params)
+
+      comment_order_line = check_comment_order(params)
+      return warn('Parameters sorted wrong', comment_order_line) unless comment_order_line == -1
     end
     true
   end

data/lib/puppet-lint-param_comment-check/param_comments.rb

@@ -4,28 +4,30 @@
 EMPTY_PARAM_COMMENT = {
   name: '',
   description: '',
-  options: []
+  options: [],
+  line: -1
 }.freeze
 
 # The empty data of a hash option
 EMPTY_OPTION_COMMENT = {
   name: '',
   type: '',
-  description: ''
+  description: '',
+  line: -1
 }.freeze
 
 # A regular expression describing a parameter header
 REGEXP_PARAM_HEADER = /^@param (?<name>[^ ]+)$/.freeze
 
 # A regular expression describing a hash option header
-REGEXP_OPTION_HEADER = /^@option (?<hash_name>[^ ]+) \[(?<type>[^\]]+)\] :(?<name>[^ ]+)$/.freeze
+REGEXP_OPTION_HEADER = /^@option (?<hash_name>[^ ]+) \[(?<type>.+)\] :(?<name>[^ ]+)$/.freeze
 
 # Comment received in an invalid state
 class InvalidCommentForState < StandardError
   def initialize(comment, state)
     @comment = comment
     @state = state
-    super "Invalid state #{@state} for comment #{@comment.value.strip}"
+    super "Can not process the comment '#{@comment.value.strip}' in the state #{@state}"
   end
 
   attr_reader :comment
@@ -66,7 +68,7 @@ class ParamComments
   # Walk through every comment and transition the workflow fsm accordingly
   #
   # @param comments A list of Comment tokens appearing before the class/defined type header
-  def process(comments) # rubocop:disable Metrics/MethodLength
+  def process(comments) # rubocop:disable Metrics/MethodLength,Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
     @current_comment = PuppetLint::Lexer::Token.new(:COMMENT, '', 1, 1)
     comments.each do |comment|
       @current_comment = comment
@@ -87,7 +89,7 @@ class ParamComments
   end
 
   # Called before the got_header event. Interpret the parameter header comment
-  def got_header_trigger(_, comment)
+  def got_header_trigger(_, comment) # rubocop:disable Metrics/AbcSize
     @params_have_started = true
     @current_param[:options].append(@current_option) if @in_option && !@current_option.nil?
     @params.append(@current_param) unless @current_param.nil?
@@ -96,6 +98,7 @@ class ParamComments
     @in_option = false
     comment.value.strip.match(REGEXP_PARAM_HEADER) do |match|
       @current_param[:name] = match.named_captures['name'].strip
+      @current_param[:line] = comment.line
     end
   end
 
@@ -120,6 +123,7 @@ class ParamComments
 
       @current_option[:name] = match.named_captures['name']
       @current_option[:type] = match.named_captures['type']
+      @current_param[:line] = comment.line
     end
   end
 

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

@@ -77,6 +77,50 @@ describe 'param_comment' do
     end
   end
 
+  context 'valid code with hash default' do
+    let(:code) do
+      <<~CODE
+        # @summary
+        #   some class
+        #
+        # @param hashparam
+        #   A hash
+        class my_class (
+            Hash $hashparam = {
+              somekey => "value"
+            }
+        ) {}
+      CODE
+    end
+
+    it 'should not detect any problems' do
+      expect(problems).to have(0).problems
+    end
+  end
+
+  context 'valid code with hash default with an enclosed hash' do
+    let(:code) do
+      <<~CODE
+        # @summary
+        #   some class
+        #
+        # @param hashparam
+        #   A hash
+        class my_class (
+            Hash $hashparam = {
+              'anotherhash' => {
+                somekey => "value"
+              },
+            },
+        ) {}
+      CODE
+    end
+
+    it 'should not detect any problems' do
+      expect(problems).to have(0).problems
+    end
+  end
+
   context 'code with missing parameter comment' do
     let(:code) do
       <<~CODE
@@ -130,15 +174,14 @@ describe 'param_comment' do
   context 'code with wrongly sorted parameter comments' do
     let(:code) do
       <<~CODE
-        # @param withdefault
-        #   A parameter with a default value
-        #
         # @param mandatory
         #   A mandatory parameter
         #
         # @param optional
         #   An optional parameter
-
+        #
+        # @param withdefault
+        #   A parameter with a default value
         class my_class (
             String $mandatory,
             Boolean $withdefault = false,
@@ -152,7 +195,7 @@ describe 'param_comment' do
     end
 
     it 'should create a warning' do
-      expect(problems).to contain_warning('Parameters sorted wrong').on_line(1).in_column(1)
+      expect(problems).to contain_warning('Parameters sorted wrong').on_line(4).in_column(1)
     end
   end
 
@@ -180,7 +223,7 @@ describe 'param_comment' do
     end
 
     it 'should create a warning' do
-      expect(problems).to contain_warning('Invalid state awaiting_separator for comment @param withdefault')
+      expect(problems).to contain_warning('Can not process the comment \'@param withdefault\' in the state awaiting_separator')
                             .on_line(3)
                             .in_column(1)
     end
@@ -272,7 +315,7 @@ describe 'param_comment' do
 
     it 'should create a warning' do
       expect(problems).to contain_warning(
-                            'Invalid state awaiting_header for comment @option mandatory [Boolean] :some_option'
+                            'Can not process the comment \'@option mandatory [Boolean] :some_option\' in the state awaiting_header'
                           )
                             .on_line(4)
                             .in_column(1)
@@ -326,10 +369,28 @@ describe 'param_comment' do
 
     it 'should create a warning' do
       expect(problems).to contain_warning(
-                            'Invalid state awaiting_separator for comment @param second'
+                            'Can not process the comment \'@param second\' in the state awaiting_separator'
                           )
                             .on_line(5)
                             .in_column(1)
     end
   end
+
+  context 'valid code with complex hash option type' do
+    let(:code) do
+      <<~CODE
+        # @param mandatory
+        #   A mandatory parameter
+        # @option mandatory [Optional[String]] :some_option
+        #   An option
+        class my_class (
+            Hash $mandatory,
+        ) {}
+      CODE
+    end
+
+    it 'should detect exactly one problem' do
+      expect(problems).to have(0).problems
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-param_comment-check
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Dennis Ploeger