checkParamNames: fixup buggy behaviour

jscs-dev · Dec 1, 2014 · 4ec300a · 4ec300a
1 parent cb5c2a5
commit 4ec300a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/lib/rules/validate-jsdoc/check-param-names.js b/lib/rules/validate-jsdoc/check-param-names.js
@@ -46,7 +46,7 @@ function validateCheckParamNames(node, err) {
                 return;
             }
 
-            if (tag.value.indexOf('.') !== -1) {
+            if (tag.name.value.indexOf('.') !== -1) {
                 skipped++;
                 return;
             }

diff --git a/test/lib/rules/validate-jsdoc/check-param-names.js b/test/lib/rules/validate-jsdoc/check-param-names.js
@@ -266,10 +266,18 @@ describe('lib/rules/validate-jsdoc/check-param-names', function() {
                      * @param {String|Boolean|Array} [mod.modVal]
                      * @param {Object} props
                      * @param {Object} [staticProps]
-                     * @returns {Function}
                      */
                     function yeah(mod, props, staticProps) {}
                 }
+            }, {
+                it: 'should not report wrong order',
+                code: function() {
+                    /**
+                     * @param {string|Array.<string>} types
+                     * @param {function(this: DocComment, DocTag): DocComment} fn
+                     */
+                    function iterateByTypes(types, fn) {}
+                }
             }
             /* jscs: enable */
             /* jshint ignore:end */
-Original file line number
+Diff line change
@@ Expand Up / @@ -46,7 +46,7 @@ function validateCheckParamNames(node, err) { @@
                     return;
                 }
-                if (tag.value.indexOf('.') !== -1) {
+                if (tag.name.value.indexOf('.') !== -1) {
                     skipped++;
                     return;
                 }
@@ Expand Down @@