Fix Request parameter being ignored by client methods

Author: jthacker

packages/openapi-fetch/src/index.js

@@ -109,7 +109,7 @@ export default function createClient(clientOptions) {
 
     let id;
     let options;
-    let request = new CustomRequest(
+    let request = new Request(
       createFinalURL(schemaPath, { baseUrl: finalBaseUrl, params, querySerializer }),
       requestInit,
     );

packages/openapi-fetch/test/common/create-client.test.ts

@@ -57,6 +57,47 @@ describe("createClient options", () => {
     expect(actualURL.href).toBe("https://api.foo.bar/v2/resources");
   });
 
+  test("Request option is used", async () => {
+    let customRequestCalled = false;
+
+    const createCustomRequest = () => {
+      return class extends Request {
+        constructor(input: URL | RequestInfo, init?: RequestInit) {
+          super(input, init);
+          customRequestCalled = true;
+        }
+      };
+    };
+
+    const client = createObservedClient<paths>(
+      { baseUrl: "https://api.foo.bar/v2", Request: createCustomRequest() },
+      async () => Response.json([]),
+    );
+
+    await client.GET("/resources");
+
+    expect(customRequestCalled).toBe(true);
+  });
+
+  test("Request option is also used when fetching ", async () => {
+    let customRequestCalled = false;
+
+    const createCustomRequest = () => {
+      return class extends Request {
+        constructor(input: URL | RequestInfo, init?: RequestInit) {
+          super(input, init);
+          customRequestCalled = true;
+        }
+      };
+    };
+
+    const client = createObservedClient<paths>({ baseUrl: "https://api.foo.bar/v2" }, async () => Response.json([]));
+
+    await client.GET("/resources", { Request: createCustomRequest() });
+
+    expect(customRequestCalled).toBe(true);
+  });
+
   describe("content-type", () => {
     const BODY_ACCEPTING_METHODS = [["PUT"], ["POST"], ["DELETE"], ["OPTIONS"], ["PATCH"]] as const;
     const ALL_METHODS = [...BODY_ACCEPTING_METHODS, ["GET"], ["HEAD"]] as const;