test: add v9 tests that was missed

Author: devjiwonchoi

test/unit/eslint-plugin-next/no-async-client-component.test.ts

@@ -1,27 +1,51 @@
-import rule from '@next/eslint-plugin-next/dist/rules/no-async-client-component'
-import { RuleTester } from 'eslint'
-;(RuleTester as any).setDefaultConfig({
-  parserOptions: {
-    ecmaVersion: 2018,
-    sourceType: 'module',
-    ecmaFeatures: {
-      modules: true,
-      jsx: true,
-    },
-  },
-})
-const ruleTester = new RuleTester()
+import { RuleTester as ESLintTesterV8 } from 'eslint'
+import { RuleTester as ESLintTesterV9 } from 'eslint-v9'
+import { rules } from '@next/eslint-plugin-next'
+
+const NextESLintRule = rules['no-async-client-component']
 
 const message =
   'Prevent client components from being async functions. See: https://nextjs.org/docs/messages/no-async-client-component'
 
-ruleTester.run('no-async-client-component single line', rule, {
+const tests = {
   valid: [
     `
     export default async function MyComponent() {
       return <></>
     }
     `,
+    `
+    "use client"
+
+    export default async function myFunction() {
+      return ''
+    }
+    `,
+    `
+    async function MyComponent() {
+      return <></>
+    }
+
+    export default MyComponent
+    `,
+    `
+    "use client"
+
+    async function myFunction() {
+      return ''
+    }
+
+    export default myFunction
+    `,
+    `
+    "use client"
+
+    const myFunction = () => {
+      return ''
+    }
+
+    export default myFunction
+    `,
   ],
   invalid: [
     {
@@ -32,26 +56,8 @@ ruleTester.run('no-async-client-component single line', rule, {
         return <></>
       }
       `,
-      errors: [
-        {
-          message,
-        },
-      ],
+      errors: [{ message }],
     },
-  ],
-})
-
-ruleTester.run('no-async-client-component single line capitalization', rule, {
-  valid: [
-    `
-    "use client"
-
-    export default async function myFunction() {
-      return ''
-    }
-    `,
-  ],
-  invalid: [
     {
       code: `
       "use client"
@@ -60,26 +66,8 @@ ruleTester.run('no-async-client-component single line capitalization', rule, {
         return ''
       }
       `,
-      errors: [
-        {
-          message,
-        },
-      ],
+      errors: [{ message }],
     },
-  ],
-})
-
-ruleTester.run('no-async-client-component multiple line', rule, {
-  valid: [
-    `
-    async function MyComponent() {
-      return <></>
-    }
-
-    export default MyComponent
-    `,
-  ],
-  invalid: [
     {
       code: `
       "use client"
@@ -90,28 +78,8 @@ ruleTester.run('no-async-client-component multiple line', rule, {
 
       export default MyComponent
       `,
-      errors: [
-        {
-          message,
-        },
-      ],
+      errors: [{ message }],
     },
-  ],
-})
-
-ruleTester.run('no-async-client-component multiple line capitalization', rule, {
-  valid: [
-    `
-    "use client"
-
-    async function myFunction() {
-      return ''
-    }
-
-    export default myFunction
-    `,
-  ],
-  invalid: [
     {
       code: `
       "use client"
@@ -122,28 +90,8 @@ ruleTester.run('no-async-client-component multiple line capitalization', rule, {
 
       export default MyFunction
       `,
-      errors: [
-        {
-          message,
-        },
-      ],
+      errors: [{ message }],
     },
-  ],
-})
-
-ruleTester.run('no-async-client-component arrow function', rule, {
-  valid: [
-    `
-    "use client"
-
-    const myFunction = () => {
-      return ''
-    }
-
-    export default myFunction
-    `,
-  ],
-  invalid: [
     {
       code: `
       "use client"
@@ -154,11 +102,33 @@ ruleTester.run('no-async-client-component arrow function', rule, {
 
       export default MyFunction
       `,
-      errors: [
-        {
-          message,
-        },
-      ],
+      errors: [{ message }],
     },
   ],
+}
+
+describe('no-async-client-component single line', () => {
+  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)
 })

test/unit/eslint-plugin-next/no-head-element.test.ts

@@ -1,8 +1,8 @@
-import rule from '@next/eslint-plugin-next/dist/rules/no-head-element'
 import { RuleTester as ESLintTesterV8 } from 'eslint'
 import { RuleTester as ESLintTesterV9 } from 'eslint-v9'
+import { rules } from '@next/eslint-plugin-next'
 
-const NextESLintRule = rule
+const NextESLintRule = rules['no-head-element']
 
 const message =
   'Do not use `<head>` element. Use `<Head />` from `next/head` instead. See: https://nextjs.org/docs/messages/no-head-element'