Merge pull request #3 from thefrontside/bugfix

🐛 Fix empty error message for missing required fields

Author: cowboyd

lib/field.ts

@@ -89,9 +89,9 @@ export function field<T>(
         value: undefined as T,
         issues,
       };
+      sources.push(source);
 
       if (!issues) {
-        sources.push(source);
         return {
           ok: true as const,
           value: source.value,

test/field.test.ts

@@ -97,4 +97,11 @@ describe("field", () => {
 
     it.skip("supports custom parsers", () => {});
   });
+
+  it("includes validation issues in the error when no input is provided", () => {
+    let result = parseSync(field(type("number")), {});
+    assert(!result.ok);
+    assert(result.error instanceof ValidationError);
+    expect(result.error.message).toMatch(/must be a number/);
+  });
 });