BREAKING: Ignore 'functions' directory in a Firebase project without a functions config in firebase.json

Don't assume all directory named `functions` are functions directory. Instead, expect developers to explicitly set up a "functions" config in their firebase.json.

Author: taeold

scripts/functions-discover-tests/fixtures/esm/firebase.json

@@ -1 +1,3 @@
-{}
+{
+  "functions": {}
+}

scripts/functions-discover-tests/fixtures/pnpm/firebase.json

@@ -1 +1,3 @@
-{}
+{
+  "functions": {}
+}

scripts/functions-discover-tests/fixtures/simple/firebase.json

@@ -1 +1,3 @@
-{}
+{
+  "functions": {}
+}

src/config.ts

@@ -76,15 +76,17 @@ export class Config {
       }
     });
 
-    // Inject default functions config and source if missing.
-    if (this.projectDir && fsutils.dirExistsSync(this.path(Config.DEFAULT_FUNCTIONS_SOURCE))) {
-      if (Array.isArray(this.get("functions"))) {
-        if (!this.get("functions.[0].source")) {
-          this.set("functions.[0].source", Config.DEFAULT_FUNCTIONS_SOURCE);
-        }
-      } else {
-        if (!this.get("functions.source")) {
-          this.set("functions.source", Config.DEFAULT_FUNCTIONS_SOURCE);
+    // Inject default functions source if missing.
+    if (this.get("functions")) {
+      if (this.projectDir && fsutils.dirExistsSync(this.path(Config.DEFAULT_FUNCTIONS_SOURCE))) {
+        if (Array.isArray(this.get("functions"))) {
+          if (!this.get("functions.[0].source")) {
+            this.set("functions.[0].source", Config.DEFAULT_FUNCTIONS_SOURCE);
+          }
+        } else {
+          if (!this.get("functions.source")) {
+            this.set("functions.source", Config.DEFAULT_FUNCTIONS_SOURCE);
+          }
         }
       }
     }