🐛 Fix minor type error in types utility

Author: ralfstx

src/print-value.test.ts

@@ -22,6 +22,7 @@ describe('printValue', () => {
     expect(printValue(new Date('2001-02-03T04:05:06.789Z'))).toEqual(
       'Date 2001-02-03T04:05:06.789Z',
     );
+    expect(printValue(new Date('invalid date'))).toEqual('Invalid Date');
   });
 
   it('prints functions', () => {

src/print-value.ts

@@ -1,7 +1,13 @@
 export function printValue(value: unknown, refs?: unknown[]) {
   if (typeof value === 'string') return `'${value}'`;
   if (Array.isArray(value)) return printArray(value, refs);
-  if (value instanceof Date) return `Date ${value.toISOString()}`;
+  if (value instanceof Date) {
+    try {
+      return `Date ${value.toISOString()}`;
+    } catch {
+      return `Invalid Date`;
+    }
+  }
   if (value instanceof Function) {
     return value.name ? `function ${value.name}` : 'anonymous function';
   }

src/types.test.ts

@@ -288,7 +288,7 @@ describe('readNumber', () => {
   });
 });
 
-describe('asDate', () => {
+describe('readDate', () => {
   it('returns date objects', () => {
     const date = new Date('2000-04-01T12:13:14.000Z');
 
@@ -305,6 +305,12 @@ describe('asDate', () => {
     expect(() => readDate(23)).toThrow(new TypeError('Expected Date, got: 23'));
     expect(() => readDate(null)).toThrow(new TypeError('Expected Date, got: null'));
   });
+
+  it('throws for invalid Date objects', () => {
+    expect(() => readDate(new Date('invalid date'))).toThrow(
+      new TypeError('Expected valid Date, got: Invalid Date'),
+    );
+  });
 });
 
 describe('readArray', () => {

src/types.ts

@@ -159,7 +159,13 @@ export function readNumber(input: unknown, options?: NumberOptions) {
 }
 
 export function readDate(value: unknown): Date {
-  if (value instanceof Date) return value as Date;
+  if (value instanceof Date) {
+    if (Number.isNaN(value.getTime())) {
+      throw typeError('valid Date', value);
+    }
+    return value as Date;
+  }
+
   throw typeError('Date', value);
 }
 
@@ -214,25 +220,22 @@ export function readObject<T extends Record<string, TypeDef<unknown>>>(
   if (options?.maxProperties != null && Object.keys(input).length > options.maxProperties) {
     throw typeError(`object with max. ${options.maxProperties} properties`, input);
   }
-  return properties ? mapObject(input, properties) : (input as any);
-}
-
-function mapObject(obj: Obj, properties: Record<string, TypeDef<unknown>>) {
+  if (!properties) return input as any;
   return pickDefined(
     Object.fromEntries(
       Object.entries(properties).map(([key, type]) => {
-        return [key, readFrom(obj, key, type)];
+        return [key, readFrom(input, key, type)];
       }),
     ),
-  );
+  ) as Partial<{ [P in keyof T]: ReturnType<T[P]> }>;
 }
 
 export function isObject(value: unknown): value is Record<string, unknown> {
   return (
     value != null &&
     typeof value === 'object' &&
     !Array.isArray(value) &&
-    value.toString() === '[object Object]'
+    Object.prototype.toString.call(value) === '[object Object]'
   );
 }