fix(require-jsdoc): proper reporting of end line numbers; fixes #843

gajus · Apr 20, 2022 · 4270b75 · 4270b75
1 parent 7db1920
commit 4270b75
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/rules/requireJsdoc.js b/src/rules/requireJsdoc.js
@@ -320,8 +320,13 @@ export default {
       };
 
       const report = () => {
+        const {
+          start,
+        } = node.loc;
         const loc = {
-          end: node.loc.start + 1,
+          end: {
+            line: start.line + 1,
+          },
           start: node.loc.start,
         };
         context.report({

diff --git a/test/rules/assertions/requireJsdoc.js b/test/rules/assertions/requireJsdoc.js
@@ -226,7 +226,8 @@ function quux (foo) {
 }`,
       errors: [
         {
-          endLine: undefined,
+          column: 1,
+          endLine: 3,
           line: 2,
           message: 'Missing JSDoc comment.',
         },
@@ -699,7 +700,7 @@ function quux (foo) {
           }`,
       errors: [
         {
-          endLine: undefined,
+          endLine: 3,
           line: 2,
           message: 'Missing JSDoc comment.',
         },