mirror of
https://github.com/vercel/next.js.git
synced 2026-09-20 02:25:18 +08:00
9b8a7a1055
1. Align the code highlight marker color with the issue severity 2. Make the issue title colored 3. Prefix issues with `Warning` or `Error` <img width="1383" height="862" alt="Bildschirmfoto 2026-06-16 um 18 59 03" src="https://github.com/user-attachments/assets/f98c606d-4ea8-40c2-836d-e395f1df904c" /> <img width="1039" height="450" alt="Bildschirmfoto 2026-06-16 um 19 03 39" src="https://github.com/user-attachments/assets/2747f914-0587-4178-a87d-344f24715c96" /> <details> <summary>Old</summary> <img width="1267" height="745" alt="Bildschirmfoto 2026-06-16 um 17 04 35" src="https://github.com/user-attachments/assets/837739ff-82f7-4ea2-8767-ca7d866a8570" /> </details>
143 lines
4.3 KiB
TypeScript
143 lines
4.3 KiB
TypeScript
import { nextTestSetup } from 'e2e-utils'
|
|
import { join } from 'node:path'
|
|
import { writeFile } from 'node:fs/promises'
|
|
|
|
const errorMessage = `This function is what Next.js runs for every request handled by this proxy (previously called middleware).
|
|
|
|
Why this happens:
|
|
- You are migrating from \`middleware\` to \`proxy\`, but haven't updated the exported function.
|
|
- The file exists but doesn't export a function.
|
|
- The export is not a function (e.g., an object or constant).
|
|
- There's a syntax error preventing the export from being recognized.
|
|
|
|
To fix it:
|
|
- Ensure this file has either a default or "proxy" function export.
|
|
|
|
Learn more: https://nextjs.org/docs/messages/middleware-to-proxy`
|
|
|
|
describe('proxy-missing-export', () => {
|
|
const { next, isNextDev, skipped } = nextTestSetup({
|
|
files: __dirname,
|
|
skipDeployment: true,
|
|
skipStart: true,
|
|
})
|
|
|
|
if (skipped) {
|
|
return
|
|
}
|
|
|
|
it('should error when proxy file has invalid export named middleware', async () => {
|
|
await writeFile(
|
|
join(next.testDir, 'proxy.ts'),
|
|
'export function middleware() {}'
|
|
)
|
|
|
|
let cliOutput: string
|
|
|
|
if (isNextDev) {
|
|
await next.start().catch(() => {})
|
|
// Use .catch() because Turbopack errors during compile and exits before runtime.
|
|
await next.browser('/').catch(() => {})
|
|
cliOutput = next.cliOutput
|
|
} else {
|
|
cliOutput = (await next.build()).cliOutput
|
|
}
|
|
|
|
// TODO: Investigate why in dev-turbo, the error is shown in the browser console, not CLI output.
|
|
if (process.env.IS_TURBOPACK_TEST && !isNextDev) {
|
|
expect(cliOutput).toContain(`./proxy.ts
|
|
Error: Proxy is missing expected function export name
|
|
${errorMessage}`)
|
|
} else {
|
|
expect(cliOutput)
|
|
.toContain(`The file "./proxy.ts" must export a function, either as a default export or as a named "proxy" export.
|
|
${errorMessage}`)
|
|
}
|
|
|
|
await next.stop()
|
|
})
|
|
|
|
it('should NOT error when proxy file has a default function export', async () => {
|
|
await writeFile(
|
|
join(next.testDir, 'proxy.ts'),
|
|
'export default function handler() {}'
|
|
)
|
|
|
|
await next.start()
|
|
|
|
const browser = await next.browser('/')
|
|
expect(await browser.elementByCss('p').text()).toBe('hello world')
|
|
|
|
await next.stop()
|
|
})
|
|
|
|
it('should NOT error when proxy file has a default arrow function export', async () => {
|
|
await writeFile(join(next.testDir, 'proxy.ts'), 'export default () => {}')
|
|
|
|
await next.start()
|
|
|
|
const browser = await next.browser('/')
|
|
expect(await browser.elementByCss('p').text()).toBe('hello world')
|
|
|
|
await next.stop()
|
|
})
|
|
|
|
it('should NOT error when proxy file has a named declaration function export', async () => {
|
|
await writeFile(
|
|
join(next.testDir, 'proxy.ts'),
|
|
'const proxy = function() {}; export { proxy };'
|
|
)
|
|
|
|
await next.start()
|
|
|
|
const browser = await next.browser('/')
|
|
expect(await browser.elementByCss('p').text()).toBe('hello world')
|
|
|
|
await next.stop()
|
|
})
|
|
|
|
it('should NOT error when proxy file has a named declaration arrow function export', async () => {
|
|
await writeFile(
|
|
join(next.testDir, 'proxy.ts'),
|
|
'const proxy = () => {}; export { proxy };'
|
|
)
|
|
|
|
await next.start()
|
|
|
|
const browser = await next.browser('/')
|
|
expect(await browser.elementByCss('p').text()).toBe('hello world')
|
|
|
|
await next.stop()
|
|
})
|
|
|
|
it('should error when proxy file has a named export with different name alias', async () => {
|
|
await writeFile(
|
|
join(next.testDir, 'proxy.ts'),
|
|
'const proxy = () => {}; export { proxy as handler };'
|
|
)
|
|
|
|
let cliOutput: string
|
|
|
|
if (isNextDev) {
|
|
await next.start().catch(() => {})
|
|
// Use .catch() because Turbopack errors during compile and exits before runtime.
|
|
await next.browser('/').catch(() => {})
|
|
cliOutput = next.cliOutput
|
|
} else {
|
|
cliOutput = (await next.build()).cliOutput
|
|
}
|
|
|
|
// TODO: Investigate why in dev-turbo, the error is shown in the browser console, not CLI output.
|
|
if (process.env.IS_TURBOPACK_TEST && !isNextDev) {
|
|
expect(cliOutput).toContain(`./proxy.ts
|
|
Error: Proxy is missing expected function export name
|
|
${errorMessage}`)
|
|
} else {
|
|
expect(cliOutput)
|
|
.toContain(`The file "./proxy.ts" must export a function, either as a default export or as a named "proxy" export.
|
|
${errorMessage}`)
|
|
}
|
|
await next.stop()
|
|
})
|
|
})
|