enforce rule should ignore anonymous functions

Alexej Yaroshevich · Alexej Yaroshevich · commit dd0873eba1b6 · 2014-09-08T00:16:43.000+04:00
diff --git a/lib/rules/validate-jsdoc/enforce-existence.js b/lib/rules/validate-jsdoc/enforce-existence.js
@@ -10,7 +10,7 @@ module.exports.options = {
  * @param {Function} err
  */
 function enforceExistence(node, err) {
-    if (!node.jsDoc) {
+    if (!node.jsDoc && node.id) {
         err('jsdoc definition required', node.loc.start);
     }
 }
diff --git a/test/lib/rules/validate-jsdoc/enforce-existence.js b/test/lib/rules/validate-jsdoc/enforce-existence.js
@@ -36,6 +36,12 @@ describe('rules/validate-jsdoc', function () {
                     function funcName(p) {
                     }
                 }
+            }, {
+                it: 'should not report jsdoc absence for anonymous functions',
+                code: function () {
+                    [].forEach(function (v) {
+                    });
+                }
             }
             /* jshint ignore:end */
         ]);

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ module.exports.options = {`
`10`	`10`	`* @param {Function} err`
`11`	`11`	`*/`
`12`	`12`	`function enforceExistence(node, err) {`
`13`		`- if (!node.jsDoc) {`
	`13`	`+ if (!node.jsDoc && node.id) {`
`14`	`14`	`err('jsdoc definition required', node.loc.start);`
`15`	`15`	`}`
`16`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,12 @@ describe('rules/validate-jsdoc', function () {`
`36`	`36`	`function funcName(p) {`
`37`	`37`	`}`
`38`	`38`	`}`
	`39`	`+ }, {`
	`40`	`+ it: 'should not report jsdoc absence for anonymous functions',`
	`41`	`+ code: function () {`
	`42`	`+ [].forEach(function (v) {`
	`43`	`+ });`
	`44`	`+ }`
`39`	`45`	`}`
`40`	`46`	`/* jshint ignore:end */`
`41`	`47`	`]);`