add header merging tests with and without redirect

Author: Brendonovich

apps/tests/src/e2e/api-call.test.ts

@@ -5,4 +5,18 @@ test.describe("api calls", () => {
     const response = await fetch("http://localhost:3000/api/text-plain");
     expect(await response.text()).toBe("test");
   });
+
+  test("should include headers from both event and returned request", async () => {
+    const okResp = await fetch("http://localhost:3000/api/header-merging?status=ok");
+    expect(okResp.headers.get("Set-Cookie")).toBeTruthy();
+    expect(okResp.headers.get("x-event-header")).toBe("value");
+    expect(okResp.headers.get("x-return-header")).toBe("value");
+    expect(okResp.headers.get("x-shared-header")).toBe("event");
+
+    const redirectResp = await fetch("http://localhost:3000/api/header-merging?status=redirect", { redirect: "manual" });
+    expect(redirectResp.headers.get("Set-Cookie")).toBeTruthy();
+    expect(redirectResp.headers.get("x-event-header")).toBe("value");
+    expect(redirectResp.headers.get("x-return-header")).toBe("value");
+    expect(redirectResp.headers.get("x-shared-header")).toBe("event");
+  })
 });

apps/tests/src/routes/api/header-merging.ts

@@ -0,0 +1,29 @@
+import { getRequestURL, setHeader, useSession } from "@solidjs/start/http";
+
+export async function GET() {
+  const url = getRequestURL();
+
+  const s = await useSession({ password: "0".repeat(32) });
+  await s.update(d => ({count: (d.count || 0) + 1}))
+
+  setHeader("x-event-header", "value");
+  setHeader("x-shared-header", "event");
+
+  if(url.searchParams.get("status") === "redirect") {
+    return new Response(null, {
+      status: 301,
+      headers: {
+        location: "http://::/abc",
+        "x-return-header": "value",
+        "x-shared-header": "return"
+      }
+    })
+  } else {
+    return new Response(null, {
+      headers: {
+        "x-return-header": "value",
+        "x-shared-header": "return"
+      }
+    })
+  }
+}