Skip to content

Commit a9bee4e

Browse files
committed
add hasCachedResults to viewSnapshot equality
1 parent 9f25aab commit a9bee4e

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

packages/firestore/src/core/view_snapshot.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,7 @@ export class ViewSnapshot {
183183
isEqual(other: ViewSnapshot): boolean {
184184
if (
185185
this.fromCache !== other.fromCache ||
186+
this.hasCachedResults !== other.hasCachedResults ||
186187
this.syncStateChanged !== other.syncStateChanged ||
187188
!this.mutatedKeys.isEqual(other.mutatedKeys) ||
188189
!queryEquals(this.query, other.query) ||

packages/firestore/test/unit/core/event_manager.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,7 @@ describe('QueryListener', () => {
225225
fromCache: snap2.fromCache,
226226
syncStateChanged: true,
227227
mutatedKeys: keys(),
228-
resumeToken: snap2.resumeToken
228+
hasCachedResults: snap2.hasCachedResults
229229
};
230230
expect(otherEvents).to.deep.equal([expectedSnap2]);
231231
});
@@ -398,7 +398,7 @@ describe('QueryListener', () => {
398398
fromCache: snap2.fromCache,
399399
syncStateChanged: snap2.syncStateChanged,
400400
mutatedKeys: snap2.mutatedKeys,
401-
resumeToken: snap2.resumeToken
401+
hasCachedResults: snap2.hasCachedResults
402402
};
403403
expect(filteredEvents).to.deep.equal([snap1, expectedSnap2]);
404404
}
@@ -485,7 +485,7 @@ describe('QueryListener', () => {
485485
fromCache: false,
486486
syncStateChanged: true,
487487
mutatedKeys: keys(),
488-
resumeToken: snap3.resumeToken
488+
hasCachedResults: snap3.hasCachedResults
489489
};
490490
expect(events).to.deep.equal([expectedSnap]);
491491
});
@@ -521,7 +521,7 @@ describe('QueryListener', () => {
521521
fromCache: true,
522522
syncStateChanged: true,
523523
mutatedKeys: keys(),
524-
resumeToken: snap1.resumeToken
524+
hasCachedResults: snap1.hasCachedResults
525525
};
526526
const expectedSnap2 = {
527527
query: query1,
@@ -531,7 +531,7 @@ describe('QueryListener', () => {
531531
fromCache: true,
532532
syncStateChanged: false,
533533
mutatedKeys: keys(),
534-
resumeToken: snap2.resumeToken
534+
hasCachedResults: snap2.hasCachedResults
535535
};
536536
expect(events).to.deep.equal([expectedSnap1, expectedSnap2]);
537537
});
@@ -558,7 +558,7 @@ describe('QueryListener', () => {
558558
fromCache: true,
559559
syncStateChanged: true,
560560
mutatedKeys: keys(),
561-
resumeToken: snap1.resumeToken
561+
hasCachedResults: snap1.hasCachedResults
562562
};
563563
expect(events).to.deep.equal([expectedSnap]);
564564
});
@@ -584,7 +584,7 @@ describe('QueryListener', () => {
584584
fromCache: true,
585585
syncStateChanged: true,
586586
mutatedKeys: keys(),
587-
resumeToken: snap1.resumeToken
587+
hasCachedResults: snap1.hasCachedResults
588588
};
589589
expect(events).to.deep.equal([expectedSnap]);
590590
});

0 commit comments

Comments
 (0)