Fix bug where empty vpc connector setting was removed.

CHANGELOG.md

@@ -0,0 +1 @@
+- Preserve empty vpc connector setting on function deploy. (#3973)

src/deploy/functions/runtimes/node/parseTriggers.ts

@@ -212,9 +212,9 @@ export function addResourcesToBackend(
       runtime: runtime,
       ...triggered,
     };
-    if (annotation.vpcConnector) {
+    if (annotation.vpcConnector != null) {
       let maybeId = annotation.vpcConnector;
-      if (!maybeId.includes("/")) {
+      if (maybeId && !maybeId.includes("/")) {
         maybeId = `projects/${projectId}/locations/${region}/connectors/${maybeId}`;
       }
       endpoint.vpcConnector = maybeId;

src/test/deploy/functions/runtimes/node/parseTriggers.spec.ts

@@ -289,4 +289,23 @@ describe("addResourcesToBackend", () => {
 
     expect(result).to.deep.equal(expected);
   });
+
+  it("should preserve empty vpc connector setting", () => {
+    const trigger: parseTriggers.TriggerAnnotation = {
+      ...BASIC_TRIGGER,
+      httpsTrigger: {},
+      vpcConnector: "",
+    };
+
+    const result = backend.empty();
+    parseTriggers.addResourcesToBackend("project", "nodejs16", trigger, result);
+
+    const expected: backend.Backend = backend.of({
+      ...BASIC_ENDPOINT,
+      httpsTrigger: {},
+      vpcConnector: "",
+    });
+
+    expect(result).to.deep.equal(expected);
+  });
 });