mirror of
https://github.com/hanzoai/next.js.git
synced 2026-09-24 17:00:51 +00:00
- rename `unstable_after` -> `after` - remove `experimental.after` flag (and warn if it's present) i separated the big find-and-replace into a separate commit so that it's easier to see what else changed apart from that. does not touch docs, that's in #73038
124 lines
4.0 KiB
TypeScript
124 lines
4.0 KiB
TypeScript
/* eslint-env jest */
|
|
import { nextTestSetup } from 'e2e-utils'
|
|
import { retry } from 'next-test-utils'
|
|
|
|
describe('setting cookies', () => {
|
|
const { next, isNextDeploy, skipped } = nextTestSetup({
|
|
files: __dirname,
|
|
})
|
|
|
|
if (skipped) return
|
|
|
|
let currentCliOutputIndex = 0
|
|
beforeEach(() => {
|
|
resetCliOutput()
|
|
})
|
|
|
|
const getCliOutput = () => {
|
|
if (next.cliOutput.length < currentCliOutputIndex) {
|
|
// cliOutput shrank since we started the test, so something (like a `sandbox`) reset the logs
|
|
currentCliOutputIndex = 0
|
|
}
|
|
return next.cliOutput.slice(currentCliOutputIndex)
|
|
}
|
|
|
|
const resetCliOutput = () => {
|
|
currentCliOutputIndex = next.cliOutput.length
|
|
}
|
|
|
|
const EXPECTED_ERROR =
|
|
/Cookies can only be modified in a Server Action or Route Handler\./
|
|
|
|
const EXPECTED_ERROR_IN_AFTER =
|
|
/An error occurred in a function passed to `after\(\)`: .+?: Cookies can only be modified in a Server Action or Route Handler\./
|
|
|
|
describe('stops cookie mutations when changing phases', () => {
|
|
it('from an action to a page render', async () => {
|
|
const path = '/cookies/action-to-render'
|
|
const session = await next.browser(path)
|
|
|
|
const timestamp1 = await session.elementById('timestamp').text()
|
|
// .set() should throw during render
|
|
expect(await session.elementById('canSetCookies').text()).toEqual('false')
|
|
if (!isNextDeploy) {
|
|
expect(getCliOutput()).toMatch(EXPECTED_ERROR)
|
|
}
|
|
// no cookie should be set
|
|
expect(await session.eval('document.cookie')).not.toInclude(
|
|
'illegalCookie'
|
|
)
|
|
|
|
resetCliOutput()
|
|
// trigger an action
|
|
await session.elementByCss('[type="submit"]').click()
|
|
// wait for page to update as a result
|
|
await retry(async () => {
|
|
const timestamp2 = await session.elementById('timestamp').text()
|
|
expect(timestamp2).not.toEqual(timestamp1)
|
|
})
|
|
|
|
// .set() should throw during render
|
|
expect(await session.elementById('canSetCookies').text()).toEqual('false')
|
|
if (!isNextDeploy) {
|
|
expect(getCliOutput()).toMatch(EXPECTED_ERROR)
|
|
}
|
|
|
|
// no cookie should be set
|
|
expect(await session.eval('document.cookie')).not.toInclude(
|
|
'illegalCookie'
|
|
)
|
|
})
|
|
|
|
// these tests inspect CLI logs to see what happened in after,
|
|
// so they won't work in deploy mode
|
|
if (!isNextDeploy) {
|
|
it('from an action to after via closure', async () => {
|
|
const path = '/cookies/action-to-after/via-closure'
|
|
const session = await next.browser(path)
|
|
|
|
// trigger an action
|
|
await session.elementByCss('[type="submit"]').click()
|
|
await retry(async () => {
|
|
// the .set() in after should error
|
|
expect(getCliOutput()).toMatch(EXPECTED_ERROR_IN_AFTER)
|
|
})
|
|
|
|
// no cookie should be set
|
|
expect(await session.eval('document.cookie')).not.toInclude(
|
|
'illegalCookie'
|
|
)
|
|
})
|
|
|
|
it('from a route handler to after via closure', async () => {
|
|
const path = '/cookies/route-handler-to-after/via-closure'
|
|
const response = await next.fetch(path, { method: 'POST' })
|
|
await response.text()
|
|
expect(response.status).toBe(200)
|
|
|
|
// no cookie should be set
|
|
expect(response.headers.get('set-cookie')).toBe(null)
|
|
|
|
// the .set() in after should error
|
|
await retry(async () => {
|
|
expect(getCliOutput()).toMatch(EXPECTED_ERROR_IN_AFTER)
|
|
})
|
|
})
|
|
|
|
it('from middleware to after via closure', async () => {
|
|
const path = '/cookies/middleware-to-after/via-closure'
|
|
const response = await next.fetch(path)
|
|
await response.text()
|
|
expect(response.status).toBe(200)
|
|
|
|
// no cookie should be set
|
|
expect(response.headers.get('set-cookie')).toBe(null)
|
|
|
|
// the .set() in after should error
|
|
await retry(async () => {
|
|
expect(getCliOutput()).toMatch(EXPECTED_ERROR_IN_AFTER)
|
|
})
|
|
})
|
|
}
|
|
})
|
|
})
|