fixed possible throw with invalid tag location

Fixes #86
jscs-dev · Mar 11, 2015 · c56322a · c56322a
1 parent 55b463e
commit c56322a
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/lib/rules/validate-jsdoc/require-param-types.js b/lib/rules/validate-jsdoc/require-param-types.js
@@ -14,6 +14,6 @@ module.exports.options = {
 function validateParamTag(node, tag, err) {
     // checking existance
     if (!tag.type) {
-        return err('missing param type');
+        return err('Missing param type', (tag.name ? tag.name.loc : null) || tag.loc);
     }
 }
diff --git a/test/lib/rules/validate-jsdoc/require-param-types.js b/test/lib/rules/validate-jsdoc/require-param-types.js
@@ -101,6 +101,20 @@ describe('lib/rules/validate-jsdoc/require-param-types', function () {
                         }
                     };
                 }
+            }, {
+                // #86
+                it: 'should report with right location',
+                code: function () {
+                    Cls.prototype = {
+                        /**
+                         * @param {} xxx
+                         */
+                        run: function(xxx) {
+                            // dummy
+                        }
+                    };
+                },
+                errors: {column: 17, line: 3}
             }
             /* jshint ignore:end */
         ]);