resolve comments

Author: milaGGL

packages/firestore/src/local/target_data.ts

@@ -69,7 +69,8 @@ export class TargetData {
     readonly resumeToken: ByteString = ByteString.EMPTY_BYTE_STRING,
     /**
      * The number of documents that last matched the query at the resume token or
-     * read time.
+     * read time. Documents are counted only when making a listen request with
+     * resume token or read time, otherwise, keep it null.
      */
     readonly expectedCount: number | null = null
   ) {}
@@ -104,7 +105,7 @@ export class TargetData {
       snapshotVersion,
       this.lastLimboFreeSnapshotVersion,
       resumeToken,
-      /** expectedCount= */ null
+      /* expectedCount= */ null
     );
   }
 

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

@@ -1826,7 +1826,7 @@ describeSpec('Listens:', [], () => {
           .expectEvents(query1, {})
           .userUnlistens(query1)
           .watchRemoves(query1)
-          // There is 0 remote document from previous listen.
+          // There are 0 remote documents from previous listen.
           .userListens(query1, {
             resumeToken: 'resume-token-1000',
             expectedCount: 0
@@ -1888,7 +1888,7 @@ describeSpec('Listens:', [], () => {
         .disableNetwork()
         .expectEvents(query1, { fromCache: true })
         .enableNetwork()
-        .restoreListen(query1, 'resume-token-1000', 1);
+        .restoreListen(query1, 'resume-token-1000', /* expectedCount= */ 1);
     }
   );
 });

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

@@ -1096,7 +1096,7 @@ export class SpecBuilder {
     resume?: ResumeSpec
   ): void {
     if (!(resume?.resumeToken || resume?.readTime) && resume?.expectedCount) {
-      fail('ExpectedCount is present without a resumeToken or readTime.');
+      fail('Expected count is present without a resume token or read time.');
     }
 
     if (this.activeTargets[targetId]) {