feat: enhance APIClient constructor to accept options for custom fetch implementation

chore: update biome configuration schema to 2.2.2 and remove deprecated settings
test: add test for APIClient constructor with options

Author: sergiodxa

.vscode/settings.json

@@ -18,7 +18,6 @@
 		"editor.defaultFormatter": "biomejs.biome"
 	},
 	"editor.codeActionsOnSave": {
-		"source.organizeImports.biome": "explicit",
-		"quickfix.biome": "explicit"
+		"source.fixAll.biome": "explicit"
 	}
 }

biome.json

@@ -1,8 +1,5 @@
 {
-	"$schema": "https://biomejs.dev/schemas/1.7.1/schema.json",
-	"organizeImports": {
-		"enabled": true
-	},
+	"$schema": "https://biomejs.dev/schemas/2.2.2/schema.json",
 	"linter": {
 		"enabled": true,
 		"rules": {
@@ -22,7 +19,6 @@
 				"useImportType": "off"
 			},
 			"suspicious": {
-				"noConsoleLog": "warn",
 				"noEmptyBlockStatements": "warn",
 				"noSkippedTests": "error"
 			}
@@ -34,11 +30,5 @@
 		"clientKind": "git",
 		"defaultBranch": "main",
 		"useIgnoreFile": true
-	},
-	"overrides": [
-		{
-			"include": ["**/*.md"],
-			"formatter": { "indentStyle": "tab" }
-		}
-	]
+	}
 }

src/index.test.ts

@@ -4,9 +4,10 @@ import {
 	beforeEach,
 	describe,
 	expect,
+	mock,
 	test,
 } from "bun:test";
-import { http, HttpResponse } from "msw";
+import { HttpResponse, http } from "msw";
 import { setupServer } from "msw/native";
 
 import { APIClient } from ".";
@@ -31,6 +32,19 @@ describe(APIClient.name, () => {
 		expect(client).toBeInstanceOf(APIClient);
 	});
 
+	test("#constructor with options", async () => {
+		let customFetch = mock<typeof fetch>().mockImplementation(globalThis.fetch);
+		let client = new APIClient(new URL("https://example.com"), {
+			fetch: customFetch as any as typeof fetch,
+		});
+
+		expect(client).toBeInstanceOf(APIClient);
+		await client.fetch("/path");
+		expect(customFetch).toHaveBeenCalledWith(
+			new Request("https://example.com/path"),
+		);
+	});
+
 	test("#fetch", async () => {
 		let client = new APIClient(new URL("https://example.com"));
 		let response = await client.fetch("/path");

src/index.ts

@@ -23,9 +23,14 @@
 export class APIClient implements APIClient.Type {
 	/** The base URL for all API requests. Must be defined in subclasses. */
 	protected readonly baseURL: URL;
+	protected options: APIClient.Options;
 
-	constructor(baseURL: URL) {
+	constructor(
+		baseURL: URL,
+		options: APIClient.Options = { fetch: globalThis.fetch },
+	) {
 		this.baseURL = baseURL;
+		this.options = options;
 	}
 
 	public interceptors = {
@@ -53,7 +58,7 @@ export class APIClient implements APIClient.Type {
 	 * @param response - The response object that was received.
 	 * @returns A promise that resolves to the (possibly processed) response.
 	 */
-	protected async after(request: Request, response: Response) {
+	protected async after(_request: Request, response: Response) {
 		return response;
 	}
 
@@ -74,7 +79,7 @@ export class APIClient implements APIClient.Type {
 			request = await listener(request);
 		}
 
-		let response = await fetch(request);
+		let response = await this.options.fetch(request);
 		response = await this.after(request, response);
 		for (let listener of this.interceptors.after.listeners) {
 			response = await listener(request, response);
@@ -179,4 +184,8 @@ export namespace APIClient {
 		patch(path: string, init?: Omit<RequestInit, "method">): Promise<Response>;
 		delete(path: string, init?: Omit<RequestInit, "method">): Promise<Response>;
 	}
+
+	export interface Options {
+		fetch: typeof globalThis.fetch;
+	}
 }