move erroring tests to separate suite

vercel · Sep 24, 2022 · c6e9513 · c6e9513
1 parent 09d3d11
commit c6e9513
Show file tree

Hide file tree

Showing 6 changed files with 110 additions and 58 deletions.
diff --git a/test/e2e/app-dir/app-invalid.test.ts b/test/e2e/app-dir/app-invalid.test.ts
@@ -0,0 +1,93 @@
+import globOrig from 'glob'
+import cheerio from 'cheerio'
+import { promisify } from 'util'
+import path, { join } from 'path'
+import { createNext, FileRef } from 'e2e-utils'
+import { NextInstance } from 'test/lib/next-modes/base'
+import { check, fetchViaHTTP, normalizeRegEx } from 'next-test-utils'
+
+const glob = promisify(globOrig)
+
+describe('app-dir erroring for invalid cases', () => {
+  if ((global as any).isNextDeploy) {
+    it('should skip next deploy for now', () => {})
+    return
+  }
+
+  if (process.env.NEXT_TEST_REACT_VERSION === '^17') {
+    it('should skip for react v17', () => {})
+    return
+  }
+  let next: NextInstance
+
+  beforeAll(async () => {
+    next = await createNext({
+      files: new FileRef(path.join(__dirname, 'app-invalid')),
+      dependencies: {
+        react: 'experimental',
+        'react-dom': 'experimental',
+      },
+    })
+  })
+  afterAll(() => next.destroy())
+
+  if ((global as any).isNextStart) {
+    it('should skip next start for now', () => {})
+    return
+  }
+
+  it('should throw an error when getServerSideProps is used', async () => {
+    const pageFile = 'app/client-with-errors/get-server-side-props/page.js'
+    const content = await next.readFile(pageFile)
+    const uncomment = content.replace(
+      '// export function getServerSideProps',
+      'export function getServerSideProps'
+    )
+    await next.patchFile(pageFile, uncomment)
+    const res = await fetchViaHTTP(
+      next.url,
+      '/client-with-errors/get-server-side-props'
+    )
+    await next.patchFile(pageFile, content)
+
+    await check(async () => {
+      const { status } = await fetchViaHTTP(
+        next.url,
+        '/client-with-errors/get-server-side-props'
+      )
+      return status
+    }, /200/)
+
+    expect(res.status).toBe(500)
+    expect(await res.text()).toContain(
+      '`getServerSideProps` is not allowed in Client Components'
+    )
+  })
+
+  it('should throw an error when getStaticProps is used', async () => {
+    const pageFile = 'app/client-with-errors/get-static-props/page.js'
+    const content = await next.readFile(pageFile)
+    const uncomment = content.replace(
+      '// export function getStaticProps',
+      'export function getStaticProps'
+    )
+    await next.patchFile(pageFile, uncomment)
+    const res = await fetchViaHTTP(
+      next.url,
+      '/client-with-errors/get-static-props'
+    )
+    await next.patchFile(pageFile, content)
+    await check(async () => {
+      const { status } = await fetchViaHTTP(
+        next.url,
+        '/client-with-errors/get-static-props'
+      )
+      return status
+    }, /200/)
+
+    expect(res.status).toBe(500)
+    expect(await res.text()).toContain(
+      '`getStaticProps` is not allowed in Client Components'
+    )
+  })
+})
diff --git a/...with-errors/get-server-side-props/page.js → ...with-errors/get-server-side-props/page.js b/...with-errors/get-server-side-props/page.js → ...with-errors/get-server-side-props/page.js
diff --git a/...ient-with-errors/get-static-props/page.js → ...ient-with-errors/get-static-props/page.js b/...ient-with-errors/get-static-props/page.js → ...ient-with-errors/get-static-props/page.js
diff --git a/test/e2e/app-dir/app-invalid/app/layout.js b/test/e2e/app-dir/app-invalid/app/layout.js
@@ -0,0 +1,10 @@
+export default function Layout({ children }) {
+  return (
+    <html lang="en">
+      <head>
+        <title>my static blog</title>
+      </head>
+      <body>{children}</body>
+    </html>
+  )
+}
diff --git a/test/e2e/app-dir/app-invalid/next.config.js b/test/e2e/app-dir/app-invalid/next.config.js
@@ -0,0 +1,7 @@
+module.exports = {
+  experimental: {
+    appDir: true,
+    legacyBrowsers: false,
+    browsersListForSwc: true,
+  },
+}
diff --git a/test/e2e/app-dir/index.test.ts b/test/e2e/app-dir/index.test.ts
@@ -1078,64 +1078,6 @@ describe('app dir', () => {
           })
         })
       })
-
-      if (isDev) {
-        it('should throw an error when getServerSideProps is used', async () => {
-          const pageFile =
-            'app/client-with-errors/get-server-side-props/page.js'
-          const content = await next.readFile(pageFile)
-          const uncomment = content.replace(
-            '// export function getServerSideProps',
-            'export function getServerSideProps'
-          )
-          await next.patchFile(pageFile, uncomment)
-          const res = await fetchViaHTTP(
-            next.url,
-            '/client-with-errors/get-server-side-props'
-          )
-          await next.patchFile(pageFile, content)
-
-          await check(async () => {
-            const { status } = await fetchViaHTTP(
-              next.url,
-              '/client-with-errors/get-server-side-props'
-            )
-            return status
-          }, /200/)
-
-          expect(res.status).toBe(500)
-          expect(await res.text()).toContain(
-            '`getServerSideProps` is not allowed in Client Components'
-          )
-        })
-
-        it('should throw an error when getStaticProps is used', async () => {
-          const pageFile = 'app/client-with-errors/get-static-props/page.js'
-          const content = await next.readFile(pageFile)
-          const uncomment = content.replace(
-            '// export function getStaticProps',
-            'export function getStaticProps'
-          )
-          await next.patchFile(pageFile, uncomment)
-          const res = await fetchViaHTTP(
-            next.url,
-            '/client-with-errors/get-static-props'
-          )
-          await next.patchFile(pageFile, content)
-          await check(async () => {
-            const { status } = await fetchViaHTTP(
-              next.url,
-              '/client-with-errors/get-static-props'
-            )
-            return status
-          }, /200/)
-
-          expect(res.status).toBe(500)
-          expect(await res.text()).toContain(
-            '`getStaticProps` is not allowed in Client Components'
-          )
-        })
-      }
     })
 
     describe('css support', () => {