don't track state for snapshots count

Author: Brian Chen

packages/firestore/test/unit/specs/spec_test_runner.ts

@@ -1018,6 +1018,8 @@ abstract class TestRunner {
         expect(this.snapshotsInSyncEvents).to.eq(
           expectation.numSnapshotsInSyncEvents
         );
+        // Reset count after checking so tests don't have to track state.
+        this.snapshotsInSyncEvents = 0;
       }
     }
 

packages/firestore/test/unit/specs/write_spec.test.ts

@@ -1470,7 +1470,7 @@ describeSpec('Writes:', [], () => {
         .addSnapshotsInSyncListener()
         .expectSnapshotsInSyncEventsCount(1)
         .userSets('collection/a', { v: 2 })
-        .expectSnapshotsInSyncEventsCount(1);
+        .expectSnapshotsInSyncEventsCount(0);
     }
   );
 
@@ -1482,7 +1482,7 @@ describeSpec('Writes:', [], () => {
         .addSnapshotsInSyncListener()
         .expectSnapshotsInSyncEventsCount(1)
         .addSnapshotsInSyncListener()
-        .expectSnapshotsInSyncEventsCount(2);
+        .expectSnapshotsInSyncEventsCount(1);
     }
   );
 
@@ -1508,19 +1508,19 @@ describeSpec('Writes:', [], () => {
         hasPendingWrites: true,
         modified: [docAv2Local]
       })
-      .expectSnapshotsInSyncEventsCount(2)
+      .expectSnapshotsInSyncEventsCount(1)
       .watchSends({ affects: [query] }, docAv2)
       .watchSnapshots(2000)
       .writeAcks('collection/a', 2000)
       .expectEvents(query, {
         metadata: [docAv2]
       })
-      .expectSnapshotsInSyncEventsCount(3);
+      .expectSnapshotsInSyncEventsCount(1);
   });
 
   specTest(
     'onSnapshotInSync fires once for multiple event snapshots',
-    ['exclusive'],
+    [],
     () => {
       const query1 = Query.atPath(path('collection'));
       const query2 = Query.atPath(path('collection/a'));
@@ -1552,7 +1552,7 @@ describeSpec('Writes:', [], () => {
           hasPendingWrites: true,
           modified: [docAv2Local]
         })
-        .expectSnapshotsInSyncEventsCount(2)
+        .expectSnapshotsInSyncEventsCount(1)
         .watchSends({ affects: [query1, query2] }, docAv2)
         .watchSnapshots(2000)
         .writeAcks('collection/a', 2000)
@@ -1562,7 +1562,7 @@ describeSpec('Writes:', [], () => {
         .expectEvents(query2, {
           metadata: [docAv2]
         })
-        .expectSnapshotsInSyncEventsCount(3);
+        .expectSnapshotsInSyncEventsCount(1);
     }
   );
 
@@ -1603,22 +1603,22 @@ describeSpec('Writes:', [], () => {
         .addSnapshotsInSyncListener()
         .addSnapshotsInSyncListener()
         // 2 original events + 2 new events from registering listeners.
-        .expectSnapshotsInSyncEventsCount(4)
+        .expectSnapshotsInSyncEventsCount(2)
         .userSets('collection/a', { v: 3 })
         .expectEvents(query, {
           hasPendingWrites: true,
           modified: [docAv3Local]
         })
         // 4 original events + 3 new events from the 3 listeners
-        .expectSnapshotsInSyncEventsCount(7)
+        .expectSnapshotsInSyncEventsCount(3)
         .removeSnapshotsInSyncListener()
         .userSets('collection/a', { v: 4 })
         .expectEvents(query, {
           hasPendingWrites: true,
           modified: [docAv4Local]
         })
         // 7 original events + 2 new events from the 2 listeners
-        .expectSnapshotsInSyncEventsCount(9)
+        .expectSnapshotsInSyncEventsCount(2)
     );
   });
 });