Fix bug where timestamp in change objects are still date objects (#290)

Author: laurenzlong

spec/providers/firestore.spec.ts

@@ -20,6 +20,8 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 // SOFTWARE.
 
+import * as admin from 'firebase-admin';
+
 import * as firestore from '../../src/providers/firestore';
 import * as _ from 'lodash';
 import { expect } from 'chai';
@@ -428,7 +430,9 @@ describe('Firestore Functions', () => {
           })
         );
         expect(snapshot.data()).to.deep.equal({
-          timestampVal: new Date('2017-06-13T00:58:40.349Z'),
+          timestampVal: admin.firestore.Timestamp.fromDate(
+            new Date('2017-06-13T00:58:40.349Z')
+          ),
         });
       });
 
@@ -444,7 +448,9 @@ describe('Firestore Functions', () => {
           })
         );
         expect(snapshot.data()).to.deep.equal({
-          timestampVal: new Date('2017-06-13T00:58:40Z'),
+          timestampVal: admin.firestore.Timestamp.fromDate(
+            new Date('2017-06-13T00:58:40Z')
+          ),
         });
       });
 

src/providers/firestore.ts

@@ -161,6 +161,7 @@ export function snapshotConstructor(event: Event): DocumentSnapshot {
 export function beforeSnapshotConstructor(event: Event): DocumentSnapshot {
   if (!firestoreInstance) {
     firestoreInstance = firebase.firestore(apps().admin);
+    firestoreInstance.settings({ timestampsInSnapshots: true });
   }
   let oldValueProto = _getValueProto(
     event.data,