test: ensure cookies are set when redirecting

Author: lubieowoce

test/e2e/app-dir/actions/app-action.test.ts

@@ -175,6 +175,27 @@ describe('app-dir action handling', () => {
     })
   })
 
+  it.each([
+    { description: 'with javascript', disableJavaScript: false },
+    { description: 'no javascript', disableJavaScript: true },
+  ])(
+    'should support setting cookies when redirecting ($description)',
+    async ({ disableJavaScript }) => {
+      const browser = await next.browser('/mutate-cookie-with-redirect', {
+        disableJavaScript,
+      })
+      expect(await browser.elementByCss('#value').text()).toBe('')
+
+      await browser.elementByCss('#update-cookie').click()
+      await browser.elementByCss('#redirect-target')
+
+      expect(await browser.elementByCss('#value').text()).toMatch(/\d+/)
+      expect(await browser.eval('document.cookie')).toMatch(
+        /(?:^|(?:; ))testCookie=\d+/
+      )
+    }
+  )
+
   it('should push new route when redirecting', async () => {
     const browser = await next.browser('/header')
 

test/e2e/app-dir/actions/app/mutate-cookie-with-redirect/page.js

@@ -0,0 +1,21 @@
+import { cookies } from 'next/headers'
+import { redirect } from 'next/navigation'
+
+async function updateCookieAndRedirect() {
+  'use server'
+  ;(await cookies()).set('testCookie', Date.now())
+  redirect('/mutate-cookie-with-redirect/redirect-target')
+}
+
+export default async function Page() {
+  return (
+    <>
+      <p id="value">{(await cookies()).get('testCookie')?.value ?? null}</p>
+      <form action={updateCookieAndRedirect}>
+        <button id="update-cookie" type="submit">
+          Update Cookie
+        </button>
+      </form>
+    </>
+  )
+}

test/e2e/app-dir/actions/app/mutate-cookie-with-redirect/redirect-target/page.js

@@ -0,0 +1,9 @@
+import { cookies } from 'next/headers'
+
+export default async function Page() {
+  return (
+    <div id="redirect-target">
+      <p id="value">{(await cookies()).get('testCookie')?.value ?? null}</p>
+    </div>
+  )
+}