Add integration test for discovering functions from a project reposit…

…ory.

.github/workflows/node-test.yml

@@ -85,6 +85,7 @@ jobs:
  - npm run test:emulator
  - npm run test:extensions-emulator
  - npm run test:frameworks
+ - npm run test:functions-discover
  - npm run test:hosting
  # - npm run test:hosting-rewrites # Long-running test that might conflict across test runs. Run this manually.
  - npm run test:import-export
@@ -143,6 +144,7 @@ jobs:
  - npm run test:emulator
  # - npm run test:import-export # Fails becuase port 4000 is taken after first run - hub not shhutting down?
  # - npm run test:extensions-emulator # Fails due to cannot find module sharp (not waiting for npm install?)
+ - npm run test:functions-discover
  - npm run test:triggers-end-to-end
  - npm run test:triggers-end-to-end:inspect
  - npm run test:storage-deploy

package.json

@@ -32,6 +32,7 @@
  "test:extensions-emulator": "bash ./scripts/extensions-emulator-tests/run.sh",
  "test:frameworks": "bash ./scripts/frameworks-tests/run.sh",
  "test:functions-deploy": "bash ./scripts/functions-deploy-tests/run.sh",
+ "test:functions-discover": "bash ./scripts/functions-discover-tests/run.sh",
  "test:hosting": "bash ./scripts/hosting-tests/run.sh",
  "test:hosting-rewrites": "bash ./scripts/hosting-tests/rewrites-tests/run.sh",
  "test:import-export": "bash ./scripts/emulator-import-export-tests/run.sh",

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

@@ -0,0 +1,26 @@
+{
+ "functions": [
+ {
+ "source": "v1",
+ "codebase": "v1",
+ "runtime": "nodejs16",
+ "ignore": [
+ "node_modules",
+ ".git",
+ "firebase-debug.log",
+ "firebase-debug.*.log"
+ ]
+ },
+ {
+ "source": "v2",
+ "codebase": "v2",
+ "runtime": "nodejs16",
+ "ignore": [
+ "node_modules",
+ ".git",
+ "firebase-debug.log",
+ "firebase-debug.*.log"
+ ]
+ }
+ ]
+}

scripts/functions-discover-tests/fixtures/codebases/install.sh

@@ -0,0 +1,6 @@
+#!/bin/bash
+set -euxo pipefail # bash strict mode
+IFS=$'\n\t'
+
+(cd v1 && npm i)
+(cd v2 && npm i)

scripts/functions-discover-tests/fixtures/codebases/v1/index.js

@@ -0,0 +1,5 @@
+const functions = require("firebase-functions");
+
+exports.hellov1 = functions.https.onRequest((request, response) => {
+ response.send("Hello from Firebase!");
+});

scripts/functions-discover-tests/fixtures/codebases/v1/package.json

@@ -0,0 +1,9 @@
+{
+ "name": "codebase-v1",
+ "dependencies": {
+ "firebase-functions": "^4.0.0"
+ },
+ "engines": {
+ "node": "16"
+ }
+}

scripts/functions-discover-tests/fixtures/codebases/v2/index.js

@@ -0,0 +1,5 @@
+import { onRequest } from "firebase-functions/v2/https";
+
+export const hellov2 = onRequest((request, response) => {
+ response.send("Hello from Firebase!");
+});

scripts/functions-discover-tests/fixtures/codebases/v2/package.json

@@ -0,0 +1,10 @@
+{
+ "name": "codebase-v2",
+ "type": "module",
+ "dependencies": {
+ "firebase-functions": "^4.0.0"
+ },
+ "engines": {
+ "node": "16"
+ }
+}

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

@@ -0,0 +1 @@
+{}

scripts/functions-discover-tests/fixtures/esm/functions/index.js

@@ -0,0 +1,11 @@
+import * as functions from "firebase-functions";
+import { onRequest } from "firebase-functions/v2/https";
+
+export const hellov1 = functions.https.onRequest((request, response) => {
+ functions.logger.info("Hello logs!", { structuredData: true });
+ response.send("Hello from Firebase!");
+});
+
+export const hellov2 = onRequest((request, response) => {
+ response.send("Hello from Firebase!");
+});

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

@@ -0,0 +1,10 @@
+{
+ "name": "esm",
+ "type": "module",
+ "dependencies": {
+ "firebase-functions": "^4.0.0"
+ },
+ "engines": {
+ "node": "16"
+ }
+}

scripts/functions-discover-tests/fixtures/esm/install.sh

@@ -0,0 +1,5 @@
+#!/bin/bash
+set -euxo pipefail # bash strict mode
+IFS=$'\n\t'
+
+cd functions && npm i

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

@@ -0,0 +1 @@
+{}

scripts/functions-discover-tests/fixtures/simple/functions/index.js

@@ -0,0 +1,10 @@
+const functions = require("firebase-functions");
+const { onRequest } = require("firebase-functions/v2/https");
+
+exports.hellov1 = functions.https.onRequest((request, response) => {
+ response.send("Hello from Firebase!");
+});
+
+exports.hellov2 = onRequest((request, response) => {
+ response.send("Hello from Firebase!");
+});

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

@@ -0,0 +1,9 @@
+{
+ "name": "simple",
+ "dependencies": {
+ "firebase-functions": "^4.0.0"
+ },
+ "engines": {
+ "node": "16"
+ }
+}

scripts/functions-discover-tests/fixtures/simple/install.sh

@@ -0,0 +1,5 @@
+#!/bin/bash
+set -euxo pipefail # bash strict mode
+IFS=$'\n\t'
+
+cd functions && npm i

scripts/functions-discover-tests/run.sh

@@ -0,0 +1,12 @@
+#!/bin/bash
+set -euxo pipefail # bash strict mode
+IFS=$'\n\t'
+
+# Globally link the CLI for the testing framework
+./scripts/npm-link.sh
+
+for dir in ./scripts/functions-discover-tests/fixtures/*; do
+ (cd $dir && ./install.sh)
+done
+
+mocha scripts/functions-discover-tests/tests.ts

scripts/functions-discover-tests/tests.ts

@@ -2,10 +2,9 @@ import * as path from "path";
 
 import { expect } from "chai";
 import { CLIProcess } from "../integration-helpers/cli";
-import { requireAuth } from "../../src/requireAuth";
 
 const FIXTURES = path.join(__dirname, "fixtures");
-const FIREBASE_PROJECT = process.env.FBTOOLS_TARGET_PROJECT || "";
+const FIREBASE_PROJECT = "demo-project";
 
 interface Testcase {
  name: string;
@@ -19,12 +18,8 @@ interface Testcase {
 describe("Function discovery test", function (this) {
  this.timeout(1000_000);
 
- before(async () => {
+ before(() => {
  expect(FIREBASE_PROJECT).to.exist.and.not.be.empty;
- if (process.env.CI) {
- // In CI, get auth credentials from environment variable.
- await requireAuth({});
- }
  });
 
  const testCases: Testcase[] = [