fix: fix failure to spy in browsers

wheresrhys · Jan 28, 2025 · bfaa5f3 · bfaa5f3
1 parent cb429dd
commit bfaa5f3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/packages/fetch-mock/src/FetchMock.ts b/packages/fetch-mock/src/FetchMock.ts
@@ -164,10 +164,10 @@ export class FetchMock {
 	): FetchMock {
 		if (matcher) {
 			//@ts-expect-error TODO findo out how to overload an overload
-			this.route(matcher, ({ args }) => this.config.fetch(...args), name);
+			this.route(matcher, ({ args }) => this.config.fetch.bind(globalThis)(...args), name);
 		} else {
 			//@ts-expect-error TODO findo out how to overload an overload
-			this.catch(({ args }) => this.config.fetch(...args));
+			this.catch(({ args }) => this.config.fetch.bind(globalThis)(...args));
 		}
 
 		return this;

diff --git a/packages/fetch-mock/src/__tests__/FetchMock/mock-and-spy.test.js b/packages/fetch-mock/src/__tests__/FetchMock/mock-and-spy.test.js
@@ -135,12 +135,12 @@ describe('mock and spy', () => {
 				method: 'post',
 			});
 		});
-
-		it('can call actual native fetch without erroring', async () => {
-			fm.spyGlobal();
-			await expect(
-				fm.fetchHandler('http://example.com/'),
-			).resolves.toBeInstanceOf(Response);
-		});
+		const isBrowser = Boolean(globalThis.location);
+		if (isBrowser) {
+			it('can call actual native fetch without erroring', async () => {
+				fm.spyGlobal();
+				await expect(fm.fetchHandler('/')).resolves.toBeInstanceOf(Response);
+			});
+		}
 	});
 });