Refactor not implemented option warning

Report an error through eslint instead of throw.
gajus · Sep 13, 2020 · 38b8b9d · 38b8b9d
1 parent 771d550
commit 38b8b9d
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/src/rules/checkLinesAlignment.js b/src/rules/checkLinesAlignment.js
@@ -178,7 +178,9 @@ export default iterateJsdoc(({
   indent,
 }) => {
   if (context.options[0] === 'never') {
-    throw new Error('The `never` option is not yet implemented for this rule.');
+    report('The `never` option is not yet implemented for this rule.');
+
+    return;
   }
 
   if (context.options[0] !== 'always') {

diff --git a/test/rules/assertions/checkLinesAlignment.js b/test/rules/assertions/checkLinesAlignment.js
@@ -364,6 +364,26 @@ export default {
          */
       `,
     },
+    {
+      code: `
+        /**
+         * Not implemented yet.
+         *
+         * @param {string} lorem Description.
+         * @param {int} sit Description multi words.
+         */
+        const fn = ( lorem, sit ) => {}
+      `,
+      errors: [
+        {
+          message: 'The `never` option is not yet implemented for this rule.',
+          type: 'Block',
+        },
+      ],
+      options: [
+        'never',
+      ],
+    },
   ],
   valid: [
     {
@@ -533,7 +553,11 @@ export default {
       code: `
         /**
          * Not validating without option.
+         *
+         * @param {string} lorem Description.
+         * @param {int} sit Description multi words.
          */
+        const fn = ( lorem, sit ) => {}
       `,
     },
   ],