Fix: add support of es6 exports in enforceExistence

Fixes #159
Closes gh-160

Author: Alexej Yaroshevich

lib/rules/validate-jsdoc.js

@@ -229,7 +229,8 @@ function patchNodesInFile(file) {
      */
     function getJsdoc() {
         if (!this.hasOwnProperty('_jsdoc')) {
-            var node = this.type === 'FunctionExpression' ? findFirstNodeInLine(this) : this;
+            var node = this.type === 'FunctionExpression' || this.type === 'FunctionDeclaration' ?
+                findFirstNodeInLine(this) : this;
             var res = findDocCommentBeforeNode(node);
             this._jsdoc = res ? jsdoc.createDocCommentByCommentNode(res) : null;
         }

lib/rules/validate-jsdoc/enforce-existence.js

@@ -72,11 +72,14 @@ function enforceExistence(node, err) {
         }
     }
     if (policy.exports === false) {
-        if (parentNode.type === 'AssignmentExpression') {
+        if (parentNode.type === 'ExportDefaultDeclaration') {
+            // don't check es6 export
+            return;
+        } else if (parentNode.type === 'AssignmentExpression') {
             var left = parentNode.left;
             if ((left.object && left.object.name) === 'module' &&
                 (left.property && left.property.name) === 'exports') {
-                // don't check exports functions
+                // don't check commonjs exports format
                 return;
             }
         }

test/init.js

@@ -70,9 +70,11 @@ function rulesChecker(opts) {
          *
          * @param {Object} rules
          */
-        rules: function(rules) {
+        rules: function(rules, opts) {
             beforeEach(function() {
-                checker.configure({jsDoc: rules});
+                opts = opts || {};
+                opts.jsDoc = rules;
+                checker.configure(opts);
             });
         },
 
@@ -109,7 +111,9 @@ function rulesChecker(opts) {
 
                     if (!test.hasOwnProperty('errors') || (typeof test.errors === 'number')) {
                         expect(checked.getErrorCount())
-                            .to.eq(test.errors || 0);
+                            .to.eq(test.errors || 0, test.errors ?
+                                'Expect ' + test.errors + ' error(s)'
+                                : 'Unexpected error(s)');
                     } else if (Array.isArray(test.errors)) {
                         expect(errors)
                             .to.containSubset(test.errors);

test/lib/rules/validate-jsdoc/enforce-existence.js

@@ -147,6 +147,33 @@ describe('lib/rules/validate-jsdoc/enforce-existence', function () {
 
     });
 
+    describe('with true and esnext', function() {
+        checker.rules({enforceExistence: true}, {esnext: true});
+
+        checker.cases([
+            /* jshint ignore:start */
+            {
+                it: 'should report jsdoc absence for named function export (#159)',
+                code: [
+                    'export default function named (v) {',
+                    '};',
+                ].join('\n'),
+                errors: 1,
+            }, {
+                it: 'should not report jsdoc absence for named function export (#159)',
+                code: [
+                    '/**',
+                    ' * Foo bar',
+                    ' */',
+                    'export default function named (v) {',
+                    '};',
+                ].join('\n'),
+                errors: 0,
+            },
+            /* jshint ignore:end */
+        ]);
+    });
+
     describe('with exceptExports', function() {
         checker.rules({enforceExistence: 'exceptExports'});
 
@@ -163,6 +190,23 @@ describe('lib/rules/validate-jsdoc/enforce-existence', function () {
         ]);
     });
 
+    describe('with exceptExports and esnext', function() {
+        checker.rules({enforceExistence: 'exceptExports'}, {esnext: true});
+
+        checker.cases([
+            /* jshint ignore:start */
+            {
+                it: 'should not report jsdoc absence for named function export (#159)',
+                code: [
+                    'export default function named (v) {',
+                    '};',
+                ].join('\n'),
+                errors: 0,
+            }
+            /* jshint ignore:end */
+        ]);
+    });
+
     describe('with allExcept exports', function() {
         checker.rules({enforceExistence: {allExcept: ['exports']}});