supabase / cli / 36978351293
67%
develop: 67%

LAST BUILD BRANCH: gh-readonly-queue/develop/pr-6996-bd1c042e5e6967e393557bc3236d5357865aa3bf
DEFAULT BRANCH: develop
Ran
Jobs 1
Files 79
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 66.737%. Remained the same
36978351293

push

github

web-flow 
test(repo): remove test Docker volumes once per run by creation label (#6923)

Docker-backed stack tests left a named `supabase-db-*` volume behind on
every run.

Docker database storage keeps one shared volume per state root and
daemon. Stack destroy deliberately clears only the stack's namespace
inside it, because the volume is shared with other stacks and holds the
snapshot cache. Tests that run a Docker database under a temporary state
root destroyed the stack and then deleted the root. After that, nothing
could find the volume again.

This PR cleans up those volumes in one place per test run instead of in
each test:

- When `SUPABASE_STACK_TEST_RUN` is set, `DockerDatabaseStorage` adds a
`com.supabase.stack-test-run=<id>` label to the volumes it creates, and
the container runtime adds it to the containers it creates. Nothing else
changes, and nothing changes when the variable is unset.
- `packages/stack/tests/docker-volume-run.ts` is a vitest `globalSetup`
for the stack and CLI integration and E2E projects. It generates a
random id unless an outer invocation already owns one. Its teardown
removes exactly the stack-managed volumes carrying that id, without
force, and fails the run if a removal fails. It skips when Docker is
unavailable.
- Vitest workers, CLI subprocesses and detached stack owners inherit the
id. `processEnvLayer` keeps it when it replaces the environment.
- Docker test state roots must stay private to a run. The
`Commands.integration` Docker variants passed a bare temporary directory
as the database root, which made them share one never-removed volume
keyed to the parent of `$TMPDIR`. They now use `makeDockerDatabaseRoot`
like the other database service tests.
- The stack-cache E2E's own marker-based volume cleanup is removed.

- Under Bun, vitest exits on SIGINT and SIGTERM without running
globalSetup teardown, so an interrupted run used to leave its volumes
behind and its containers running. The owning setup now records a
per-us... (continued)

4097 of 6139 relevant lines covered (66.74%)

7.37 hits per line

Jobs
ID Job ID Ran Files Coverage
1 36978351293.1 79
66.74
GitHub Action Run
Source Files on build 36978351293