Fix build

Author: schmidt-sebastian

packages/firestore/src/local/memory_remote_document_cache.ts

@@ -177,14 +177,14 @@ export class MemoryRemoteDocumentCache implements RemoteDocumentCache {
   }
 
   getNewDocumentChanges(
-      transaction: PersistenceTransaction,
-      sinceReadTime: SnapshotVersion
+    transaction: PersistenceTransaction,
+    sinceReadTime: SnapshotVersion
   ): PersistencePromise<{
     changedDocs: MaybeDocumentMap;
     readTime: SnapshotVersion;
   }> {
     throw new Error(
-        'getNewDocumentChanges() is not supported with MemoryPersistence'
+      'getNewDocumentChanges() is not supported with MemoryPersistence'
     );
   }
 

packages/firestore/src/local/simple_db.ts

@@ -257,7 +257,7 @@ export class SimpleDb {
 
   async runTransaction<T>(
     mode: SimpleDbTransactionMode,
-    objectStores: string[],x
+    objectStores: string[],
     transactionFn: (transaction: SimpleDbTransaction) => PersistencePromise<T>
   ): Promise<T> {
     const readonly = mode.startsWith('readonly');

packages/firestore/test/unit/local/local_store.test.ts

@@ -1487,54 +1487,51 @@ function genericLocalStoreTests(
   // Comment back in when done with the idempotent migration.
   // (queryEngine instanceof IndexFreeQueryEngine && !gcIsEager ? it : it.skip)(
   // eslint-disable-next-line no-restricted-properties
-  it.skip(
-    'uses target mapping to execute queries',
-    () => {
-      // This test verifies that once a target mapping has been written, only
-      // documents that match the query are read from the RemoteDocumentCache.
+  it.skip('uses target mapping to execute queries', () => {
+    // This test verifies that once a target mapping has been written, only
+    // documents that match the query are read from the RemoteDocumentCache.
 
-      const query = Query.atPath(path('foo')).addFilter(
-        filter('matches', '==', true)
-      );
-      return (
-        expectLocalStore()
-          .afterAllocatingQuery(query)
-          .toReturnTargetId(2)
-          .after(setMutation('foo/a', { matches: true }))
-          .after(setMutation('foo/b', { matches: true }))
-          .after(setMutation('foo/ignored', { matches: false }))
-          .afterAcknowledgingMutation({ documentVersion: 10 })
-          .afterAcknowledgingMutation({ documentVersion: 10 })
-          .afterAcknowledgingMutation({ documentVersion: 10 })
-          .afterExecutingQuery(query)
-          // Execute the query, but note that we read all existing documents
-          // from the RemoteDocumentCache since we do not yet have target
-          // mapping.
-          .toHaveRead({ documentsByQuery: 2 })
-          .after(
-            docAddedRemoteEvent(
-              [
-                doc('foo/a', 10, { matches: true }),
-                doc('foo/b', 10, { matches: true })
-              ],
-              [2],
-              []
-            )
-          )
-          .after(
-            noChangeEvent(/* targetId= */ 2, /* snapshotVersion= */ 10, 'foo')
-          )
-          .after(localViewChanges(2, /* fromCache= */ false, {}))
-          .afterExecutingQuery(query)
-          .toHaveRead({ documentsByKey: 2, documentsByQuery: 0 })
-          .toReturnChanged(
-            doc('foo/a', 10, { matches: true }),
-            doc('foo/b', 10, { matches: true })
+    const query = Query.atPath(path('foo')).addFilter(
+      filter('matches', '==', true)
+    );
+    return (
+      expectLocalStore()
+        .afterAllocatingQuery(query)
+        .toReturnTargetId(2)
+        .after(setMutation('foo/a', { matches: true }))
+        .after(setMutation('foo/b', { matches: true }))
+        .after(setMutation('foo/ignored', { matches: false }))
+        .afterAcknowledgingMutation({ documentVersion: 10 })
+        .afterAcknowledgingMutation({ documentVersion: 10 })
+        .afterAcknowledgingMutation({ documentVersion: 10 })
+        .afterExecutingQuery(query)
+        // Execute the query, but note that we read all existing documents
+        // from the RemoteDocumentCache since we do not yet have target
+        // mapping.
+        .toHaveRead({ documentsByQuery: 2 })
+        .after(
+          docAddedRemoteEvent(
+            [
+              doc('foo/a', 10, { matches: true }),
+              doc('foo/b', 10, { matches: true })
+            ],
+            [2],
+            []
           )
-          .finish()
-      );
-    }
-  );
+        )
+        .after(
+          noChangeEvent(/* targetId= */ 2, /* snapshotVersion= */ 10, 'foo')
+        )
+        .after(localViewChanges(2, /* fromCache= */ false, {}))
+        .afterExecutingQuery(query)
+        .toHaveRead({ documentsByKey: 2, documentsByQuery: 0 })
+        .toReturnChanged(
+          doc('foo/a', 10, { matches: true }),
+          doc('foo/b', 10, { matches: true })
+        )
+        .finish()
+    );
+  });
 
   it('last limbo free snapshot is advanced during view processing', async () => {
     // This test verifies that the `lastLimboFreeSnapshot` version for QueryData