Updated database emulator script and rules

Author: maneesht

config/database.rules.json

@@ -1,6 +1,9 @@
 {
   "rules": {
     ".read": true,
-    ".write": true
+    ".write": true,
+    "testing": {
+      ".indexOn": "testIndex"
+    }
   }
 }

packages/database/test/exp/integration.test.ts

@@ -122,9 +122,7 @@ describe('Database@exp Tests', () => {
 
   it('can properly handle unknown deep merges', async () => {
     // Note: This test requires `testIndex` to be added as an index.
-    // Without an index, the test will erroneously pass.
-    // But we are unable to add indexes on the fly to the actual db, so
-    // running on the emulator is the best way to test this.
+    // Please run `yarn test:setup` to ensure that this gets added.
     const database = getDatabase(defaultApp);
     const root = ref(database, 'testing');
     await set(root, {});

scripts/emulator-testing/emulators/database-emulator.ts

@@ -19,6 +19,8 @@ import * as request from 'request';
 
 import { Emulator } from './emulator';
 
+import * as rulesJSON from '../../../config/database.rules.json';
+
 export class DatabaseEmulator extends Emulator {
   namespace: string;
 
@@ -35,13 +37,14 @@ export class DatabaseEmulator extends Emulator {
   }
 
   setPublicRules(): Promise<number> {
-    console.log('Setting rule {".read": true, ".write": true} to emulator ...');
+    const jsonRules = JSON.stringify(rulesJSON);
+    console.log(`Setting rule ${jsonRules} to emulator ...`);
     return new Promise<number>((resolve, reject) => {
       request.put(
         {
           uri: `http://localhost:${this.port}/.settings/rules.json?ns=${this.namespace}`,
           headers: { Authorization: 'Bearer owner' },
-          body: '{ "rules": { ".read": true, ".write": true, "testing": { ".indexOn": "testIndex" } } }' // TODO: If this gets more complex, we should move this to a file.
+          body: jsonRules
         },
         (error, response, body) => {
           if (error) reject(error);