fix(types): Array required() and defined() will no longer return any (#…

…1256)

* Array required() and defined() will no longer return any

* Array required and defined test improvements

* Array nullable() will not return any + use ESLint comment to test array defined and array required

src/array.ts

@@ -261,6 +261,20 @@ export default class ArraySchema<
     if (this.innerType) base.innerType = this.innerType.describe();
     return base;
   }
+
+  nullable(isNullable?: true): ArraySchema<T, C, TIn | null>;
+  nullable(isNullable: false): ArraySchema<T, C, Exclude<TIn, null>>;
+  nullable(isNullable = true): ArraySchema<T, C, TIn | null> {
+    return super.nullable(isNullable as any);
+  }
+
+  defined(): DefinedArraySchema<T, C, TIn> {
+    return super.defined();
+  }
+
+  required(msg?: MixedLocale['required']): RequiredArraySchema<T, C, TIn> {
+    return super.required(msg);
+  }
 }
 
 create.prototype = ArraySchema.prototype;

test/types.ts

@@ -264,6 +264,20 @@ SchemaOf: {
   const _c1 = array(number())
     .concat(array(number()).required())
     .validateSync([]);
+
+  // $ExpectType AssertsShape<{ a: RequiredNumberSchema<number | undefined, Record<string, any>>; }>[] | null
+  const _definedArray: Array<{ a: number }> | null = array()
+    .of(object({ a: number().required() }))
+    .nullable()
+    .defined()
+    .validateSync([]);
+
+  // $ExpectType AssertsShape<{ a: RequiredNumberSchema<number | undefined, Record<string, any>>; }>[]
+  const _requiredArray: Array<{ a: number }> = array()
+    .of(object({ a: number().required() }))
+    .nullable()
+    .required()
+    .validateSync([]);
 }
 
 {