Skip to content

Commit 82eeab4

Browse files
committed
skip test if build turbopack
1 parent 9099e78 commit 82eeab4

File tree

1 file changed

+36
-20
lines changed

1 file changed

+36
-20
lines changed

test/e2e/app-dir/loader-file-named-export-custom-loader-error/loader-file-named-export-custom-loader-error.test.ts

Lines changed: 36 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -10,28 +10,44 @@ async function testDev(browser, errorRegex) {
1010
}
1111

1212
describe('Error test if the loader file export a named function', () => {
13-
const { next, isNextDev } = nextTestSetup({
14-
skipDeployment: true,
15-
skipStart: true,
16-
files: __dirname,
17-
})
18-
19-
if (isNextDev) {
20-
it('should show the error when using `Image` component', async () => {
21-
const browser = await next.browser('/')
22-
await testDev(browser, errorMessage)
13+
describe('in Development', () => {
14+
const { next, isNextDev } = nextTestSetup({
15+
skipDeployment: true,
16+
files: __dirname,
2317
})
2418

25-
it('should show the error when using `getImageProps` method', async () => {
26-
const browser = await next.browser('/get-img-props')
27-
await testDev(browser, errorMessage)
19+
;(isNextDev ? describe : describe.skip)('development only', () => {
20+
it('should show the error when using `Image` component', async () => {
21+
const browser = await next.browser('/')
22+
await testDev(browser, errorMessage)
23+
})
24+
25+
it('should show the error when using `getImageProps` method', async () => {
26+
const browser = await next.browser('/get-img-props')
27+
await testDev(browser, errorMessage)
28+
})
2829
})
29-
} else {
30-
it('should show the build error', async () => {
31-
await expect(next.start()).rejects.toThrow(
32-
'next build failed with code/signal 1'
33-
)
34-
expect(next.cliOutput).toContain(errorMessage)
30+
})
31+
32+
describe('in Build and Start', () => {
33+
const { next, isNextStart } = nextTestSetup({
34+
skipDeployment: true,
35+
skipStart: true,
36+
files: __dirname,
3537
})
36-
}
38+
39+
// next build doesn't support turbopack yet
40+
// see https://nextjs.org/docs/architecture/turbopack#unsupported-features
41+
;(isNextStart && !process.env.TURBOPACK ? describe : describe.skip)(
42+
'build and start only',
43+
() => {
44+
it('should show the build error', async () => {
45+
await expect(next.start()).rejects.toThrow(
46+
'next build failed with code/signal 1'
47+
)
48+
expect(next.cliOutput).toContain(errorMessage)
49+
})
50+
}
51+
)
52+
})
3753
})

0 commit comments

Comments
 (0)