Merge pull request #426 from lo1tuma/make-rules-js-only

Declare rules as JS-only

Author: screendriver

source/plugin.test.ts

@@ -54,6 +54,12 @@ describe('eslint-plugin-mocha', function () {
         }
     });
 
+    it('should declare all rules as js-only', function () {
+        for (const rule of Object.values(plugin.rules)) {
+            assert.deepStrictEqual(rule.meta?.languages, ['js/js']);
+        }
+    });
+
     describe('documentation', function () {
         it('should have each rule documented', async function () {
             const ruleFiles = await determineAllRuleFiles();

source/rules/consistent-interface.ts

@@ -288,6 +288,7 @@ export function reportUnexpectedImportBindingsInModule(
 export const consistentInterfaceRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'problem',
+        languages: ['js/js'],
         docs: {
             description: 'Enforces consistent use of mocha interfaces',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/consistent-interface.md'

source/rules/consistent-spacing-between-blocks.ts

@@ -105,6 +105,7 @@ function getSpacingCheck(
 export const consistentSpacingBetweenBlocksRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'suggestion',
+        languages: ['js/js'],
         fixable: 'whitespace',
         schema: [],
         messages: {

source/rules/handle-done-callback.ts

@@ -30,6 +30,7 @@ export function findParamInScope(
 export const handleDoneCallbackRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'problem',
+        languages: ['js/js'],
         docs: {
             description: 'Enforces handling of callbacks for async tests',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/handle-done-callback.md'

source/rules/max-top-level-suites.ts

@@ -24,6 +24,7 @@ function isTopLevelScope(scope: Readonly<Scope.Scope>): boolean {
 export const maxTopLevelSuitesRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'suggestion',
+        languages: ['js/js'],
         docs: {
             description: 'Enforce the number of top-level suites in a single file',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/max-top-level-suites.md'

source/rules/no-async-suite.ts

@@ -93,6 +93,7 @@ export function fixAsyncFunction(
 export const noAsyncSuiteRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'problem',
+        languages: ['js/js'],
         docs: {
             description: 'Disallow async functions passed to a suite',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/no-async-suite.md'

source/rules/no-empty-title.ts

@@ -59,6 +59,7 @@ function isValidDescriptionArgumentNode(node: Except<Rule.Node, 'parent'> | unde
 export const noEmptyTitleRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'suggestion',
+        languages: ['js/js'],
         docs: {
             description: 'Disallow empty test descriptions',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/no-empty-title.md'

source/rules/no-exclusive-tests.ts

@@ -5,6 +5,7 @@ import type { CallExpression, MemberExpression } from '../ast/node-types.js';
 export const noExclusiveTestsRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'problem',
+        languages: ['js/js'],
         docs: {
             description: 'Disallow exclusive tests',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/no-exclusive-tests.md'

source/rules/no-exports.ts

@@ -11,6 +11,7 @@ export const noExportsRule: Readonly<Rule.RuleModule> = {
             unexpectedExport: 'Unexpected export from a test file'
         },
         type: 'suggestion',
+        languages: ['js/js'],
         schema: []
     },
     create(context) {

source/rules/no-global-tests.ts

@@ -8,6 +8,7 @@ function isGlobalScope(scope: Readonly<Scope.Scope>): boolean {
 export const noGlobalTestsRule: Readonly<Rule.RuleModule> = {
     meta: {
         type: 'suggestion',
+        languages: ['js/js'],
         docs: {
             description: 'Disallow global tests',
             url: 'https://github.com/lo1tuma/eslint-plugin-mocha/blob/main/documentation/rules/no-global-tests.md'