Fix a build error

Author: wu-hui

packages/firestore/src/local/lru_garbage_collector_impl.ts

@@ -177,9 +177,9 @@ export class LruScheduler implements Scheduler {
 }
 
 /**
- * Implements the steps for LRU garbage collection. Exported for testing purpose only.
+ * Implements the steps for LRU garbage collection.
  */
-export class LruGarbageCollectorImpl implements LruGarbageCollector {
+class LruGarbageCollectorImpl implements LruGarbageCollector {
   constructor(
     private readonly delegate: LruDelegate,
     readonly params: LruParams

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

@@ -33,7 +33,6 @@ import {
 } from '../../../src/local/indexeddb_persistence';
 import { LocalStore } from '../../../src/local/local_store';
 import { LruParams } from '../../../src/local/lru_garbage_collector';
-import { LruGarbageCollectorImpl } from '../../../src/local/lru_garbage_collector_impl';
 import {
   MemoryEagerDelegate,
   MemoryLruDelegate,
@@ -228,10 +227,6 @@ export class MockMemoryOfflineComponentProvider extends MemoryOfflineComponentPr
   }
 }
 
-// A LRU garbage collector whose params are mutable.
-type Mutable<T> = { -readonly [P in keyof T]: T[P] };
-export type MutableLruGarbageCollector = Mutable<LruGarbageCollectorImpl>;
-
 export class MockConnection implements Connection {
   watchStream: StreamBridge<api.ListenRequest, api.ListenResponse> | null =
     null;

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

@@ -79,6 +79,7 @@ import {
 } from '../../../src/local/indexeddb_sentinels';
 import { LocalStore } from '../../../src/local/local_store';
 import { localStoreConfigureFieldIndexes } from '../../../src/local/local_store_impl';
+import { LruGarbageCollector } from '../../../src/local/lru_garbage_collector';
 import { MemoryLruDelegate } from '../../../src/local/memory_persistence';
 import {
   ClientId,
@@ -173,7 +174,6 @@ import {
   MockMemoryPersistence,
   MockMultiTabOfflineComponentProvider,
   MockOnlineComponentProvider,
-  MutableLruGarbageCollector,
   QueryEvent,
   SharedWriteTracker
 } from './spec_test_components';
@@ -248,7 +248,7 @@ abstract class TestRunner {
   private localStore!: LocalStore;
   private remoteStore!: RemoteStore;
   private persistence!: MockMemoryPersistence | MockIndexedDbPersistence;
-  private lruGarbageCollector!: MutableLruGarbageCollector;
+  private lruGarbageCollector!: LruGarbageCollector;
   protected sharedClientState!: SharedClientState;
 
   private useEagerGCForMemory: boolean;
@@ -889,9 +889,12 @@ abstract class TestRunner {
   private async doTriggerLruGC(cacheThreshold: number): Promise<void> {
     return this.queue.enqueue(async () => {
       if (!!this.lruGarbageCollector) {
-        this.lruGarbageCollector.params.cacheSizeCollectionThreshold =
-          cacheThreshold;
-        this.lruGarbageCollector.params.percentileToCollect = 100;
+        const params = this.lruGarbageCollector.params as {
+          cacheSizeCollectionThreshold: number;
+          percentileToCollect: number;
+        };
+        params.cacheSizeCollectionThreshold = cacheThreshold;
+        params.percentileToCollect = 100;
         await this.localStore.collectGarbage(this.lruGarbageCollector);
       }
     });