fix(request) handle undefined/null/empty signal on request (#5503)

* handle undefined/null/empty signal on request

* better approach

1 files changed, 37 insertions, 0 deletions

diff --git a/test/js/web/request/request.test.ts b/test/js/web/request/request.test.ts
new file mode 100644
index 000000000..ac6008eb4
--- /dev/null
+++ b/test/js/web/request/request.test.ts
@@ -0,0 +1,37 @@
+import { test, expect } from "bun:test";
+
+test("request can receive undefined signal", async () => {
+  const request = new Request("http://example.com/", {
+    method: "POST",
+    headers: {
+      "Content-Type": "text/bun;charset=utf-8",
+    },
+    body: "bun",
+    signal: undefined,
+  });
+  expect(request.method).toBe("POST");
+  // @ts-ignore
+  const clone = new Request(request);
+  expect(clone.method).toBe("POST");
+  expect(clone.headers.get("content-type")).toBe("text/bun;charset=utf-8");
+  expect(await request.text()).toBe("bun");
+  expect(await clone.text()).toBe("bun");
+});
+
+test("request can receive null signal", async () => {
+  const request = new Request("http://example.com/", {
+    method: "POST",
+    headers: {
+      "Content-Type": "text/bun;charset=utf-8",
+    },
+    body: "bun",
+    signal: null,
+  });
+  expect(request.method).toBe("POST");
+  // @ts-ignore
+  const clone = new Request(request);
+  expect(clone.method).toBe("POST");
+  expect(clone.headers.get("content-type")).toBe("text/bun;charset=utf-8");
+  expect(await request.text()).toBe("bun");
+  expect(await clone.text()).toBe("bun");
+});