Feedback

Author: var-const

packages/firestore/test/unit/api/geo_point.test.ts

@@ -102,17 +102,21 @@ describe('GeoPoint', () => {
   });
 
   it('serializes to JSON', () => {
-    expect(JSON.stringify(new GeoPoint(1, 2))).to.equal(
-      '{"latitude":1,"longitude":2}'
-    );
-    expect(JSON.stringify(new GeoPoint(0, 0))).to.equal(
-      '{"latitude":0,"longitude":0}'
-    );
-    expect(JSON.stringify(new GeoPoint(-0, -0))).to.equal(
-      '{"latitude":0,"longitude":0}'
-    );
-    expect(JSON.stringify(new GeoPoint(90, 180))).to.equal(
-      '{"latitude":90,"longitude":180}'
-    );
+    expect(new GeoPoint(1, 2).toJSON()).to.deep.equal({
+      latitude: 1,
+      longitude: 2
+    });
+    expect(new GeoPoint(0, 0).toJSON()).to.deep.equal({
+      latitude: 0,
+      longitude: 0
+    });
+    expect(new GeoPoint(-0, -0).toJSON()).to.deep.equal({
+      latitude: 0,
+      longitude: 0
+    });
+    expect(new GeoPoint(90, 180).toJSON()).to.deep.equal({
+      latitude: 90,
+      longitude: 180
+    });
   });
 });

packages/firestore/test/unit/api/timestamp.test.ts

@@ -134,14 +134,17 @@ describe('Timestamp', () => {
   });
 
   it('serializes to JSON', () => {
-    expect(JSON.stringify(new Timestamp(123, 456))).to.equal(
-      '{"seconds":123,"nanoseconds":456}'
-    );
-    expect(JSON.stringify(new Timestamp(0, 0))).to.equal(
-      '{"seconds":0,"nanoseconds":0}'
-    );
-    expect(JSON.stringify(new Timestamp(-123, 456))).to.equal(
-      '{"seconds":-123,"nanoseconds":456}'
-    );
+    expect(new Timestamp(123, 456).toJSON()).to.deep.equal({
+      seconds: 123,
+      nanoseconds: 456
+    });
+    expect(new Timestamp(0, 0).toJSON()).to.deep.equal({
+      seconds: 0,
+      nanoseconds: 0
+    });
+    expect(new Timestamp(-123, 456).toJSON()).to.deep.equal({
+      seconds: -123,
+      nanoseconds: 456
+    });
   });
 });