Skip to content

Commit fe89754

Browse files
committed
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.
1 parent 954412b commit fe89754

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -191,3 +191,15 @@ jobs:
191191
run: bun test apps/hub/test
192192
env:
193193
E2E_REQUIRED: "1"
194+
195+
# Every DB-gated suite under packages/* and apps/hub/src gates on
196+
# DATABASE_URL and describe.skip's without one, so build-test (no
197+
# Postgres) never runs them. Find them by the same signal they
198+
# gate on rather than hardcoding a file list, so a newly added
199+
# suite gets picked up automatically. E2E_REQUIRED=1 turns a
200+
# would-be skip into a hard failure.
201+
- name: Run the database-backed package suites
202+
run: |
203+
bun test $(grep -rl DATABASE_URL --include='*.test.ts' packages apps | grep -v apps/hub/test)
204+
env:
205+
E2E_REQUIRED: "1"

0 commit comments

Comments
 (0)