fix: handle unexpected date type reflection

Author: benjamin658

src/utils.ts

@@ -19,14 +19,26 @@ export function encodeByType(type: string, value: any): string | null {
     case 'string': {
       return encodeURIComponent(value);
     }
-    default: {
-      throw new Error(`unknown type in cursor: [${type}]${value}`);
+    case 'object': {
+      /**
+       * if reflection type is Object, check whether an object is a date.
+       * see: https://github.com/rbuckton/reflect-metadata/issues/84
+       */
+      if (typeof value.getTime === 'function') {
+        return (value as Date).getTime().toString();
+      }
+
+      break;
     }
+    default: break;
   }
+
+  throw new Error(`unknown type in cursor: [${type}]${value}`);
 }
 
 export function decodeByType(type: string, value: string): string | number | Date {
   switch (type) {
+    case 'object':
     case 'date': {
       const timestamp = parseInt(value, 10);
 

test/reflection.ts

@@ -0,0 +1,19 @@
+import { expect } from 'chai';
+
+import { encodeByType, decodeByType } from '../src/utils';
+
+describe('Reflect.getMetadata Date type is Object test', () => {
+  it('should encode cursor correctly with object type and date value', () => {
+    const date = new Date();
+    const encoded = encodeByType('object', date);
+
+    expect(encoded).to.be.a('string');
+  });
+
+  it('should decode cursor correctly with object type and date string value', () => {
+    const value = new Date().getTime().toString();
+    const decoded = decodeByType('object', value);
+
+    expect(decoded).to.be.a('date');
+  });
+});