fix(vitest-pool-workers): fix support for query params with repeated keys (#7668)

Author: romeupalos

.changeset/moody-bikes-cross.md

@@ -0,0 +1,7 @@
+---
+"@cloudflare/vitest-pool-workers": patch
+---
+
+fix: Add support interception of URLs with repeated key/name in its query params.
+
+e.g., `https://example.com/foo/bar?a=1&a=2`

fixtures/vitest-pool-workers-examples/misc/test/fetch-mock.test.ts

@@ -32,6 +32,17 @@ it("falls through to global fetch() if unmatched", async () => {
 	expect(await response.text()).toBe("fallthrough:GET https://example.com/bad");
 });
 
+it("intercepts URLs with query parameters with repeated keys", async () => {
+	fetchMock
+		.get("https://example.com")
+		.intercept({ path: "/foo/bar?a=1&a=2" })
+		.reply(200, "body");
+
+	let response = await fetch("https://example.com/foo/bar?a=1&a=2");
+	expect(response.url).toEqual("https://example.com/foo/bar?a=1&a=2");
+	expect(await response.text()).toBe("body");
+});
+
 describe("AbortSignal", () => {
 	let abortSignalTimeoutMock: MockInstance;
 

packages/vitest-pool-workers/src/worker/fetch-mock.ts

@@ -93,11 +93,10 @@ globalThis.fetch = async (input, init) => {
 	const bodyText = bodyArray === null ? "" : DECODER.decode(bodyArray);
 	const dispatchOptions: Dispatcher.DispatchOptions = {
 		origin: url.origin,
-		path: url.pathname,
+		path: url.pathname + url.search,
 		method: request.method as Dispatcher.HttpMethod,
 		body: bodyText,
 		headers: requestHeaders,
-		query: Object.fromEntries(url.searchParams),
 	};
 	requests.set(dispatchOptions, { request, body: bodyArray });