Files
next.js/test/e2e/trailing-slashes/without-trailing-slash.test.ts
Benjamin WoodruffandGitHub 7049b93e38 chore(CI): Split trailing-slashes suite cases into smaller suites, port to more modern e2e framework (#78890)
Same basic idea as https://github.com/vercel/next.js/pull/78787

This test suite has a lot of timeouts with rspack, which causes it to take a long time to run (though not quite long enough that it breaks the job).

In general, large test suites are good to break up, because they benefit from parallelism.

---

In addition to breaking this test suite up, I also ported it to the new framework, as the way it was patching `next.config.js` was causing problems, since the old integration tests are not run in isolated directories.
2025-05-08 08:38:51 -07:00

81 lines
2.1 KiB
TypeScript

/* eslint-env jest */
/* eslint-disable jest/no-standalone-expect */
import {
testShouldRedirect,
testLinkShouldRewriteTo,
testShouldResolve,
testExternalLinkShouldRewriteTo,
} from './shared-tests.util'
import { nextTestSetup } from 'e2e-utils'
import { join } from 'path'
describe('Trailing slashes with trailingSlash: false', () => {
const { next, isNextStart } = nextTestSetup({
files: __dirname,
nextConfig: {
trailingSlash: false,
},
})
testShouldRedirect(next, [
['/about/', '/about'],
['/catch-all/hello/world/', '/catch-all/hello/world'],
['/catch-all/hello.world/', '/catch-all/hello.world'],
])
testShouldResolve(next, [
// visited url, expected page, expected router path
['/', '/index.js', '/'],
['/about', '/about.js', '/about'],
[
'/catch-all/hello/world',
'/catch-all/[...slug].js',
'/catch-all/[...slug]',
],
['/about?hello=world', '/about.js', '/about'],
])
testLinkShouldRewriteTo(next, [
['/linker?href=/', '/'],
['/linker?href=/about', '/about'],
['/linker?href=/about/', '/about'],
['/linker?href=/about?hello=world', '/about?hello=world'],
['/linker?href=/about/?hello=world', '/about?hello=world'],
['/linker?href=/catch-all/hello/', '/catch-all/hello'],
['/linker?href=/catch-all/hello.world/', '/catch-all/hello.world'],
])
testExternalLinkShouldRewriteTo(next, [
[
`/external-linker?href=${encodeURI('https://nextjs.org')}`,
'https://nextjs.org',
],
[
`/external-linker?href=${encodeURI('https://nextjs.org/')}`,
'https://nextjs.org/',
],
])
// only prod builds have a manifest
;(isNextStart ? it : it.skip)(
'should have a redirect in the routesmanifest',
async () => {
const manifest = await next.readJSON(
join('.next', 'routes-manifest.json')
)
expect(manifest).toEqual(
expect.objectContaining({
redirects: expect.arrayContaining([
expect.objectContaining({
source: '/:path+/',
destination: '/:path+',
statusCode: 308,
}),
]),
})
)
}
)
})