test: added tests for chaining modifyRoute

Author: wheresrhys

packages/fetch-mock/src/Router.ts

@@ -14,13 +14,6 @@ import { FetchMockConfig } from './FetchMock.js';
 import { hasCredentialsInUrl } from './RequestUtils.js';
 import type { CallLog } from './CallHistory.js';
 
-// const nullablePerson: Nullable<Person> = {
-//   name: 'Daffodil',
-//   email: null,
-//   age: null,
-//   admin: true,
-// };
-
 export type ResponseConfigProp =
 	| 'body'
 	| 'headers'

packages/fetch-mock/src/__tests__/FetchMock/mock-and-spy.test.js

@@ -141,11 +141,6 @@ describe('mock and spy', () => {
 			const isBrowser = Boolean(globalThis.location);
 			// avoids getting caught by a cors error
 			const testUrl = isBrowser ? '/' : 'http://example.com/';
-			try {
-				await fm.fetchHandler('testUrl');
-			} catch (err) {
-				console.log(err);
-			}
 			await expect(fm.fetchHandler(testUrl)).resolves.toBeInstanceOf(Response);
 		});
 	});

packages/fetch-mock/src/__tests__/FetchMock/routing.test.js

@@ -251,7 +251,12 @@ describe('Routing', () => {
 		});
 	});
 	describe('modifyRoute', () => {
-		// testChainableMethod(`modifyRoute`);
+		describe('chainability', () => {
+			beforeEach(() => {
+				fm.route('http://a.com', 200, 'named');
+			});
+			testChainableMethod(`modifyRoute`, 'named', { response: 200 });
+		});
 		it('can modify a matcher', async () => {
 			fm.route('http://a.com/', 200, 'named');
 			fm.modifyRoute('named', {