test(linter/plugins): standardize test fixture structure

apps/oxlint/test/e2e.test.ts

@@ -236,7 +236,7 @@ describe('oxlint CLI', () => {
   });
 
   it('should not apply fixes when `--fix` is disabled', async () => {
-    const fixtureFilePath = pathJoin(PACKAGE_ROOT_PATH, 'test/fixtures/fixes/index.js');
+    const fixtureFilePath = pathJoin(PACKAGE_ROOT_PATH, 'test/fixtures/fixes/files/index.js');
     const codeBefore = fs.readFileSync(fixtureFilePath, 'utf8');
 
     let error = true;
@@ -256,7 +256,7 @@ describe('oxlint CLI', () => {
   });
 
   it('should apply fixes when `--fix` is enabled', async () => {
-    const fixtureFilePath = pathJoin(PACKAGE_ROOT_PATH, 'test/fixtures/fixes/index.js');
+    const fixtureFilePath = pathJoin(PACKAGE_ROOT_PATH, 'test/fixtures/fixes/files/index.js');
     const codeBefore = fs.readFileSync(fixtureFilePath, 'utf8');
 
     let error = true;

apps/oxlint/test/fixtures/basic_custom_plugin/.oxlintrc.json

@@ -1,7 +1,7 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "rules": {
         "basic-custom-plugin/no-debugger": "error"
     },
-    "ignorePatterns": ["test_plugin/**"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/basic_custom_plugin_many_files/.oxlintrc.json

@@ -1,7 +1,7 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "rules": {
         "basic-custom-plugin/no-debugger": "error"
     },
-    "ignorePatterns": ["test_plugin/**"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/basic_custom_plugin_multiple_rules/.oxlintrc.json

@@ -1,9 +1,9 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "rules": {
         "basic-custom-plugin/no-debugger": "error",
         "basic-custom-plugin/no-debugger-2": "error",
         "basic-custom-plugin/no-identifiers-named-foo": "error"
     },
-    "ignorePatterns": ["test_plugin/**"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/basic_custom_plugin_warn_severity/.oxlintrc.json

@@ -1,7 +1,7 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "rules": {
         "basic-custom-plugin/no-debugger": "warn"
     },
-    "ignorePatterns": ["test_plugin/**"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/built_in_errors/.oxlintrc.json

@@ -1,4 +1,5 @@
 {
   "categories": { "correctness": "off" },
-  "rules": { "no-debugger": "error" }
+  "rules": { "no-debugger": "error" },
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/built_in_no_errors/.oxlintrc.json

@@ -1,4 +1,5 @@
 {
   "categories": { "correctness": "off" },
-  "rules": { "no-debugger": "error" }
+  "rules": { "no-debugger": "error" },
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/context_properties/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "categories": {"correctness": "off"},
     "rules": {
         "context-plugin/log-context": "error"
     },
-    "ignorePatterns": ["test_plugin/**"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/context_properties/test_plugin/index.js → apps/oxlint/test/fixtures/context_properties/plugin.js

@@ -1,12 +1,12 @@
-import { dirname, sep } from 'node:path';
+import { sep } from 'node:path';
 
 const SPAN = { start: 0, end: 0 };
 
-const PARENT_DIR_PATH_LEN = dirname(import.meta.dirname).length + 1;
+const DIR_PATH_LEN = import.meta.dirname.length + 1;
 
 const relativePath = sep === '/'
-  ? path => path.slice(PARENT_DIR_PATH_LEN)
-  : path => path.slice(PARENT_DIR_PATH_LEN).replace(/\\/g, '/');
+  ? path => path.slice(DIR_PATH_LEN)
+  : path => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
 
 const rule = {
   create(context) {

apps/oxlint/test/fixtures/createOnce/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "categories": {"correctness": "off"},
     "rules": {
         "create-once-plugin/always-run": "error",
@@ -8,5 +8,5 @@
         "create-once-plugin/after-only": "error",
         "create-once-plugin/no-hooks": "error"
     },
-    "ignorePatterns": ["test_plugin/**"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/createOnce/test_plugin/index.js → apps/oxlint/test/fixtures/createOnce/plugin.js

@@ -1,12 +1,12 @@
-import { dirname, sep } from 'node:path';
+import { sep } from 'node:path';
 
 const SPAN = { start: 0, end: 0 };
 
-const PARENT_DIR_PATH_LEN = dirname(import.meta.dirname).length + 1;
+const DIR_PATH_LEN = import.meta.dirname.length + 1;
 
 const relativePath = sep === '/'
-  ? path => path.slice(PARENT_DIR_PATH_LEN)
-  : path => path.slice(PARENT_DIR_PATH_LEN).replace(/\\/g, '/');
+  ? path => path.slice(DIR_PATH_LEN)
+  : path => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
 
 let createOnceCallCount = 0;
 const alwaysRunRule = {

apps/oxlint/test/fixtures/custom_plugin_disable_directives/.oxlintrc.json

@@ -1,9 +1,9 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "test-plugin/no-var": "error",
     "no-debugger": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_import_error/.oxlintrc.json

@@ -1,7 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "basic-custom-plugin/unknown-rule": "error"
-  }
+  },
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_lint_after_hook_error/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "error-plugin/error": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_lint_before_hook_error/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "error-plugin/error": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_lint_createOnce_error/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "error-plugin/error": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_lint_create_error/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "error-plugin/error": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_lint_fix_error/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "error-plugin/error": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_lint_visit_error/.oxlintrc.json

@@ -1,8 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "error-plugin/error": "error"
   },
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_missing_rule/.oxlintrc.json

@@ -1,7 +1,8 @@
 {
-  "plugins": ["./test_plugin"],
+  "plugins": ["./plugin.js"],
   "categories": { "correctness": "off" },
   "rules": {
     "basic-custom-plugin/unknown-rule": "error"
-  }
+  },
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_via_overrides/.oxlintrc.json

@@ -2,11 +2,11 @@
   "overrides": [
     {
       "files": ["*.js"],
-      "plugins": ["./test_plugin"],
+      "plugins": ["./plugin.js"],
       "rules": {
         "basic-custom-plugin/no-debugger": "error"
       }
     }
   ],
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/custom_plugin_via_overrides_missing_rule/.oxlintrc.json

@@ -1,15 +1,15 @@
 {
   "rules": {
-    "no-debugger": "off",
+    "no-debugger": "off"
   },
   "overrides": [
     {
       "files": ["*.js"],
-      "plugins": ["./test_plugin"],
+      "plugins": ["./plugin.js"],
       "rules": {
         "basic-custom-plugin/missing": "error"
       }
     }
   ],
-  "ignorePatterns": ["test_plugin/**"]
+  "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/definePlugin/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "categories": {"correctness": "off"},
     "rules": {
         "define-plugin-plugin/create": "error",
@@ -9,5 +9,5 @@
         "define-plugin-plugin/create-once-after-only": "error",
         "define-plugin-plugin/create-once-no-hooks": "error"
     },
-    "ignorePatterns": ["test_plugin/**", "eslint.config.js"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/definePlugin/eslint.config.js

@@ -1,10 +1,10 @@
-import testPlugin from './test_plugin/index.js';
+import plugin from './plugin.js';
 
 export default [
   {
     files: ["files/*.js"],
     plugins: {
-      "define-plugin-plugin": testPlugin,
+      "define-plugin-plugin": plugin,
     },
     rules: {
       "define-plugin-plugin/create": "error",

apps/oxlint/test/fixtures/definePlugin/test_plugin/index.js → apps/oxlint/test/fixtures/definePlugin/plugin.js

@@ -1,5 +1,5 @@
-import { dirname, sep } from 'node:path';
-import { definePlugin } from '../../../../dist/index.js';
+import { sep } from 'node:path';
+import { definePlugin } from '../../../dist/index.js';
 
 // `loc` field is required for ESLint.
 // TODO: Remove this workaround when AST nodes have a `loc` field.
@@ -12,11 +12,11 @@ const SPAN = {
   },
 };
 
-const PARENT_DIR_PATH_LEN = dirname(import.meta.dirname).length + 1;
+const DIR_PATH_LEN = import.meta.dirname.length + 1;
 
 const relativePath = sep === '/'
-  ? path => path.slice(PARENT_DIR_PATH_LEN)
-  : path => path.slice(PARENT_DIR_PATH_LEN).replace(/\\/g, '/');
+  ? path => path.slice(DIR_PATH_LEN)
+  : path => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
 
 const createRule = {
   create(context) {

apps/oxlint/test/fixtures/definePlugin_and_defineRule/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "categories": {"correctness": "off"},
     "rules": {
         "define-plugin-and-rule-plugin/create": "error",
@@ -9,5 +9,5 @@
         "define-plugin-and-rule-plugin/create-once-after-only": "error",
         "define-plugin-and-rule-plugin/create-once-no-hooks": "error"
     },
-    "ignorePatterns": ["test_plugin/**", "eslint.config.js"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/definePlugin_and_defineRule/eslint.config.js

@@ -1,10 +1,10 @@
-import testPlugin from './test_plugin/index.js';
+import plugin from './plugin.js';
 
 export default [
   {
     files: ["files/*.js"],
     plugins: {
-      "define-plugin-and-rule-plugin": testPlugin,
+      "define-plugin-and-rule-plugin": plugin,
     },
     rules: {
       "define-plugin-and-rule-plugin/create": "error",

apps/oxlint/test/fixtures/definePlugin_and_defineRule/test_plugin/index.js → apps/oxlint/test/fixtures/definePlugin_and_defineRule/plugin.js

@@ -1,5 +1,5 @@
-import { dirname, sep } from 'node:path';
-import { definePlugin, defineRule } from '../../../../dist/index.js';
+import { sep } from 'node:path';
+import { definePlugin, defineRule } from '../../../dist/index.js';
 
 // `loc` field is required for ESLint.
 // TODO: Remove this workaround when AST nodes have a `loc` field.
@@ -12,11 +12,11 @@ const SPAN = {
   },
 };
 
-const PARENT_DIR_PATH_LEN = dirname(import.meta.dirname).length + 1;
+const DIR_PATH_LEN = import.meta.dirname.length + 1;
 
 const relativePath = sep === '/'
-  ? path => path.slice(PARENT_DIR_PATH_LEN)
-  : path => path.slice(PARENT_DIR_PATH_LEN).replace(/\\/g, '/');
+  ? path => path.slice(DIR_PATH_LEN)
+  : path => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
 
 const createRule = defineRule({
   create(context) {

apps/oxlint/test/fixtures/defineRule/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-    "plugins": ["./test_plugin"],
+    "plugins": ["./plugin.js"],
     "categories": {"correctness": "off"},
     "rules": {
         "define-rule-plugin/create": "error",
@@ -9,5 +9,5 @@
         "define-rule-plugin/create-once-after-only": "error",
         "define-rule-plugin/create-once-no-hooks": "error"
     },
-    "ignorePatterns": ["test_plugin/**", "eslint.config.js"]
+    "ignorePatterns": ["**/*", "!files/**"]
 }

apps/oxlint/test/fixtures/defineRule/eslint.config.js

@@ -1,10 +1,10 @@
-import testPlugin from './test_plugin/index.js';
+import plugin from './plugin.js';
 
 export default [
   {
     files: ["files/*.js"],
     plugins: {
-      "define-rule-plugin": testPlugin,
+      "define-rule-plugin": plugin,
     },
     rules: {
       "define-rule-plugin/create": "error",