Skip production tests for Turbopack

vercel · Sep 26, 2023 · f0d5d8a · f0d5d8a
1 parent 7102e29
commit f0d5d8a
Show file tree

Hide file tree

Showing 92 changed files with 355 additions and 394 deletions.
diff --git a/test/integration/404-page-app/test/index.test.js b/test/integration/404-page-app/test/index.test.js
@@ -21,7 +21,7 @@ let appPort
 let app
 
 describe('404 Page Support with _app', () => {
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     afterAll(() => killApp(app))
 
     it('should build successfully', async () => {

diff --git a/test/integration/amphtml/test/index.test.js b/test/integration/amphtml/test/index.test.js
@@ -25,7 +25,7 @@ let app
 const context = {}
 
 describe('AMP Usage', () => {
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     let output = ''
 
     beforeAll(async () => {

diff --git a/test/integration/app-tree/test/index.test.js b/test/integration/app-tree/test/index.test.js
@@ -56,8 +56,7 @@ describe('AppTree', () => {
     afterAll(() => killApp(app))
     runTests()
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/app-types/app-types.test.js b/test/integration/app-types/app-types.test.js
@@ -7,7 +7,7 @@ import { nextBuild } from 'next-test-utils'
 const appDir = __dirname
 
 describe('app type checking', () => {
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     let stderr, errors
     beforeAll(async () => {
       stderr = (await nextBuild(appDir, [], { stderr: true })).stderr

diff --git a/test/integration/async-modules/test/index.test.js b/test/integration/async-modules/test/index.test.js
@@ -115,8 +115,7 @@ describe('Async modules', () => {
 
     runTests(true)
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/basepath-root-catch-all/test/index.test.js b/test/integration/basepath-root-catch-all/test/index.test.js
@@ -37,8 +37,7 @@ describe('dev mode', () => {
   afterAll(() => killApp(app))
   runTests()
 })
-
-describe('production mode', () => {
+;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
   beforeAll(async () => {
     await nextBuild(appDir)
     buildId = await fs.readFile(join(appDir, '.next/BUILD_ID'), 'utf8')

diff --git a/test/integration/client-404/test/index.test.js b/test/integration/client-404/test/index.test.js
@@ -35,8 +35,7 @@ describe('Client 404', () => {
 
     runTests()
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       context.appPort = await findPort()

diff --git a/test/integration/client-shallow-routing/test/index.test.js b/test/integration/client-shallow-routing/test/index.test.js
@@ -97,8 +97,7 @@ describe('Client Shallow Routing', () => {
 
     runTests()
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/css/test/css-modules.test.js b/test/integration/css/test/css-modules.test.js
@@ -93,8 +93,7 @@ describe('Basic CSS Modules Ordering', () => {
 
     tests()
   })
-
-  describe('Production Mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await remove(join(appDir, '.next'))
     })
@@ -436,8 +435,7 @@ describe('CSS Modules Composes Ordering', () => {
 
     tests(true)
   })
-
-  describe('Production Mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await remove(join(appDir, '.next'))
     })

diff --git a/test/integration/css/test/css-rendering.test.js b/test/integration/css/test/css-rendering.test.js
@@ -168,21 +168,24 @@ describe('CSS Support', () => {
       })
     }
 
-    describe('Production Mode', () => {
-      beforeAll(async () => {
-        await remove(join(appDir, '.next'))
-      })
-      beforeAll(async () => {
-        await nextBuild(appDir, [], {})
-        appPort = await findPort()
-        app = await nextStart(appDir, appPort)
-      })
-      afterAll(async () => {
-        await killApp(app)
-      })
-
-      tests()
-    })
+    ;(process.env.TURBOPACK ? describe.skip : describe)(
+      'production mode',
+      () => {
+        beforeAll(async () => {
+          await remove(join(appDir, '.next'))
+        })
+        beforeAll(async () => {
+          await nextBuild(appDir, [], {})
+          appPort = await findPort()
+          app = await nextStart(appDir, appPort)
+        })
+        afterAll(async () => {
+          await killApp(app)
+        })
+
+        tests()
+      }
+    )
   })
 
   describe('CSS Cleanup on Render Failure', () => {
@@ -231,21 +234,24 @@ describe('CSS Support', () => {
       })
     }
 
-    describe('Production Mode', () => {
-      beforeAll(async () => {
-        await remove(join(appDir, '.next'))
-      })
-      beforeAll(async () => {
-        await nextBuild(appDir, [], {})
-        appPort = await findPort()
-        app = await nextStart(appDir, appPort)
-      })
-      afterAll(async () => {
-        await killApp(app)
-      })
-
-      tests()
-    })
+    ;(process.env.TURBOPACK ? describe.skip : describe)(
+      'production mode',
+      () => {
+        beforeAll(async () => {
+          await remove(join(appDir, '.next'))
+        })
+        beforeAll(async () => {
+          await nextBuild(appDir, [], {})
+          appPort = await findPort()
+          app = await nextStart(appDir, appPort)
+        })
+        afterAll(async () => {
+          await killApp(app)
+        })
+
+        tests()
+      }
+    )
   })
 
   describe('Page reload on CSS missing', () => {
@@ -280,31 +286,34 @@ describe('CSS Support', () => {
       })
     }
 
-    describe('Production Mode', () => {
-      beforeAll(async () => {
-        await remove(join(appDir, '.next'))
-      })
-      beforeAll(async () => {
-        await nextBuild(appDir, [], {})
-        appPort = await findPort()
-        app = await nextStart(appDir, appPort)
-
-        // Remove other page CSS files:
-        const manifest = await readJSON(
-          join(appDir, '.next', 'build-manifest.json')
-        )
-        const files = manifest['pages']['/other'].filter((e) =>
-          e.endsWith('.css')
-        )
-        if (files.length < 1) throw new Error()
-        await Promise.all(files.map((f) => remove(join(appDir, '.next', f))))
-      })
-      afterAll(async () => {
-        await killApp(app)
-      })
-
-      tests()
-    })
+    ;(process.env.TURBOPACK ? describe.skip : describe)(
+      'production mode',
+      () => {
+        beforeAll(async () => {
+          await remove(join(appDir, '.next'))
+        })
+        beforeAll(async () => {
+          await nextBuild(appDir, [], {})
+          appPort = await findPort()
+          app = await nextStart(appDir, appPort)
+
+          // Remove other page CSS files:
+          const manifest = await readJSON(
+            join(appDir, '.next', 'build-manifest.json')
+          )
+          const files = manifest['pages']['/other'].filter((e) =>
+            e.endsWith('.css')
+          )
+          if (files.length < 1) throw new Error()
+          await Promise.all(files.map((f) => remove(join(appDir, '.next', f))))
+        })
+        afterAll(async () => {
+          await killApp(app)
+        })
+
+        tests()
+      }
+    )
   })
 
   describe('Page hydrates with CSS and not waiting on dependencies', () => {
@@ -373,34 +382,37 @@ describe('CSS Support', () => {
       })
     }
 
-    describe('Production Mode', () => {
-      beforeAll(async () => {
-        await remove(join(appDir, '.next'))
-      })
-      beforeAll(async () => {
-        await nextBuild(appDir, [], {})
-        appPort = await findPort()
-        app = await nextStart(appDir, appPort)
-
-        const buildId = (
-          await readFile(join(appDir, '.next', 'BUILD_ID'), 'utf8')
-        ).trim()
-        const fileName = join(
-          appDir,
-          '.next/static/',
-          buildId,
-          '_buildManifest.js'
-        )
-        if (!(await pathExists(fileName))) {
-          throw new Error('Missing build manifest')
-        }
-        await remove(fileName)
-      })
-      afterAll(async () => {
-        await killApp(app)
-      })
-
-      tests()
-    })
+    ;(process.env.TURBOPACK ? describe.skip : describe)(
+      'production mode',
+      () => {
+        beforeAll(async () => {
+          await remove(join(appDir, '.next'))
+        })
+        beforeAll(async () => {
+          await nextBuild(appDir, [], {})
+          appPort = await findPort()
+          app = await nextStart(appDir, appPort)
+
+          const buildId = (
+            await readFile(join(appDir, '.next', 'BUILD_ID'), 'utf8')
+          ).trim()
+          const fileName = join(
+            appDir,
+            '.next/static/',
+            buildId,
+            '_buildManifest.js'
+          )
+          if (!(await pathExists(fileName))) {
+            throw new Error('Missing build manifest')
+          }
+          await remove(fileName)
+        })
+        afterAll(async () => {
+          await killApp(app)
+        })
+
+        tests()
+      }
+    )
   })
 })
diff --git a/test/integration/custom-error/test/index.test.js b/test/integration/custom-error/test/index.test.js
@@ -79,8 +79,7 @@ describe('Custom _error', () => {
       expect(html).toContain('An error 404 occurred on server')
     })
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     let buildOutput = ''
 
     beforeAll(async () => {

diff --git a/test/integration/custom-page-extension/test/index.test.js b/test/integration/custom-page-extension/test/index.test.js
@@ -35,8 +35,7 @@ describe('Custom page extension', () => {
     afterAll(() => killApp(app))
     runTests()
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/custom-routes-catchall/test/index.test.js b/test/integration/custom-routes-catchall/test/index.test.js
@@ -53,8 +53,7 @@ describe('Custom routes', () => {
     afterAll(() => killApp(app))
     runTests(true)
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/custom-routes-i18n-index-redirect/test/index.test.js b/test/integration/custom-routes-i18n-index-redirect/test/index.test.js
@@ -69,8 +69,7 @@ describe('Custom routes i18n with index redirect', () => {
     afterAll(() => killApp(app))
     runTests(true)
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/custom-routes-i18n/test/index.test.js b/test/integration/custom-routes-i18n/test/index.test.js
@@ -176,8 +176,7 @@ describe('Custom routes i18n', () => {
     afterAll(() => killApp(app))
     runTests(true)
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await nextBuild(appDir)
       appPort = await findPort()

diff --git a/test/integration/custom-routes/test/index.test.js b/test/integration/custom-routes/test/index.test.js
@@ -2970,9 +2970,11 @@ describe('Custom routes', () => {
     describe('dev mode', () => {
       runSoloTests(true)
     })
-
-    describe('production mode', () => {
-      runSoloTests()
-    })
+    ;(process.env.TURBOPACK ? describe.skip : describe)(
+      'production mode',
+      () => {
+        runSoloTests()
+      }
+    )
   })
 })
diff --git a/test/integration/disable-js/test/index.test.js b/test/integration/disable-js/test/index.test.js
@@ -18,7 +18,7 @@ let app
 const context = {}
 
 describe('disabled runtime JS', () => {
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       appPort = await findPort()
 

diff --git a/test/integration/document-file-dependencies/test/index.test.js b/test/integration/document-file-dependencies/test/index.test.js
@@ -10,7 +10,7 @@ let appPort
 let app
 
 describe('File Dependencies', () => {
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       appPort = await findPort()
       await nextBuild(appDir)

diff --git a/test/integration/dynamic-optional-routing-root-fallback/test/index.test.js b/test/integration/dynamic-optional-routing-root-fallback/test/index.test.js
@@ -62,8 +62,7 @@ describe('Dynamic Optional Routing Root Fallback', () => {
 
     runTests()
   })
-
-  describe('production mode', () => {
+  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
     beforeAll(async () => {
       await fs.remove(join(appDir, '.next'))