Add rule documentation URLs and schemas (and eslint-plugin-eslint-plugin)

.eslintrc.json

@@ -1,6 +1,13 @@
 {
-    "extends": "holidaycheck/es2015",
-    "plugins": [ "node", "unicorn" ],
+    "extends": [
+        "holidaycheck/es2015",
+        "plugin:eslint-plugin/recommended"
+    ],
+    "plugins": [
+        "eslint-plugin",
+        "node",
+        "unicorn"
+    ],
 
     "env": {
         "node": true
@@ -12,6 +19,11 @@
     },
 
     "rules": {
+        "eslint-plugin/require-meta-docs-url": [
+            "error",
+            { "pattern": "https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/{{name}}.md" }
+        ],
+
         "node/no-unsupported-features": "error",
         "unicorn/prefer-includes": "error"
     },

lib/rules/handle-done-callback.js

@@ -7,7 +7,8 @@ module.exports = {
     meta: {
         type: 'problem',
         docs: {
-            description: 'Enforces handling of callbacks for async tests'
+            description: 'Enforces handling of callbacks for async tests',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/handle-done-callback.md'
         },
         schema: [
             {
@@ -38,7 +39,7 @@ module.exports = {
         }
 
         function isReferenceHandled(reference) {
-            const parent = context.getNodeByRangeIndex(reference.identifier.range[0]).parent;
+            const parent = context.getSourceCode().getNodeByRangeIndex(reference.identifier.range[0]).parent;
 
             return parent.type === 'CallExpression';
         }
@@ -54,7 +55,10 @@ module.exports = {
             const callbackVariable = findParamInScope(callbackName, scope);
 
             if (callbackVariable && !hasHandledReferences(callbackVariable.references)) {
-                context.report(callback, 'Expected "{{name}}" callback to be handled.', { name: callbackName });
+                context.report({
+                    node: callback,
+                    message: 'Expected "{{name}}" callback to be handled.', data: { name: callbackName }
+                });
             }
         }
 

lib/rules/max-top-level-suites.js

@@ -18,7 +18,8 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Limit the number of top-level suites in a single file'
+            description: 'Limit the number of top-level suites in a single file',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/max-top-level-suites.md'
         },
         schema: [
             {

lib/rules/no-async-describe.js

@@ -12,9 +12,11 @@ module.exports = {
     meta: {
         type: 'problem',
         docs: {
-            description: 'Disallow async functions passed to describe'
+            description: 'Disallow async functions passed to describe',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-async-describe.md'
         },
-        fixable: 'code'
+        fixable: 'code',
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-exclusive-tests.js

@@ -6,8 +6,10 @@ module.exports = {
     meta: {
         type: 'problem',
         docs: {
-            description: 'Disallow exclusive tests'
-        }
+            description: 'Disallow exclusive tests',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-exclusive-tests.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-exports.js

@@ -5,12 +5,14 @@ const createAstUtils = require('../util/ast');
 module.exports = {
     meta: {
         docs: {
-            description: 'Disallow exports from test files'
+            description: 'Disallow exports from test files',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-exports.md'
         },
         messages: {
             unexpectedExport: 'Unexpected export from a test file'
         },
-        type: 'suggestion'
+        type: 'suggestion',
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-global-tests.js

@@ -6,8 +6,10 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow global tests'
-        }
+            description: 'Disallow global tests',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-global-tests.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);
@@ -22,7 +24,7 @@ module.exports = {
                 const scope = context.getScope();
 
                 if (astUtils.isTestCase(node) && isGlobalScope(scope)) {
-                    context.report(callee, 'Unexpected global mocha test.');
+                    context.report({ node: callee, message: 'Unexpected global mocha test.' });
                 }
             }
         };

lib/rules/no-hooks-for-single-case.js

@@ -14,7 +14,8 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow hooks for a single test or test suite'
+            description: 'Disallow hooks for a single test or test suite',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-hooks-for-single-case.md'
         },
         schema: [
             {

lib/rules/no-hooks.js

@@ -6,7 +6,8 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow hooks'
+            description: 'Disallow hooks',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-hooks.md'
         },
         schema: [
             {

lib/rules/no-identical-title.js

@@ -21,8 +21,10 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow identical titles'
-        }
+            description: 'Disallow identical titles',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-identical-title.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-mocha-arrows.js

@@ -75,9 +75,11 @@ module.exports = {
         type: 'suggestion',
         docs: {
             description:
-                'Disallow arrow functions as arguments to mocha functions'
+                'Disallow arrow functions as arguments to mocha functions',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-mocha-arrows.md'
         },
-        fixable: 'code'
+        fixable: 'code',
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-nested-tests.js

@@ -8,8 +8,10 @@ module.exports = {
     meta: {
         type: 'problem',
         docs: {
-            description: 'Disallow tests to be nested within other tests '
-        }
+            description: 'Disallow tests to be nested within other tests ',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-nested-tests.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-pending-tests.js

@@ -6,8 +6,10 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow pending tests'
-        }
+            description: 'Disallow pending tests',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-pending-tests.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-return-and-callback.js

@@ -23,8 +23,10 @@ module.exports = {
     meta: {
         type: 'problem',
         docs: {
-            description: 'Disallow returning in a test or hook function that uses a callback'
-        }
+            description: 'Disallow returning in a test or hook function that uses a callback',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-return-and-callback.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-return-from-async.js

@@ -17,8 +17,10 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow returning from an async test or hook'
-        }
+            description: 'Disallow returning from an async test or hook',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-return-from-async.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-setup-in-describe.js

@@ -39,8 +39,10 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow setup in describe blocks'
-        }
+            description: 'Disallow setup in describe blocks',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-setup-in-describe.md'
+        },
+        schema: []
     },
     create(context) {
         const nesting = [];

lib/rules/no-sibling-hooks.js

@@ -16,8 +16,10 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow duplicate uses of a hook at the same level inside a describe'
-        }
+            description: 'Disallow duplicate uses of a hook at the same level inside a describe',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-sibling-hooks.md'
+        },
+        schema: []
     },
     create(context) {
         const isUsed = [];

lib/rules/no-skipped-tests.js

@@ -5,9 +5,11 @@ const createAstUtils = require('../util/ast');
 module.exports = {
     meta: {
         docs: {
-            description: 'Disallow skipped tests'
+            description: 'Disallow skipped tests',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-skipped-tests.md'
         },
-        type: 'problem'
+        type: 'problem',
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/no-synchronous-tests.js

@@ -44,7 +44,8 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Disallow synchronous tests'
+            description: 'Disallow synchronous tests',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-synchronous-tests.md'
         },
         schema: [
             {
@@ -92,7 +93,7 @@ module.exports = {
                 const isAsyncTest = testAsyncMethods.includes(true);
 
                 if (!isAsyncTest) {
-                    context.report(node, 'Unexpected synchronous test.');
+                    context.report({ node, message: 'Unexpected synchronous test.' });
                 }
             }
         }

lib/rules/no-top-level-hooks.js

@@ -6,8 +6,10 @@ module.exports = {
     meta: {
         type: 'problem',
         docs: {
-            description: 'Disallow top-level hooks'
-        }
+            description: 'Disallow top-level hooks',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/no-top-level-hooks.md'
+        },
+        schema: []
     },
     create(context) {
         const astUtils = createAstUtils(context.settings);

lib/rules/prefer-arrow-callback.js

@@ -120,7 +120,7 @@ module.exports = {
             description: 'Require using arrow functions for callbacks',
             category: 'ECMAScript 6',
             recommended: false,
-            url: 'https://eslint.org/docs/rules/prefer-arrow-callback'
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/prefer-arrow-callback.md'
         },
 
         schema: [

lib/rules/valid-suite-description.js

@@ -50,7 +50,8 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Match suite descriptions against a pre-configured regular expression'
+            description: 'Match suite descriptions against a pre-configured regular expression',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/valid-suite-description.md'
         },
         schema: [
             {
@@ -104,7 +105,7 @@ module.exports = {
 
                 if (isSuite(node)) {
                     if (!hasValidOrNoSuiteDescription(node)) {
-                        context.report(node, message || `Invalid "${ callee.name }()" description found.`);
+                        context.report({ node, message: message || `Invalid "${ callee.name }()" description found.` });
                     }
                 }
             }

lib/rules/valid-test-description.js

@@ -49,7 +49,8 @@ module.exports = {
     meta: {
         type: 'suggestion',
         docs: {
-            description: 'Match test descriptions against a pre-configured regular expression'
+            description: 'Match test descriptions against a pre-configured regular expression',
+            url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/master/docs/rules/valid-test-description.md'
         },
         schema: [
             {
@@ -103,7 +104,7 @@ module.exports = {
 
                 if (isTest(node)) {
                     if (!hasValidOrNoTestDescription(node)) {
-                        context.report(node, message || `Invalid "${ callee.name }()" description found.`);
+                        context.report({ node, message: message || `Invalid "${ callee.name }()" description found.` });
                     }
                 }
             }