Add support for date in jest-get-type (#4621)

jestjs · Oct 7, 2017 · 6646c3b · 6646c3b
1 parent f264780
commit 6646c3b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/jest-get-type/src/__tests__/index.test.js b/packages/jest-get-type/src/__tests__/index.test.js
@@ -23,4 +23,5 @@ describe('.getType()', () => {
   test('regexp', () => expect(getType(/abc/)).toBe('regexp'));
   test('map', () => expect(getType(new Map())).toBe('map'));
   test('set', () => expect(getType(new Set())).toBe('set'));
+  test('date', () => expect(getType(new Date())).toBe('date'));
 });
diff --git a/packages/jest-get-type/src/index.js b/packages/jest-get-type/src/index.js
@@ -19,14 +19,15 @@ export type ValueType =
   | 'regexp'
   | 'map'
   | 'set'
+  | 'date'
   | 'string'
   | 'symbol'
   | 'undefined';
 
 // get the type of a value with handling the edge cases like `typeof []`
 // and `typeof null`
 const getType = (value: any): ValueType => {
-  if (typeof value === 'undefined') {
+  if (value === undefined) {
     return 'undefined';
   } else if (value === null) {
     return 'null';
@@ -47,6 +48,8 @@ const getType = (value: any): ValueType => {
       return 'map';
     } else if (value.constructor === Set) {
       return 'set';
+    } else if (value.constructor === Date) {
+      return 'date';
     }
     return 'object';
     // $FlowFixMe https://github.com/facebook/flow/issues/1015