fix: Request without a body should not throw an error (#49)

fixes #48

Author: nzakas

src/util.js

@@ -94,18 +94,27 @@ export function parseUrl(url) {
  * @param {Request} request The request to read the body from.
  * @returns {Promise<string|any|FormData|null>} The body of the request.
  */
-export function getBody(request) {
+export async function getBody(request) {
+	
 	// first get the content type
 	const contentType = request.headers.get("content-type");
 
 	// if there's no content type, there's no body
 	if (!contentType) {
 		return Promise.resolve(null);
+	}	
+	
+	// next try to read the body as text to see if there's a body
+	const text = await request.clone().text();
+	
+	// if there's no body, return null
+	if (!text) {
+		return Promise.resolve(null);
 	}
 
 	// if it's a text format then return a string
 	if (contentType.startsWith("text")) {
-		return request.text();
+		return text;
 	}
 
 	// if the content type is JSON, parse the body as JSON

tests/mock-server.test.js

@@ -563,6 +563,29 @@ describe("MockServer", () => {
 				"application/json",
 			);
 		});
+		
+		it("should return response when JSON request is sent without a body", async () => {
+			server.post("/test", {
+				status: 200,
+				body: { foo: "bar" },
+			});
+
+			const request = createRequest({
+				method: "POST",
+				url: `${BASE_URL}/test`,
+				headers: {
+					"content-type": "application/json",
+				},
+			});
+
+			const { response } = await server.traceReceive(request);
+			assert.strictEqual(response.status, 200);
+			assert.strictEqual(response.statusText, "OK");
+			assert.strictEqual(
+				response.headers.get("content-type"),
+				"application/json",
+			);
+		});
 	});
 
 	describe("Query Strings", () => {