Only use emulator for integration tests.

Author: tom-andersen

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

@@ -1713,14 +1713,14 @@ apiDescribe('Database', (persistence: boolean) => {
     return withNamedTestDbs(persistence, ['db1', 'db2'], async ([db1, db2]) => {
       const data = { name: 'Rafi', email: '[email protected]' };
 
-      const ref1 = await doc(collection(db1, 'users'));
+      const ref1 = await doc(collection(db1, 'users'), 'doc1');
       await setDoc(ref1, data);
       const snapshot1 = await getDoc(ref1);
       expect(snapshot1.exists()).to.be.ok;
       expect(snapshot1.data()).to.be.deep.equals(data);
 
-      const ref2 = await doc(collection(db2, 'users'));
-      const snapshot2 = await getDoc(ref2);
+      const ref2 = await doc(collection(db2, 'users'), 'doc1');
+      const snapshot2 = await getDocFromServer(ref2);
       expect(snapshot2.exists()).to.not.be.ok;
     });
   });

packages/firestore/test/integration/util/helpers.ts

@@ -37,7 +37,8 @@ import {
 import {
   ALT_PROJECT_ID,
   DEFAULT_PROJECT_ID,
-  DEFAULT_SETTINGS
+  DEFAULT_SETTINGS,
+  USE_EMULATOR
 } from './settings';
 
 /* eslint-disable no-restricted-globals */
@@ -207,6 +208,12 @@ export async function withNamedTestDbs(
   dbNames: string[],
   fn: (db: Firestore[]) => Promise<void>
 ): Promise<void> {
+  // Named DBs requires preparing project with DBs beforehand.
+  // Emulator does not DB to have been created beforehand.
+  // TODO: Improve integration testing to have more than just (default) available.
+  if (!USE_EMULATOR) {
+    return Promise.resolve();
+  }
   const app = newTestApp(DEFAULT_PROJECT_ID);
   const dbs: Firestore[] = [];
   for (const dbName of dbNames) {