🐛 Fix all validator not merging primitive errors correctly

CHANGELOG.md

@@ -2,7 +2,9 @@
 
 #### 🚀 Enhancement
 
-- :sparkles: Add someOf validator [#12](https://github.com/opencreek/vlad/pull/12) ([@reckter](https://github.com/reckter))
+- :sparkles: Add someOf validator
+  [#12](https://github.com/opencreek/vlad/pull/12)
+  ([@reckter](https://github.com/reckter))
 
 #### Authors: 1
 

src/validators/all.ts

@@ -58,6 +58,10 @@ export function all(...validators) {
             return cur;
           }
 
+          if (Array.isArray(acc) && Array.isArray(cur)) {
+            return [...acc, ...cur];
+          }
+
           return deepMerge(acc, cur);
         },
         undefined,

test/validators/all.test.ts

@@ -1,4 +1,5 @@
 import { all } from "../../src/validators/all.ts";
+import { object } from "../../src/validators/object.ts";
 import {
   allItems,
   is,
@@ -56,3 +57,15 @@ Deno.test("should correctly type check with multiple validators", () => {
 
   assertEquals(a, ["Must have at least 2 items"]);
 });
+
+Deno.test("should merge complicated errors correctly", () => {
+  const validator = all(
+    requiredPrimitive("date required"),
+    (value: number) => {
+      return ["other date error"];
+    },
+  );
+
+  const output = validator(undefined);
+  assertEquals(output, ["date required", "other date error"]);
+});