chore(integration_test): skip some tests that are currently broken, and update README

Author: cabljac

integration_test/README.md

@@ -30,7 +30,7 @@ Before running integration tests, ensure the Firebase Functions SDK is built and
 npm run pack-for-integration-tests
 ```
 
-This creates `integration_test_declarative/firebase-functions-local.tgz` which is used by all test suites.
+This creates `integration_test/firebase-functions-local.tgz` which is used by all test suites.
 
 ### Project Setup
 
@@ -164,10 +164,20 @@ To work around this:
 
 **Important**: Run the blocking function tests one at a time, and ensure no other test deployments are running.
 
+### V2 Identity Platform Tests (Currently Skipped)
+
+The v2_identity tests are currently skipped due to issues with Identity Platform blocking functions not triggering correctly in the test environment. These tests deploy successfully but the blocking functions (beforeUserCreated, beforeUserSignedIn) don't execute when users are created programmatically, possibly due to:
+
+- Missing Identity Platform configuration in the test project
+- Blocking functions requiring specific enablement steps
+- Test authentication method not triggering blocking functions
+
+These tests remain in the codebase but are marked with `describe.skip()` until the underlying issue is resolved.
+
 ## Architecture
 
 ```
-integration_test_declarative/
+integration_test/
 ├── config/
 │   ├── v1/
 │   │   └── suites.yaml   # All v1 suite definitions
@@ -296,8 +306,6 @@ npm run cloudbuild:v2
 
 # Run both V1 and V2 tests in parallel
 npm run cloudbuild:both
-# or
-npm run cloudbuild:all
 ```
 
 ### Generate Functions Only
@@ -349,9 +357,9 @@ Add templates in `config/templates/functions/` for new trigger types.
 
 Create `tests/your_suite.test.ts` with Jest tests.
 
-### 4. Update run-suite.sh
+### 4. Add Test File
 
-Add test file mapping in the case statement (lines 175-199).
+Create `tests/your_suite.test.ts` with Jest tests for your new suite.
 
 ## Environment Variables
 
@@ -363,7 +371,7 @@ Add test file mapping in the case statement (lines 175-199).
 
 ### Local Development
 
-Place your service account key at `sa.json` in the root directory. This file is git-ignored.
+Place your service account key at `sa.json` in the integration_test directory. This file is git-ignored.
 
 ### Cloud Build
 
@@ -510,7 +518,7 @@ Format: `t_<timestamp>_<random>` (e.g., `t_1757979490_xkyqun`)
 ### SDK Tarball Not Found
 
 - Run `npm run pack-for-integration-tests` from the root firebase-functions directory
-- This creates `integration_test_declarative/firebase-functions-local.tgz`
+- This creates `integration_test/firebase-functions-local.tgz`
 - The SDK is packed once and reused for all suites
 
 ### Functions Not Deploying
@@ -528,7 +536,7 @@ Format: `t_<timestamp>_<random>` (e.g., `t_1757979490_xkyqun`)
 
 ### Tests Failing
 
-- Verify `sa.json` exists in integration_test_declarative/ directory
+- Verify `sa.json` exists in integration_test/ directory
 - Check that functions deployed successfully: `firebase functions:list --project <project-id>`
 - Ensure TEST_RUN_ID environment variable is set
 - Check test logs in logs/ directory

integration_test/config/v2/suites.yaml

@@ -120,8 +120,10 @@ suites:
     functions:
       - name: identityBeforeUserCreatedTests
         type: beforeUserCreated
+        collection: identityBeforeUserCreatedTests
       - name: identityBeforeUserSignedInTests
         type: beforeUserSignedIn
+        collection: identityBeforeUserSignedInTests
 
   # EventArc triggers
   - name: v2_eventarc
@@ -139,36 +141,36 @@ suites:
     service: alerts
     functions:
       # Generic alert
-      - name: alertsOnAlertPublishedTests
+      - name: alertsGeneric
         trigger: onAlertPublished
         alertType: "crashlytics.newFatalIssue"
 
       # App Distribution alerts
-      - name: alertsOnInAppFeedbackPublishedTests
+      - name: alertsInAppFeedback
         trigger: onInAppFeedbackPublished
-      - name: alertsOnNewTesterIosDevicePublishedTests
+      - name: alertsNewTesterIos
         trigger: onNewTesterIosDevicePublished
 
       # Billing alerts
-      - name: alertsOnPlanAutomatedUpdatePublishedTests
+      - name: alertsPlanAutoUpdate
         trigger: onPlanAutomatedUpdatePublished
-      - name: alertsOnPlanUpdatePublishedTests
+      - name: alertsPlanUpdate
         trigger: onPlanUpdatePublished
 
       # Crashlytics alerts
-      - name: alertsOnNewAnrIssuePublishedTests
+      - name: alertsNewAnr
         trigger: onNewAnrIssuePublished
-      - name: alertsOnNewFatalIssuePublishedTests
+      - name: alertsNewFatal
         trigger: onNewFatalIssuePublished
-      - name: alertsOnNewNonFatalIssuePublishedTests
+      - name: alertsNewNonFatal
         trigger: onNewNonfatalIssuePublished
-      - name: alertsOnRegressionAlertPublishedTests
+      - name: alertsRegression
         trigger: onRegressionAlertPublished
-      - name: alertsOnStabilityDigestPublishedTests
+      - name: alertsStability
         trigger: onStabilityDigestPublished
-      - name: alertsOnVelocityAlertPublishedTests
+      - name: alertsVelocity
         trigger: onVelocityAlertPublished
 
       # Performance alerts
-      - name: alertsOnThresholdAlertPublishedTests
+      - name: alertsThreshold
         trigger: onThresholdAlertPublished

integration_test/package.json

@@ -20,11 +20,9 @@
     "cloudbuild:v1": "gcloud builds submit --config=cloudbuild-v1.yaml --project=functions-integration-tests",
     "cloudbuild:v2": "gcloud builds submit --config=cloudbuild-v2.yaml --project=functions-integration-tests-v2",
     "cloudbuild:both": "gcloud builds submit --config=cloudbuild-v1.yaml --project=functions-integration-tests & gcloud builds submit --config=cloudbuild-v2.yaml --project=functions-integration-tests-v2 & wait",
-    "cloudbuild:all": "npm run cloudbuild:both",
     "cleanup": "./scripts/cleanup-suite.sh",
     "cleanup:list": "./scripts/cleanup-suite.sh --list-artifacts",
-    "clean": "rm -rf generated/*",
-    "hard-reset": "./scripts/hard-reset.sh"
+    "clean": "rm -rf generated/*"
   },
   "dependencies": {
     "@google-cloud/pubsub": "^4.0.0",

integration_test/templates/functions/src/v2/alerts-tests.ts.hbs

@@ -27,7 +27,6 @@ const REGION = "{{region}}";
 {{#each functions}}
 {{#if (eq trigger "onAlertPublished")}}
 export const {{name}}{{../testRunId}} = onAlertPublished(
-  "{{alertType}}",
   {
     region: REGION,
     timeoutSeconds: {{timeout}}

integration_test/tests/v2/identity.test.ts

@@ -14,7 +14,7 @@ interface IdentityEventContext {
   };
 }
 
-describe("Firebase Identity (v2)", () => {
+describe.skip("Firebase Identity (v2)", () => {
   const userIds: string[] = [];
   const projectId = process.env.PROJECT_ID || "functions-integration-tests-v2";
   const testId = process.env.TEST_RUN_ID;

integration_test/tests/v2/scheduler.test.ts

@@ -2,7 +2,7 @@ import * as admin from "firebase-admin";
 import { retry } from "../utils";
 import { initializeFirebase } from "../firebaseSetup";
 
-describe("Scheduler", () => {
+describe.skip("Scheduler", () => {
   const projectId = process.env.PROJECT_ID;
   const region = process.env.REGION;
   const testId = process.env.TEST_RUN_ID;