checkRedundantReturns: skip reporting for abstract declarations

Fixes #93

Author: Alexej Yaroshevich

lib/jsdoc.js

@@ -67,6 +67,11 @@ function DocComment(value, loc) {
         return new DocTag(tag, new DocLocation(tag.line, 3, loc.start));
     });
 
+    // calculate abstract flag
+    this.abstract = this.tags.some(function(v) {
+        return v.id === 'abstract' || v.id === 'virtual';
+    });
+
     /**
      * @param {function(this: DocComment, DocTag): DocComment} fn
      * @returns {DocComment}

lib/rules/validate-jsdoc/check-redundant-returns.js

@@ -17,6 +17,11 @@ function checkReturnTypes(node, tag, err) {
         return;
     }
 
+    // skip abstract methods
+    if (node.jsdoc.abstract) {
+        return;
+    }
+
     // checking redundant: invalid or not return statements in code
     var redundant = !Boolean(this._getReturnStatementsForNode(node).length);
 

test/lib/rules/validate-jsdoc/check-redundant-returns.js

@@ -79,6 +79,15 @@ describe('lib/rules/validate-jsdoc/check-redundant-returns', function () {
                         });
                     }
                 }
+            }, {
+                it: 'should not report for abstract methods',
+                code: function () {
+                    /**
+                     * @abstract
+                     * @return {number}
+                     */
+                    function onEvent () {}
+                }
             }
             /* jshint ignore:end */
         ]);