resolve comments

Author: milaGGL

packages/firestore/src/core/view.ts

@@ -321,8 +321,9 @@ export class View {
         newSyncState === SyncState.Local,
         syncStateChanged,
         /* excludesMetadataChanges= */ false,
-        targetChange !== undefined &&
-          targetChange.resumeToken.approximateByteSize() > 0
+        targetChange
+          ? targetChange.resumeToken.approximateByteSize() > 0
+          : false
       );
       return {
         snapshot: snap,

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

@@ -177,7 +177,7 @@ describe('QuerySnapshot', () => {
         querySnapshot('foo', {}, { a: { a: 1 } }, keys('foo/a'), false, true)
       )
     ).to.be.false;
-    // hasCachedResults should effect querySnapshot equality
+    // hasCachedResults should affect querySnapshot equality
     expect(
       snapshotEqual(
         querySnapshot(