Rename process.env.TURBOPACK -> process.env.TURBOPACK_DEV in test skips

test/e2e/app-dir/ppr-errors/ppr-errors.test.ts

@@ -1,47 +1,50 @@
 import { nextBuild } from 'next-test-utils'
 
 describe('ppr build errors', () => {
-  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
-    let stderr: string
-    let stdout: string
+  ;(process.env.TURBOPACK_DEV ? describe.skip : describe)(
+    'production mode',
+    () => {
+      let stderr: string
+      let stdout: string
 
-    beforeAll(async () => {
-      const output = await nextBuild(__dirname, [], {
-        stderr: true,
-        stdout: true,
+      beforeAll(async () => {
+        const output = await nextBuild(__dirname, [], {
+          stderr: true,
+          stdout: true,
+        })
+        stderr = output.stderr
+        stdout = output.stdout
       })
-      stderr = output.stderr
-      stdout = output.stdout
-    })
 
-    describe('within a suspense boundary', () => {
-      it('should fail the build for uncaught prerender errors', async () => {
-        expect(stderr).toContain(
-          'Error occurred prerendering page "/regular-error-suspense-boundary".'
-        )
-        expect(stderr).toContain(
-          'Error occurred prerendering page "/re-throwing-error".'
-        )
+      describe('within a suspense boundary', () => {
+        it('should fail the build for uncaught prerender errors', async () => {
+          expect(stderr).toContain(
+            'Error occurred prerendering page "/regular-error-suspense-boundary".'
+          )
+          expect(stderr).toContain(
+            'Error occurred prerendering page "/re-throwing-error".'
+          )
+        })
       })
-    })
 
-    describe('outside of a suspense boundary', () => {
-      it('should fail the build for uncaught errors', async () => {
-        expect(stderr).toContain(
-          'Error occurred prerendering page "/regular-error".'
-        )
-        expect(stderr).toContain(
-          'Error occurred prerendering page "/no-suspense-boundary-re-throwing-error".'
-        )
+      describe('outside of a suspense boundary', () => {
+        it('should fail the build for uncaught errors', async () => {
+          expect(stderr).toContain(
+            'Error occurred prerendering page "/regular-error".'
+          )
+          expect(stderr).toContain(
+            'Error occurred prerendering page "/no-suspense-boundary-re-throwing-error".'
+          )
+        })
       })
-    })
 
-    describe('when a postpone call is caught and logged it should', () => {
-      it('should include a message telling why', async () => {
-        expect(stdout).toContain(
-          'User land logged error: Route /logging-error needs to bail out of prerendering at this point because it used cookies.'
-        )
+      describe('when a postpone call is caught and logged it should', () => {
+        it('should include a message telling why', async () => {
+          expect(stdout).toContain(
+            'User land logged error: Route /logging-error needs to bail out of prerendering at this point because it used cookies.'
+          )
+        })
       })
-    })
-  })
+    }
+  )
 })

test/integration/404-page-app/test/index.test.js

@@ -21,43 +21,46 @@ let appPort
 let app
 
 describe('404 Page Support with _app', () => {
-  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
-    afterAll(() => killApp(app))
+  ;(process.env.TURBOPACK_DEV ? describe.skip : describe)(
+    'production mode',
+    () => {
+      afterAll(() => killApp(app))
 
-    it('should build successfully', async () => {
-      const { code, stderr, stdout } = await nextBuild(appDir, [], {
-        stderr: true,
-        stdout: true,
-      })
+      it('should build successfully', async () => {
+        const { code, stderr, stdout } = await nextBuild(appDir, [], {
+          stderr: true,
+          stdout: true,
+        })
 
-      expect(code).toBe(0)
-      expect(stderr).not.toMatch(gip404Err)
-      expect(stdout).not.toMatch(gip404Err)
+        expect(code).toBe(0)
+        expect(stderr).not.toMatch(gip404Err)
+        expect(stdout).not.toMatch(gip404Err)
 
-      appPort = await findPort()
-      app = await nextStart(appDir, appPort)
-    })
+        appPort = await findPort()
+        app = await nextStart(appDir, appPort)
+      })
 
-    it('should not output static 404 if _app has getInitialProps', async () => {
-      const browser = await webdriver(appPort, '/404')
-      const isAutoExported = await browser.eval('__NEXT_DATA__.autoExport')
-      expect(isAutoExported).toBeFalsy()
-    })
+      it('should not output static 404 if _app has getInitialProps', async () => {
+        const browser = await webdriver(appPort, '/404')
+        const isAutoExported = await browser.eval('__NEXT_DATA__.autoExport')
+        expect(isAutoExported).toBeFalsy()
+      })
 
-    it('specify to use the 404 page still in the routes-manifest', async () => {
-      const manifest = await fs.readJSON(
-        join(appDir, '.next/routes-manifest.json')
-      )
-      expect(manifest.pages404).toBe(true)
-    })
+      it('specify to use the 404 page still in the routes-manifest', async () => {
+        const manifest = await fs.readJSON(
+          join(appDir, '.next/routes-manifest.json')
+        )
+        expect(manifest.pages404).toBe(true)
+      })
 
-    it('should still use 404 page', async () => {
-      const res = await fetchViaHTTP(appPort, '/abc')
-      expect(res.status).toBe(404)
-      const $ = cheerio.load(await res.text())
-      expect($('#404-title').text()).toBe('Hi There')
-    })
-  })
+      it('should still use 404 page', async () => {
+        const res = await fetchViaHTTP(appPort, '/abc')
+        expect(res.status).toBe(404)
+        const $ = cheerio.load(await res.text())
+        expect($('#404-title').text()).toBe('Hi There')
+      })
+    }
+  )
 
   describe('dev mode', () => {
     let stderr = ''

test/integration/404-page-custom-error/test/index.test.js

@@ -51,24 +51,27 @@ const runTests = (mode) => {
 }
 
 describe('Default 404 Page with custom _error', () => {
-  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
-    afterAll(() => killApp(app))
+  ;(process.env.TURBOPACK_DEV ? describe.skip : describe)(
+    'production mode',
+    () => {
+      afterAll(() => killApp(app))
 
-    it('should build successfully', async () => {
-      const { code } = await nextBuild(appDir, [], {
-        stderr: true,
-        stdout: true,
-      })
+      it('should build successfully', async () => {
+        const { code } = await nextBuild(appDir, [], {
+          stderr: true,
+          stdout: true,
+        })
 
-      expect(code).toBe(0)
+        expect(code).toBe(0)
 
-      appPort = await findPort()
+        appPort = await findPort()
 
-      app = await nextStart(appDir, appPort)
-    })
+        app = await nextStart(appDir, appPort)
+      })
 
-    runTests('server')
-  })
+      runTests('server')
+    }
+  )
 
   describe('dev mode', () => {
     beforeAll(async () => {

test/integration/404-page-ssg/test/index.test.js

@@ -76,40 +76,43 @@ const runTests = (isDev) => {
 }
 
 describe('404 Page Support SSG', () => {
-  ;(process.env.TURBOPACK ? describe.skip : describe)('production mode', () => {
-    afterAll(() => killApp(app))
-
-    it('should build successfully', async () => {
-      const {
-        code,
-        stderr: buildStderr,
-        stdout: buildStdout,
-      } = await nextBuild(appDir, [], {
-        stderr: true,
-        stdout: true,
+  ;(process.env.TURBOPACK_DEV ? describe.skip : describe)(
+    'production mode',
+    () => {
+      afterAll(() => killApp(app))
+
+      it('should build successfully', async () => {
+        const {
+          code,
+          stderr: buildStderr,
+          stdout: buildStdout,
+        } = await nextBuild(appDir, [], {
+          stderr: true,
+          stdout: true,
+        })
+
+        expect(code).toBe(0)
+        expect(buildStderr).not.toMatch(gip404Err)
+        expect(buildStdout).not.toMatch(gip404Err)
+
+        appPort = await findPort()
+        stderr = ''
+        stdout = ''
+
+        app = await nextStart(appDir, appPort, {
+          onStdout(msg) {
+            stdout += msg
+          },
+          onStderr(msg) {
+            stderr += msg
+          },
+        })
+        buildId = await fs.readFile(join(appDir, '.next/BUILD_ID'), 'utf8')
       })
 
-      expect(code).toBe(0)
-      expect(buildStderr).not.toMatch(gip404Err)
-      expect(buildStdout).not.toMatch(gip404Err)
-
-      appPort = await findPort()
-      stderr = ''
-      stdout = ''
-
-      app = await nextStart(appDir, appPort, {
-        onStdout(msg) {
-          stdout += msg
-        },
-        onStderr(msg) {
-          stderr += msg
-        },
-      })
-      buildId = await fs.readFile(join(appDir, '.next/BUILD_ID'), 'utf8')
-    })
-
-    runTests()
-  })
+      runTests()
+    }
+  )
 
   describe('dev mode', () => {
     beforeAll(async () => {