test: with v8 and remove path as it duplicates test

devjiwonchoi · devjiwonchoi · commit 57366a2ee322 · 2024-10-14T19:25:10.000+09:00
diff --git a/test/unit/eslint-plugin-next/no-document-import-in-page.test.ts b/test/unit/eslint-plugin-next/no-document-import-in-page.test.ts
@@ -1,20 +1,10 @@
-import path from 'path'
+import { RuleTester as ESLintTesterV8 } from 'eslint'
+import { RuleTester as ESLintTesterV9 } from 'eslint-v9'
+import { rules } from '@next/eslint-plugin-next'
 
-import rule from '@next/eslint-plugin-next/dist/rules/no-document-import-in-page'
-import { RuleTester } from 'eslint'
-;(RuleTester as any).setDefaultConfig({
-  parserOptions: {
-    ecmaVersion: 2018,
-    sourceType: 'module',
-    ecmaFeatures: {
-      modules: true,
-      jsx: true,
-    },
-  },
-})
-const ruleTester = new RuleTester()
+const NextESLintRule = rules['no-document-import-in-page']
 
-ruleTester.run('no-document-import-in-page', rule, {
+const tests = {
   valid: [
     {
       code: `import Document from "next/document"
@@ -33,34 +23,6 @@ ruleTester.run('no-document-import-in-page', rule, {
     {
       code: `import Document from "next/document"
 
-    export default class MyDocument extends Document {
-      render() {
-        return (
-          <Html>
-          </Html>
-        );
-      }
-    }
-    `,
-      filename: `pages${path.sep}_document.js`,
-    },
-    {
-      code: `import NextDocument from "next/document"
-
-    export default class MyDocument extends NextDocument {
-      render() {
-        return (
-          <Html>
-          </Html>
-        );
-      }
-    }
-    `,
-      filename: `pages${path.posix.sep}_document.tsx`,
-    },
-    {
-      code: `import Document from "next/document"
-
     export default class MyDocument extends Document {
       render() {
         return (
@@ -144,19 +106,31 @@ ruleTester.run('no-document-import-in-page', rule, {
         },
       ],
     },
-    {
-      code: `import Document from "next/document"
+  ],
+}
 
-      export const Test = () => <p>Test</p>
-      `,
-      filename: `pages${path.sep}test.js`,
-      errors: [
-        {
-          message:
-            '`<Document />` from `next/document` should not be imported outside of `pages/_document.js`. See: https://nextjs.org/docs/messages/no-document-import-in-page',
-          type: 'ImportDeclaration',
+describe('no-document-import-in-page', () => {
+  new ESLintTesterV8({
+    parserOptions: {
+      ecmaVersion: 2018,
+      sourceType: 'module',
+      ecmaFeatures: {
+        modules: true,
+        jsx: true,
+      },
+    },
+  }).run('eslint-v8', NextESLintRule, tests)
+
+  new ESLintTesterV9({
+    languageOptions: {
+      ecmaVersion: 2018,
+      sourceType: 'module',
+      parserOptions: {
+        ecmaFeatures: {
+          modules: true,
+          jsx: true,
         },
-      ],
+      },
     },
-  ],
+  }).run('eslint-v9', NextESLintRule, tests)
 })
