Fix and add more bin test.

Author: taeold

scripts/bin-test/sources/commonjs-preserve/index.js

@@ -0,0 +1,18 @@
+const functions = require("firebase-functions");
+const functionsv2 = require("firebase-functions/v2");
+
+exports.v1http = functions.https.onRequest((req, resp) => {
+  resp.status(200).send("PASS");
+});
+
+exports.v1httpPreserve = functions
+  .runWith({ preserveExternalChanges: true })
+  .https.onRequest((req, resp) => {
+    resp.status(200).send("PASS");
+  });
+
+functionsv2.setGlobalOptions({ preserveExternalChanges: true });
+
+exports.v2http = functionsv2.https.onRequest((req, resp) => {
+  resp.status(200).send("PASS");
+});

scripts/bin-test/sources/commonjs-preserve/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "commonjs-preserve"
+}

scripts/bin-test/test.ts

@@ -14,13 +14,15 @@ const TIMEOUT_M = 5_000;
 const TIMEOUT_S = 1_000;
 
 const DEFAULT_OPTIONS = {
-  memory: null,
+  availableMemoryMb: null,
   maxInstances: null,
   minInstances: null,
   timeoutSeconds: null,
-  vpcConnector: null,
-  vpcConnectorEgressSettings: null,
-  serviceAccount: null,
+  vpc: {
+    connector: null,
+    egressSettings: null,
+  },
+  serviceAccountEmail: null,
   ingressSettings: null,
 };
 
@@ -217,6 +219,33 @@ describe("functions.yaml", () => {
           },
         },
       },
+      {
+        name: "preserveChange",
+        modulePath: "./scripts/bin-test/sources/commonjs-preserve",
+        expected: {
+          endpoints: {
+            v1http: {
+              ...DEFAULT_V1_OPTIONS,
+              platform: "gcfv1",
+              entryPoint: "v1http",
+              httpsTrigger: {},
+            },
+            v1httpPreserve: {
+              platform: "gcfv1",
+              entryPoint: "v1httpPreserve",
+              httpsTrigger: {},
+            },
+            v2http: {
+              platform: "gcfv2",
+              entryPoint: "v2http",
+              labels: {},
+              httpsTrigger: {},
+            },
+          },
+          requiredAPIs: [],
+          specVersion: "v1alpha1",
+        },
+      },
     ];
 
     for (const tc of testcases) {