Some UX polish for enabling/disabling functions (#3203)

Author: samtstern

CHANGELOG.md

@@ -0,0 +1 @@
+- Fixes extra logging and race conditions when disabling emulated background functions.

src/emulator/functionsEmulator.ts

@@ -876,9 +876,9 @@ export class FunctionsEmulator implements EmulatorInstance {
     return this.workerPool.submitWork(frb.triggerId, frb, opts);
   }
 
-  disableBackgroundTriggers() {
+  async disableBackgroundTriggers() {
     Object.values(this.triggers).forEach((record) => {
-      if (record.def.eventTrigger) {
+      if (record.def.eventTrigger && record.enabled) {
         this.logger.logLabeled(
           "BULLET",
           `functions[${record.def.entryPoint}]`,
@@ -887,6 +887,8 @@ export class FunctionsEmulator implements EmulatorInstance {
         record.enabled = false;
       }
     });
+
+    await this.workQueue.flush();
   }
 
   async reloadTriggers() {

src/emulator/hub.ts

@@ -104,7 +104,7 @@ export class EmulatorHub implements EmulatorInstance {
       }
     });
 
-    this.hub.put(EmulatorHub.PATH_DISABLE_FUNCTIONS, (req, res) => {
+    this.hub.put(EmulatorHub.PATH_DISABLE_FUNCTIONS, async (req, res) => {
       utils.logLabeledBullet(
         "emulators",
         `Disabling Cloud Functions triggers, non-HTTP functions will not execute.`
@@ -117,7 +117,7 @@ export class EmulatorHub implements EmulatorInstance {
       }
 
       const emu = instance as FunctionsEmulator;
-      emu.disableBackgroundTriggers();
+      await emu.disableBackgroundTriggers();
       res.status(200).json({ enabled: false });
     });