chore: rename eslint-rule-tester to test-helpers

aladdin-add · aladdin-add · commit 0d0745522cb4 · 2024-05-07T10:02:04.000+08:00
diff --git a/package.json b/package.json
@@ -117,6 +117,6 @@
         "*.md": "markdownlint --fix"
     },
     "imports": {
-        "#eslint-rule-tester": "./tests/eslint-rule-tester.js"
+        "#test-helpers": "./tests/test-helpers.js"
     }
 }
diff --git a/tests/helpers.js b/tests/helpers.js
diff --git a/tests/lib/configs/eslintrc.js b/tests/lib/configs/eslintrc.js
@@ -3,8 +3,6 @@
 const assert = require("assert")
 const path = require("path")
 const { LegacyESLint } = require("eslint/use-at-your-own-risk")
-// const {ESLint} = require("eslint")
-const { gtEslintV8 } = require("../../helpers")
 const originalCwd = process.cwd()
 
 // this is needed as `recommended` config was cached
@@ -15,7 +13,7 @@ function clearRequireCache() {
 }
 
 describe("node/recommended config", () => {
-    ;(gtEslintV8 ? describe : describe.skip)("in CJS directory", () => {
+    describe("in CJS directory", () => {
         const root = path.resolve(__dirname, "../../fixtures/configs/cjs/")
 
         /** @type {Linter} */
@@ -131,27 +129,24 @@ describe("node/recommended config", () => {
                 },
             ])
         })
-        ;(gtEslintV8 ? it : it.skip)(
-            "*.cjs files should be a script.",
-            async () => {
-                const report = await linter.lintText("import 'foo'", {
-                    filePath: path.join(root, "test.cjs"),
-                })
-
-                assert.deepStrictEqual(report[0].messages, [
-                    {
-                        column: 1,
-                        fatal: true,
-                        line: 1,
-                        message:
-                            "Parsing error: 'import' and 'export' may appear only with 'sourceType: module'",
-                        ruleId: null,
-                        nodeType: null,
-                        severity: 2,
-                    },
-                ])
-            }
-        )
+        it("*.cjs files should be a script.", async () => {
+            const report = await linter.lintText("import 'foo'", {
+                filePath: path.join(root, "test.cjs"),
+            })
+
+            assert.deepStrictEqual(report[0].messages, [
+                {
+                    column: 1,
+                    fatal: true,
+                    line: 1,
+                    message:
+                        "Parsing error: 'import' and 'export' may appear only with 'sourceType: module'",
+                    ruleId: null,
+                    nodeType: null,
+                    severity: 2,
+                },
+            ])
+        })
 
         it("*.mjs files should be a module.", async () => {
             const report = await linter.lintText("import 'foo'", {
diff --git a/tests/lib/rules/callback-return.js b/tests/lib/rules/callback-return.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/callback-return")
 const ruleTester = new RuleTester()
 
diff --git a/tests/lib/rules/exports-style.js b/tests/lib/rules/exports-style.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/exports-style")
 
 new RuleTester({ languageOptions: { ecmaVersion: 11 } }).run(
diff --git a/tests/lib/rules/file-extension-in-import.js b/tests/lib/rules/file-extension-in-import.js
@@ -6,7 +6,7 @@
 
 const path = require("path")
 const { Linter } = require("eslint")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("../../test-helpers").RuleTester
 const rule = require("../../../lib/rules/file-extension-in-import")
 
 const DynamicImportSupported = (() => {
diff --git a/tests/lib/rules/global-require.js b/tests/lib/rules/global-require.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/global-require")
 
 const ERROR = { messageId: "unexpected", type: "CallExpression" }
diff --git a/tests/lib/rules/handle-callback-err.js b/tests/lib/rules/handle-callback-err.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/handle-callback-err")
 const ruleTester = new RuleTester()
 
diff --git a/tests/lib/rules/hashbang.js b/tests/lib/rules/hashbang.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/shebang")
 
 /**
diff --git a/tests/lib/rules/no-callback-literal.js b/tests/lib/rules/no-callback-literal.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-callback-literal")
 const tsParser = require("@typescript-eslint/parser")
 
diff --git a/tests/lib/rules/no-deprecated-api.js b/tests/lib/rules/no-deprecated-api.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const { RuleTester } = require("#eslint-rule-tester")
+const { RuleTester } = require("#test-helpers")
 const rule = require("../../../lib/rules/no-deprecated-api")
 
 const ruleTester = new RuleTester()
diff --git a/tests/lib/rules/no-exports-assign.js b/tests/lib/rules/no-exports-assign.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const { RuleTester } = require("#eslint-rule-tester")
+const { RuleTester } = require("#test-helpers")
 const rule = require("../../../lib/rules/no-exports-assign.js")
 
 new RuleTester({
diff --git a/tests/lib/rules/no-extraneous-import.js b/tests/lib/rules/no-extraneous-import.js
@@ -6,7 +6,7 @@
 
 const path = require("path")
 const { Linter } = require("eslint")
-const { RuleTester } = require("#eslint-rule-tester")
+const { RuleTester } = require("#test-helpers")
 const rule = require("../../../lib/rules/no-extraneous-import")
 
 const DynamicImportSupported = (() => {
diff --git a/tests/lib/rules/no-extraneous-require.js b/tests/lib/rules/no-extraneous-require.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-extraneous-require")
 
 /**
diff --git a/tests/lib/rules/no-hide-core-modules.js b/tests/lib/rules/no-hide-core-modules.js
@@ -14,7 +14,7 @@
 //------------------------------------------------------------------------------
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-hide-core-modules")
 
 //------------------------------------------------------------------------------
diff --git a/tests/lib/rules/no-missing-import.js b/tests/lib/rules/no-missing-import.js
@@ -6,7 +6,7 @@
 
 const path = require("path")
 const { Linter } = require("eslint")
-const { RuleTester, isCaseSensitiveFileSystem } = require("#eslint-rule-tester")
+const { RuleTester, isCaseSensitiveFileSystem } = require("../../test-helpers")
 const rule = require("../../../lib/rules/no-missing-import")
 
 const DynamicImportSupported = (() => {
diff --git a/tests/lib/rules/no-missing-require.js b/tests/lib/rules/no-missing-require.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const { RuleTester, isCaseSensitiveFileSystem } = require("#eslint-rule-tester")
+const { RuleTester, isCaseSensitiveFileSystem } = require("../../test-helpers")
 const rule = require("../../../lib/rules/no-missing-require")
 
 const tsReactExtensionMap = [
diff --git a/tests/lib/rules/no-mixed-requires.js b/tests/lib/rules/no-mixed-requires.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-mixed-requires")
 const ruleTester = new RuleTester()
 
diff --git a/tests/lib/rules/no-new-require.js b/tests/lib/rules/no-new-require.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-new-require")
 
 new RuleTester().run("no-new-require", rule, {
diff --git a/tests/lib/rules/no-path-concat.js b/tests/lib/rules/no-path-concat.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-path-concat")
 
 new RuleTester().run("no-path-concat", rule, {
diff --git a/tests/lib/rules/no-process-env.js b/tests/lib/rules/no-process-env.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-process-env")
 
 new RuleTester().run("no-process-env", rule, {
diff --git a/tests/lib/rules/no-process-exit.js b/tests/lib/rules/no-process-exit.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-process-exit")
 const ruleTester = new RuleTester()
 
diff --git a/tests/lib/rules/no-restricted-import.js b/tests/lib/rules/no-restricted-import.js
@@ -6,7 +6,7 @@
 
 const path = require("path")
 const { Linter } = require("eslint")
-const { RuleTester } = require("#eslint-rule-tester")
+const { RuleTester } = require("#test-helpers")
 const rule = require("../../../lib/rules/no-restricted-import")
 
 const DynamicImportSupported = (() => {
diff --git a/tests/lib/rules/no-restricted-require.js b/tests/lib/rules/no-restricted-require.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-restricted-require")
 
 new RuleTester().run("no-restricted-require", rule, {
diff --git a/tests/lib/rules/no-sync.js b/tests/lib/rules/no-sync.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-sync")
 
 new RuleTester().run("no-sync", rule, {
diff --git a/tests/lib/rules/no-unpublished-bin.js b/tests/lib/rules/no-unpublished-bin.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-unpublished-bin")
 
 /**
diff --git a/tests/lib/rules/no-unpublished-import.js b/tests/lib/rules/no-unpublished-import.js
@@ -6,7 +6,7 @@
 
 const path = require("path")
 const { Linter } = require("eslint")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-unpublished-import")
 const globals = require("globals")
 
diff --git a/tests/lib/rules/no-unpublished-require.js b/tests/lib/rules/no-unpublished-require.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../lib/rules/no-unpublished-require")
 
 /**
diff --git a/tests/lib/rules/no-unsupported-features/es-builtins.js b/tests/lib/rules/no-unsupported-features/es-builtins.js
@@ -3,7 +3,7 @@
  * See LICENSE file in root directory for full license.
  */
 "use strict"
-const { RuleTester } = require("#eslint-rule-tester")
+const { RuleTester } = require("#test-helpers")
 const rule = require("../../../../lib/rules/no-unsupported-features/es-builtins")
 const globals = require("globals")
 
diff --git a/tests/lib/rules/no-unsupported-features/es-syntax.js b/tests/lib/rules/no-unsupported-features/es-syntax.js
@@ -5,7 +5,7 @@
 "use strict"
 
 const path = require("path")
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/no-unsupported-features/es-syntax")
 
 /**
diff --git a/tests/lib/rules/no-unsupported-features/node-builtins.js b/tests/lib/rules/no-unsupported-features/node-builtins.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("../../../test-helpers").RuleTester
 const rule = require("../../../../lib/rules/no-unsupported-features/node-builtins")
 
 /**
diff --git a/tests/lib/rules/prefer-global/buffer.js b/tests/lib/rules/prefer-global/buffer.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/buffer")
 
 new RuleTester().run("prefer-global/buffer", rule, {
diff --git a/tests/lib/rules/prefer-global/console.js b/tests/lib/rules/prefer-global/console.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/console")
 
 new RuleTester().run("prefer-global/console", rule, {
diff --git a/tests/lib/rules/prefer-global/process.js b/tests/lib/rules/prefer-global/process.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/process")
 
 new RuleTester().run("prefer-global/process", rule, {
diff --git a/tests/lib/rules/prefer-global/text-decoder.js b/tests/lib/rules/prefer-global/text-decoder.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/text-decoder")
 
 new RuleTester().run("prefer-global/text-decoder", rule, {
diff --git a/tests/lib/rules/prefer-global/text-encoder.js b/tests/lib/rules/prefer-global/text-encoder.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/text-encoder")
 
 new RuleTester().run("prefer-global/text-encoder", rule, {
diff --git a/tests/lib/rules/prefer-global/url-search-params.js b/tests/lib/rules/prefer-global/url-search-params.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/url-search-params")
 
 new RuleTester().run("prefer-global/url-search-params", rule, {
diff --git a/tests/lib/rules/prefer-global/url.js b/tests/lib/rules/prefer-global/url.js
@@ -4,7 +4,7 @@
  */
 "use strict"
 
-const RuleTester = require("#eslint-rule-tester").RuleTester
+const RuleTester = require("#test-helpers").RuleTester
 const rule = require("../../../../lib/rules/prefer-global/url")
 
 new RuleTester().run("prefer-global/url", rule, {
diff --git a/tests/lib/rules/prefer-node-protocol.js b/tests/lib/rules/prefer-node-protocol.js
diff --git a/tests/lib/rules/prefer-promises/dns.js b/tests/lib/rules/prefer-promises/dns.js
diff --git a/tests/lib/rules/prefer-promises/fs.js b/tests/lib/rules/prefer-promises/fs.js
diff --git a/tests/test-helpers.js b/tests/test-helpers.js

Original file line number	Diff line number	Diff line change
`@@ -117,6 +117,6 @@`
`117`	`117`	`"*.md": "markdownlint --fix"`
`118`	`118`	`},`
`119`	`119`	`"imports": {`
`120`		`- "#eslint-rule-tester": "./tests/eslint-rule-tester.js"`
	`120`	`+ "#test-helpers": "./tests/test-helpers.js"`
`121`	`121`	`}`
`122`	`122`	`}`