fix: ensure that mocked fetch responses have a valid URL property (#6362

)

.changeset/proud-chefs-count.md

@@ -0,0 +1,7 @@
+---
+"@cloudflare/vitest-pool-workers": patch
+---
+
+fix: ensure that mocked fetch responses have a valid URL property
+
+Fixes [#6350](https://github.com/cloudflare/workers-sdk/issues/6350)

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

@@ -11,8 +11,10 @@ it("falls through to global fetch() if unmatched", async () => {
 		.reply(200, "body");
 
 	let response = await fetch("https://example.com");
+	expect(response.url).toEqual("https://example.com/");
 	expect(await response.text()).toBe("body");
 
 	response = await fetch("https://example.com/bad");
+	expect(response.url).toEqual("https://example.com/bad");
 	expect(await response.text()).toBe("fallthrough:GET https://example.com/bad");
 });

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

@@ -136,6 +136,7 @@ globalThis.fetch = async (input, init) => {
 					statusText: responseStatusText,
 					headers: responseHeaders,
 				});
+				Object.defineProperty(response, "url", { value: url.href });
 				responseResolve(response);
 			} else {
 				responseResolve(maybeResponse);