nonNullable disallows undefined

colinhacks · Apr 2, 2024 · 82eddeb · 82eddeb
1 parent 991750c
commit 82eddeb
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 2 deletions.
diff --git a/deno/lib/__tests__/nonNullable.test.ts b/deno/lib/__tests__/nonNullable.test.ts
@@ -26,6 +26,14 @@ test("fails is passed null", () => {
  }).toThrow(z.ZodError)
 });
 
+test("fails is passed undefined", () => {
+ const a = z.nullable(z.number()).nonNullable();
+
+ expect(() => {
+ a.parse(undefined);
+ }).toThrow(z.ZodError)
+});
+
 test("unwrap", () => {
  const unwrapped = z.any().nonNullable().unwrap();
  expect(unwrapped).toBeInstanceOf(z.ZodAny);

diff --git a/deno/lib/types.ts b/deno/lib/types.ts
@@ -4480,7 +4480,10 @@ export class ZodNonNullable<T extends ZodTypeAny> extends ZodType<
 > {
  _parse(input: ParseInput): ParseReturnType<this["_output"]> {
  const parsedType = this._getType(input);
- if (parsedType === ZodParsedType.null) {
+ if (
+ parsedType === ZodParsedType.null ||
+ parsedType === ZodParsedType.undefined
+ ) {
  const { ctx } = this._processInputParams(input);
  addIssueToContext(ctx, {
  code: ZodIssueCode.invalid_type,

diff --git a/src/__tests__/nonNullable.test.ts b/src/__tests__/nonNullable.test.ts
@@ -25,6 +25,14 @@ test("fails is passed null", () => {
  }).toThrow(z.ZodError)
 });
 
+test("fails is passed undefined", () => {
+ const a = z.nullable(z.number()).nonNullable();
+
+ expect(() => {
+ a.parse(undefined);
+ }).toThrow(z.ZodError)
+});
+
 test("unwrap", () => {
  const unwrapped = z.any().nonNullable().unwrap();
  expect(unwrapped).toBeInstanceOf(z.ZodAny);

diff --git a/src/types.ts b/src/types.ts
@@ -4480,7 +4480,10 @@ export class ZodNonNullable<T extends ZodTypeAny> extends ZodType<
 > {
  _parse(input: ParseInput): ParseReturnType<this["_output"]> {
  const parsedType = this._getType(input);
- if (parsedType === ZodParsedType.null) {
+ if (
+ parsedType === ZodParsedType.null ||
+ parsedType === ZodParsedType.undefined
+ ) {
  const { ctx } = this._processInputParams(input);
  addIssueToContext(ctx, {
  code: ZodIssueCode.invalid_type,