Some checks failed
Test examples / Test Examples (20) (push) Has been cancelled
Test examples / Test Examples (22) (push) Has been cancelled
Lock Threads / action (push) Has been cancelled
Trigger Release / start (push) Has been cancelled
Stale issue handler / stale (push) Has been cancelled
Update Font Data / create-pull-request (push) Has been cancelled
build-and-deploy / deploy-target (push) Has been cancelled
build-and-deploy / build (push) Has been cancelled
build-and-deploy / stable - aarch64-unknown-linux-musl - node@16 (push) Has been cancelled
build-and-deploy / stable - x86_64-unknown-linux-musl - node@16 (push) Has been cancelled
build-and-deploy / stable - aarch64-unknown-linux-gnu - node@16 (push) Has been cancelled
build-and-deploy / stable - x86_64-unknown-linux-gnu - node@16 (push) Has been cancelled
build-and-deploy / stable - aarch64-pc-windows-msvc - node@16 (push) Has been cancelled
build-and-deploy / stable - x86_64-pc-windows-msvc - node@16 (push) Has been cancelled
build-and-deploy / stable - aarch64-apple-darwin - node@16 (push) Has been cancelled
build-and-deploy / stable - x86_64-apple-darwin - node@16 (push) Has been cancelled
build-and-deploy / build-wasm (nodejs) (push) Has been cancelled
build-and-deploy / build-wasm (web) (push) Has been cancelled
build-and-deploy / Deploy preview tarball (push) Has been cancelled
build-and-deploy / Potentially publish release (push) Has been cancelled
build-and-deploy / publish-turbopack-npm-packages (push) Has been cancelled
build-and-deploy / Deploy examples (push) Has been cancelled
build-and-deploy / thank you, build (push) Has been cancelled
build-and-deploy / Upload Turbopack Bytesize metrics to Datadog (push) Has been cancelled
Rspack Next.js development integration tests / Rspack integration tests (push) Has been cancelled
Rspack Next.js production integration tests / Rspack integration tests (push) Has been cancelled
Turbopack Next.js development integration tests / Next.js integration tests (push) Has been cancelled
Turbopack Next.js production integration tests / Next.js integration tests (push) Has been cancelled
Update Rspack test manifest / Update and upload Rspack development test manifest (push) Has been cancelled
Update Rspack test manifest / Update and upload Rspack production test manifest (push) Has been cancelled
Upload bundler test manifests to areweturboyet.com / Upload test results (push) Has been cancelled
Update React / create-pull-request (push) Has been cancelled
test-e2e-project-reset-cron / reset-test-project (push) Has been cancelled
Notify about the top 15 issues/PRs/feature requests (most reacted) in the last 90 days / run (push) Has been cancelled
140 lines
4.9 KiB
TypeScript
140 lines
4.9 KiB
TypeScript
import { nextTestSetup } from 'e2e-utils'
|
||
import {
|
||
waitForRedbox,
|
||
waitForNoRedbox,
|
||
getRedboxSource,
|
||
retry,
|
||
} from 'next-test-utils'
|
||
|
||
describe('app-dir - error-on-next-codemod-comment', () => {
|
||
const { next, isNextDev } = nextTestSetup({
|
||
files: __dirname,
|
||
skipStart: true,
|
||
skipDeployment: true,
|
||
})
|
||
|
||
if (isNextDev) {
|
||
beforeAll(async () => {
|
||
await next.start()
|
||
})
|
||
|
||
it('should error with swc if you have codemod comments left', async () => {
|
||
const browser = await next.browser('/')
|
||
|
||
await waitForRedbox(browser)
|
||
|
||
if (process.env.IS_TURBOPACK_TEST) {
|
||
expect(await getRedboxSource(browser)).toMatchInlineSnapshot(`
|
||
"./app/page.tsx (2:2)
|
||
You have an unresolved @next/codemod comment "remove jsx of next line" that needs review.
|
||
After review, either remove the comment if you made the necessary changes or replace "@next-codemod-error" with "@next-codemod-ignore" to bypass the build error if no action at this line can be taken.
|
||
1 | export default function Page() {
|
||
> 2 | // @next-codemod-error remove jsx of next line
|
||
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
||
3 | return <p>hello world</p>
|
||
4 | }
|
||
5 |
|
||
|
||
Ecmascript file had an error"
|
||
`)
|
||
} else if (process.env.NEXT_RSPACK) {
|
||
expect(await getRedboxSource(browser)).toMatchInlineSnapshot(`
|
||
"./app/page.tsx
|
||
╰─▶ × Error: x You have an unresolved @next/codemod comment "remove jsx of next line" that needs review.
|
||
│ | After review, either remove the comment if you made the necessary changes or replace "@next-codemod-error" with "@next-codemod-ignore" to bypass the build error if no action at this line can be taken.
|
||
│
|
||
│ ,-[2:1]
|
||
│ 1 | export default function Page() {
|
||
│ 2 | // @next-codemod-error remove jsx of next line
|
||
│ : ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
||
│ 3 | return <p>hello world</p>
|
||
│ 4 | }
|
||
│ \`----
|
||
│"
|
||
`)
|
||
} else {
|
||
expect(await getRedboxSource(browser)).toMatchInlineSnapshot(`
|
||
"./app/page.tsx
|
||
Error: x You have an unresolved @next/codemod comment "remove jsx of next line" that needs review.
|
||
| After review, either remove the comment if you made the necessary changes or replace "@next-codemod-error" with "@next-codemod-ignore" to bypass the build error if no action at this line can be taken.
|
||
|
||
,-[2:1]
|
||
1 | export default function Page() {
|
||
2 | // @next-codemod-error remove jsx of next line
|
||
: ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
||
3 | return <p>hello world</p>
|
||
4 | }
|
||
\`----"
|
||
`)
|
||
}
|
||
})
|
||
|
||
it('should error with inline comment as well', async () => {
|
||
let originFileContent
|
||
await next.patchFile('app/page.tsx', (code) => {
|
||
originFileContent = code
|
||
return code.replace(
|
||
'// @next-codemod-error remove jsx of next line',
|
||
'/* @next-codemod-error remove jsx of next line */'
|
||
)
|
||
})
|
||
|
||
const browser = await next.browser('/')
|
||
|
||
await waitForRedbox(browser)
|
||
|
||
// Recover the original file content
|
||
await next.patchFile('app/page.tsx', originFileContent)
|
||
})
|
||
|
||
it('should disappear the error when you rre the codemod comment', async () => {
|
||
const browser = await next.browser('/')
|
||
|
||
await waitForRedbox(browser)
|
||
|
||
let originFileContent
|
||
await next.patchFile('app/page.tsx', (code) => {
|
||
originFileContent = code
|
||
return code.replace(
|
||
'// @next-codemod-error remove jsx of next line',
|
||
''
|
||
)
|
||
})
|
||
|
||
await retry(async () => {
|
||
await waitForNoRedbox(browser)
|
||
})
|
||
|
||
// Recover the original file content
|
||
await next.patchFile('app/page.tsx', originFileContent)
|
||
})
|
||
|
||
it('should disappear the error when you replace with bypass comment', async () => {
|
||
const browser = await next.browser('/')
|
||
|
||
await waitForRedbox(browser)
|
||
|
||
let originFileContent
|
||
await next.patchFile('app/page.tsx', (code) => {
|
||
originFileContent = code
|
||
return code.replace('@next-codemod-error', '@next-codemod-bypass')
|
||
})
|
||
|
||
await retry(async () => {
|
||
await waitForNoRedbox(browser)
|
||
})
|
||
|
||
// Recover the original file content
|
||
await next.patchFile('app/page.tsx', originFileContent)
|
||
})
|
||
} else {
|
||
it('should fail the build with next build', async () => {
|
||
const res = await next.build()
|
||
expect(res.exitCode).toBe(1)
|
||
expect(res.cliOutput).toContain(
|
||
'You have an unresolved @next/codemod comment "remove jsx of next line" that needs review.'
|
||
)
|
||
})
|
||
}
|
||
})
|