Merge pull request #90 from jaredwray/fix-delete-endpoing-allows-string,-JSON,-or-any

fix: delete endpoing allows string, JSON, or any

Author: jaredwray

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@jaredwray/mockhttp",
-	"version": "1.0.0",
+	"version": "1.0.1",
 	"description": "Mock Http - Easy way to mock http with httpbin replacement",
 	"type": "module",
 	"main": "dist/index.js",

src/routes/http-methods/delete.ts

@@ -3,19 +3,6 @@ import type { FastifyInstance, FastifyRequest, FastifySchema } from "fastify";
 const deleteSchema: FastifySchema = {
 	description: "Handles a DELETE request and returns request information",
 	tags: ["HTTP Methods"],
-	body: {
-		oneOf: [
-			{
-				type: "object",
-				additionalProperties: true,
-				description: "JSON body of the DELETE request",
-			},
-			{
-				type: "string",
-				description: "String body of the DELETE request",
-			},
-		],
-	},
 	response: {
 		200: {
 			type: "object",
@@ -26,10 +13,11 @@ const deleteSchema: FastifySchema = {
 					oneOf: [
 						{ type: "object", additionalProperties: true },
 						{ type: "string" },
+						{ type: "null" },
 					],
 				},
 			},
-			required: ["method", "headers", "body"],
+			required: ["method", "headers"],
 		},
 	},
 };

test/routes/http-methods/delete.test.ts

@@ -83,4 +83,17 @@ describe("DELETE /delete", () => {
 		expect(responseBody.headers["content-type"]).toBe("text/plain");
 		expect(responseBody.body).toBe("This is a plain text body");
 	});
+
+	it("should handle no body", async () => {
+		const response = await fastify.inject({
+			method: "DELETE",
+			url: "/delete",
+		});
+
+		expect(response.statusCode).toBe(200);
+
+		const responseBody = response.json();
+		expect(responseBody.method).toBe("DELETE");
+		expect(responseBody.body).toBeUndefined();
+	});
 });