diff --git a/packages/firestore/test/integration/api/query.test.ts b/packages/firestore/test/integration/api/query.test.ts index 7514fa836f9..e87f474bc13 100644 --- a/packages/firestore/test/integration/api/query.test.ts +++ b/packages/firestore/test/integration/api/query.test.ts @@ -2139,16 +2139,6 @@ apiDescribe('Queries', persistence => { ); } - // Skip the verification of the existence filter mismatch when the local - // cache is configured to use eager garbage collection because with eager - // GC there is no resume token specified in the subsequent call to - // getDocs(), and, therefore, Watch will _not_ send an existence filter. - // TODO(b/272754156) Re-write this test using a snapshot listener instead - // of calls to getDocs() and remove this check for disabled persistence. - if (persistence.gc === 'eager') { - return 'passed'; - } - // Skip the verification of the existence filter mismatch when testing // against the Firestore emulator because the Firestore emulator fails to // to send an existence filter at all. @@ -2209,7 +2199,10 @@ apiDescribe('Queries', persistence => { while (true) { attemptNumber++; const iterationResult = await withTestCollection( - persistence, + // Ensure that the local cache is configured to use LRU garbage + // collection (rather than eager garbage collection) so that the resume + // token and document data does not get prematurely evicted. + persistence.toLruGc(), testDocs, runTestIteration );