From d970e3bfac3f539d4fb77a67b1a8ab01cca2e08b Mon Sep 17 00:00:00 2001 From: Sawyer Cutler Date: Fri, 28 Aug 2026 04:57:45 -0700 Subject: [PATCH] CI: run database-backed package suites in the walking-skeleton job The ~40 DB-gated test suites under packages/* and apps/hub/src gate on DATABASE_URL and skip without one. build-test (the only job running `bun run test`) has no Postgres and never sets E2E_REQUIRED, so these suites have been skipping green on every CI run despite headers claiming E2E_REQUIRED=1 turns a skip into a failure. Add a step to walking-skeleton (the job with Postgres) that runs exactly those suites with E2E_REQUIRED=1, discovering them by the same DATABASE_URL grep signal they gate on so a newly added suite is picked up automatically. This is a CI-only change; there is no unit test to write. Fixes CL-7120. --- .github/workflows/ci.yml | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 868b4b51a..04f05bdb3 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -191,3 +191,15 @@ jobs: run: bun test apps/hub/test env: E2E_REQUIRED: "1" + + # Every DB-gated suite under packages/* and apps/hub/src gates on + # DATABASE_URL and describe.skip's without one, so build-test (no + # Postgres) never runs them. Find them by the same signal they + # gate on rather than hardcoding a file list, so a newly added + # suite gets picked up automatically. E2E_REQUIRED=1 turns a + # would-be skip into a hard failure. + - name: Run the database-backed package suites + run: | + bun test $(grep -rl DATABASE_URL --include='*.test.ts' packages apps | grep -v apps/hub/test) + env: + E2E_REQUIRED: "1"