Skip to content

Commit c1e7dfb

Browse files
author
CI Runner
committed
fix: database-image.yml never installed workspace members or ran migrations
Discovered opening PR #2: this workflow only triggers on pull_request (and push to main), so it never ran against this branch until the PR was opened -- meaning this bug was pre-existing and unrelated to today's new tests. uv run pytest implicitly syncs, but not with --all-packages, so starter_core/reflex/reflex_local_auth were never installed; even pre-existing tests/test_auth.py and tests/test_vertical_slice.py failed collection here, not just the new auth/bootstrap tests. Added uv sync --all-packages (matching ci.yml) and uv run alembic upgrade head (the app-level tables test_vertical_slice.py and the new auth tests need don't exist without it -- this workflow never ran migrations). Verified locally against a truly fresh container (docker compose down -v, rebuild, up) running the exact new step sequence: 12/12 database or integration tests pass.
1 parent a7f49f7 commit c1e7dfb

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

.github/workflows/database-image.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@ jobs:
2828
- run: docker compose -f compose.test.yaml build postgres
2929
- run: docker compose -f compose.test.yaml up -d postgres
3030
- run: ./scripts/verify-extensions
31+
- run: uv sync --all-packages
32+
- run: uv run alembic upgrade head
3133
- run: uv run pytest -m "database or integration"
3234
- if: always()
3335
run: docker compose -f compose.test.yaml logs postgres

0 commit comments

Comments
 (0)