body option is ignored for get requests

Author: leggsimon

src/lib/generate-matcher.js

@@ -74,7 +74,12 @@ const getFunctionMatcher = ({ matcher, functionMatcher = () => true }) =>
 	typeof matcher === 'function' ? matcher : functionMatcher;
 
 const getBodyMatcher = ({ body: expectedBody }) => {
-	return (url, { headers = {}, body }) => {
+	return (url, { headers = {}, body, method = 'get' }) => {
+		if (method.toLowerCase() === 'get') {
+			// GET requests don’t send a body so the body matcher should be ignored for them
+			return true;
+		}
+
 		const lowerCaseHeaders = headerUtils.toLowerCase(
 			headerUtils.normalize(headers)
 		);

test/specs/routing.test.js

@@ -622,6 +622,18 @@ module.exports = fetchMock => {
 					});
 					expect(fm.calls(true).length).to.equal(1);
 				});
+
+				it('should ignore the body option matcher if request was GET', async () => {
+					fm.mock('http://it.at.there/', 200, {
+						body: {
+							foo: 'bar',
+							baz: 'qux'
+						}
+					}).catch();
+
+					await fm.fetchHandler('http://it.at.there/');
+					expect(fm.calls(true).length).to.equal(1);
+				});
 			});
 		});