mirror of
https://github.com/hanzoai/next.js.git
synced 2026-10-30 14:46:38 +00:00
Ben GublerandGitHub
41e39426eb
### What? This PR removes the automatic ESLint linting that previously ran during `next build`. The documentation has been updated to reflect this change. ### Why? As part of Next.js 16, we're removing the automatic build-time linting that was previously included in the build process. This gives developers more control over when linting runs by explicitly invoking their linter via npm scripts. ### How? - Updated documentation to clarify that automatic build-time linting has been removed in Next.js 16 - Removed the `runLint` parameter and related linting code from the build process - Removed the `verifyAndLint.ts` file which is no longer needed - Simplified type checking code by removing lint-related functionality - Removed ESLint telemetry during builds
128 lines
4.0 KiB
TypeScript
128 lines
4.0 KiB
TypeScript
import { nextTestSetup } from 'e2e-utils'
|
|
import stripAnsi from 'strip-ansi'
|
|
import { outdent } from 'outdent'
|
|
|
|
describe('production - app dir - build output', () => {
|
|
const { next } = nextTestSetup({
|
|
files: __dirname,
|
|
})
|
|
|
|
let output = ''
|
|
beforeAll(() => {
|
|
output = stripAnsi(next.cliOutput)
|
|
})
|
|
|
|
it('should only log app routes', async () => {
|
|
expect(output).toContain('Route (app)')
|
|
expect(output).not.toContain('Route (pages)')
|
|
expect(output).not.toContain('/favicon.ico')
|
|
})
|
|
|
|
it('should always log version first then the rest jobs', async () => {
|
|
const indexOfVersion = output.indexOf('▲ Next.js')
|
|
const indexOfStartCompiling = output.indexOf(
|
|
'Creating an optimized production build'
|
|
)
|
|
const indexOfLinting = output.indexOf('Running TypeScript')
|
|
expect(indexOfVersion).toBeLessThan(indexOfLinting)
|
|
expect(indexOfStartCompiling).toBeLessThan(indexOfLinting)
|
|
})
|
|
|
|
it('should match the expected output format', async () => {
|
|
// output type
|
|
expect(output).toContain('○ (Static) prerendered as static content')
|
|
})
|
|
|
|
it('should log errors not caught by the worker without terminating the process', async () => {
|
|
expect(output).toContain('Error: Boom')
|
|
expect(output).not.toContain('Next.js build worker exited with code: 78')
|
|
|
|
const $ = await next.render$('/uncaught-error')
|
|
expect($('#sentinel').text()).toEqual('at buildtime')
|
|
})
|
|
|
|
it('should fail the build if you use a dynamic API outside of a render context - cookies', async () => {
|
|
await next.stop()
|
|
await next.patchFile(
|
|
'app/out-of-band-dynamic-api/page.tsx',
|
|
outdent`
|
|
import { cookies } from 'next/headers'
|
|
|
|
export default async function Page() {
|
|
setTimeout(() => {
|
|
cookies();
|
|
}, 0)
|
|
return <div>Hello World</div>
|
|
}
|
|
`
|
|
)
|
|
const { cliOutput } = await next.build()
|
|
await next.deleteFile('app/out-of-band-dynamic-api/page.tsx')
|
|
|
|
expect(cliOutput).toContain('Next.js build worker exited with code: 78')
|
|
})
|
|
|
|
it('should fail the build if you use a dynamic API outside of a render context - headers', async () => {
|
|
await next.stop()
|
|
await next.patchFile(
|
|
'app/out-of-band-dynamic-api/page.tsx',
|
|
outdent`
|
|
import { headers } from 'next/headers'
|
|
|
|
export default async function Page({ searchParams }) {
|
|
setTimeout(() => {
|
|
headers()
|
|
}, 0)
|
|
return <div>Hello World</div>
|
|
}
|
|
`
|
|
)
|
|
const { cliOutput } = await next.build()
|
|
await next.deleteFile('app/out-of-band-dynamic-api/page.tsx')
|
|
|
|
expect(cliOutput).toContain('Next.js build worker exited with code: 78')
|
|
})
|
|
|
|
it('should fail the build if you use a dynamic API outside of a render context - redirect', async () => {
|
|
await next.stop()
|
|
await next.patchFile(
|
|
'app/out-of-band-dynamic-api/page.tsx',
|
|
outdent`
|
|
import { redirect } from 'next/navigation'
|
|
|
|
export default async function Page({ searchParams }) {
|
|
setTimeout(() => {
|
|
redirect('/whatever')
|
|
}, 0)
|
|
return <div>Hello World</div>
|
|
}
|
|
`
|
|
)
|
|
const { cliOutput } = await next.build()
|
|
await next.deleteFile('app/out-of-band-dynamic-api/page.tsx')
|
|
|
|
expect(cliOutput).toContain('Next.js build worker exited with code: 78')
|
|
})
|
|
|
|
it('should fail the build if you use a dynamic API outside of a render context - notFound', async () => {
|
|
await next.stop()
|
|
await next.patchFile(
|
|
'app/out-of-band-dynamic-api/page.tsx',
|
|
outdent`
|
|
import { notFound } from 'next/navigation'
|
|
|
|
export default async function Page({ searchParams }) {
|
|
setTimeout(() => {
|
|
notFound()
|
|
}, 0)
|
|
return <div>Hello World</div>
|
|
}
|
|
`
|
|
)
|
|
const { cliOutput } = await next.build()
|
|
await next.deleteFile('app/out-of-band-dynamic-api/page.tsx')
|
|
|
|
expect(cliOutput).toContain('Next.js build worker exited with code: 78')
|
|
})
|
|
})
|