Add test for multiple cookies coming from middleware (#34465)

This PR adds a test that ensures that we can set multiple cookies in a middleware response
vercel · Feb 18, 2022 · e251a8b · e251a8b
1 parent 271dff5
commit e251a8b
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/test/integration/middleware/core/pages/responses/_middleware.js b/test/integration/middleware/core/pages/responses/_middleware.js
@@ -32,6 +32,15 @@ export async function middleware(request, ev) {
     return next
   }
 
+  if (url.pathname === '/responses/two-cookies') {
+    const headers = new Headers()
+    headers.append('set-cookie', 'foo=chocochip')
+    headers.append('set-cookie', 'bar=chocochip')
+    return new Response('cookies set', {
+      headers,
+    })
+  }
+
   // Streams a basic response
   if (url.pathname === '/responses/stream-a-response') {
     ev.waitUntil(

diff --git a/test/integration/middleware/core/test/index.test.js b/test/integration/middleware/core/test/index.test.js
@@ -473,6 +473,18 @@ function redirectTests(locale = '') {
 }
 
 function responseTests(locale = '') {
+  it(`${locale} responds with multiple cookies`, async () => {
+    const res = await fetchViaHTTP(
+      context.appPort,
+      `${locale}/responses/two-cookies`
+    )
+
+    expect(res.headers.raw()['set-cookie']).toEqual([
+      'foo=chocochip',
+      'bar=chocochip',
+    ])
+  })
+
   it(`${locale} should stream a response`, async () => {
     const res = await fetchViaHTTP(
       context.appPort,