handle null service account on CF deploy (#8381)

* handle null service account

* changelog

* use ternary instead

* Update CHANGELOG.md

* Fix bug for v1 and add test cases.

---------

Co-authored-by: Daniel Young Lee <[email protected]>

Author: leoortizz

CHANGELOG.md

@@ -1 +1,2 @@
 - Fixed a bug where the Admin SDK would not automatically connect to the Data Connect emulator when run in the Functions emulator. (#8379)
+- Fix Cloud Functions deployment failure when service account is null. (#8381)

src/gcp/cloudfunctions.spec.ts

@@ -288,6 +288,24 @@ describe("cloudfunctions", () => {
         serviceAccountEmail: `robot@${ENDPOINT.project}.iam.gserviceaccount.com`,
       });
     });
+
+    it("should handle null service account", () => {
+      expect(
+        cloudfunctions.functionFromEndpoint(
+          {
+            ...ENDPOINT,
+            httpsTrigger: {},
+            serviceAccount: null,
+          },
+          UPLOAD_URL,
+        ),
+      ).to.deep.equal({
+        ...CLOUD_FUNCTION,
+        sourceUploadUrl: UPLOAD_URL,
+        httpsTrigger: {},
+        serviceAccountEmail: null,
+      });
+    });
   });
 
   describe("endpointFromFunction", () => {

src/gcp/cloudfunctions.ts

@@ -681,7 +681,7 @@ export function functionFromEndpoint(
   );
 
   proto.convertIfPresent(gcfFunction, endpoint, "serviceAccountEmail", "serviceAccount", (from) =>
-    proto.formatServiceAccount(from!, endpoint.project, true /* removeTypePrefix */),
+    !from ? null : proto.formatServiceAccount(from, endpoint.project, true /* removeTypePrefix */),
   );
   proto.convertIfPresent(
     gcfFunction,

src/gcp/cloudfunctionsv2.spec.ts

@@ -421,7 +421,7 @@ describe("cloudfunctionsv2", () => {
       });
     });
 
-    it("should expand shorthand SA to full email", () => {
+    it("should expand shorthand service account to full email", () => {
       expect(
         cloudfunctionsv2.functionFromEndpoint({
           ...ENDPOINT,
@@ -436,6 +436,22 @@ describe("cloudfunctionsv2", () => {
         },
       });
     });
+
+    it("should handle null service account", () => {
+      expect(
+        cloudfunctionsv2.functionFromEndpoint({
+          ...ENDPOINT,
+          serviceAccount: null,
+          httpsTrigger: {},
+        }),
+      ).to.deep.equal({
+        ...CLOUD_FUNCTION_V2,
+        serviceConfig: {
+          ...CLOUD_FUNCTION_V2.serviceConfig,
+          serviceAccountEmail: null,
+        },
+      });
+    });
   });
 
   describe("endpointFromFunction", () => {

src/gcp/cloudfunctionsv2.ts

@@ -518,7 +518,10 @@ export function functionFromEndpoint(endpoint: backend.Endpoint): InputCloudFunc
     endpoint,
     "serviceAccountEmail",
     "serviceAccount",
-    (from) => proto.formatServiceAccount(from!, endpoint.project, true /* removeTypePrefix */),
+    (from) =>
+      !from
+        ? null
+        : proto.formatServiceAccount(from, endpoint.project, true /* removeTypePrefix */),
   );
   // Memory must be set because the default value of GCF gen 2 is Megabytes and
   // we use mebibytes