Merge branch 'issue-2760-tdd-loose' into issue-2760-final-intersection

RobinTail · RobinTail · commit ac8056b638f1 · 2025-06-22T17:25:23.000+02:00
diff --git a/express-zod-api/tests/endpoints-factory.spec.ts b/express-zod-api/tests/endpoints-factory.spec.ts
@@ -1,5 +1,6 @@
 import { RequestHandler } from "express";
 import createHttpError from "http-errors";
+import { expectTypeOf } from "vitest";
 import {
   EndpointsFactory,
   Middleware,
@@ -81,6 +82,28 @@ describe("EndpointsFactory", () => {
         >
       >();
     });
+
+    test("Issue #2760: should strip excessive props by default", () => {
+      defaultEndpointsFactory.build({
+        input: z.object({ foo: z.string() }),
+        output: z.object({ foo: z.string() }),
+        handler: async ({ input }) => {
+          expectTypeOf(input).not.toHaveProperty("bar");
+          return input;
+        },
+      });
+    });
+
+    test("Issue #2760: should allow excessive props when using loose object schema", () => {
+      defaultEndpointsFactory.build({
+        input: z.looseObject({ foo: z.string() }),
+        output: z.object({ foo: z.string() }),
+        handler: async ({ input }) => {
+          expectTypeOf(input).toHaveProperty("bar").toEqualTypeOf<unknown>();
+          return input;
+        },
+      });
+    });
   });
 
   describe(".addOptions()", () => {
