5fd2c1a129
* fix(cli): separate unit tests from integration tests and fix test cleanup - Rename 14 non-backend test files to *_unit.test.ts convention - Add UNIT_ONLY env var guard in setup.ts to skip cargo build/backend startup - Add test:unit and test:integration scripts to package.json - Use setsid on Linux for process group management so stop() kills both cargo and the windmill child process - Fix exit handler to kill process group instead of just the direct child - Add cleanupStaleTestResources() to drop orphaned windmill_test_* databases and kill orphaned backend processes on startup - Rewrite TESTING.md with current bun-based instructions Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(cli): fix process group approach - kill by db name instead of setsid The setsid approach didn't work because setsid forks, making the PID we get from Bun.spawn ephemeral. Instead, kill orphaned windmill child processes by matching our unique database name in /proc/pid/environ. Also add afterAll hook in setup.ts so full async cleanup (process kill + database drop) runs when all tests complete normally, not just on SIGINT/SIGTERM. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(cli): address PR review feedback - Remove duplicate cleanupStaleTestResources() call in getTestBackend() (already called in setup.ts) - Add regex guard on database names before SQL interpolation - Extract shared killWindmillProcessesByEnvMatch() helper to deduplicate process-killing logic - Remove redundant test:integration script (test already runs everything) - Flip setup.ts to if/else pattern for readability Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2.0 KiB
2.0 KiB
Testing Guide for Windmill CLI
Running Tests
# Run unit tests only (fast — no backend, no database, no cargo build)
bun run test:unit
# Run all tests (unit + integration — requires PostgreSQL + cargo)
DATABASE_URL=postgres://postgres:changeme@localhost:5432 bun run test
# Run specific test files
bun test test/sync_pull_push.test.ts
bun test test/workspace_conflicts_unit.test.ts
Test Categories
Unit tests (*_unit.test.ts)
Pure local tests — no backend, no database. Uses bunfig.unit.toml (no preload).
Examples: git_unit, lint_command_unit, tar_creation_unit, workspace_conflicts_unit
Integration tests
Require a running backend and PostgreSQL. The setup.ts preload builds the backend
binary and starts a shared backend instance.
Examples: sync_pull_push, dev_server, standalone_commands
Environment Variables
| Variable | Purpose | Default |
|---|---|---|
DATABASE_URL |
PostgreSQL connection string (without database name) | postgres://postgres:changeme@localhost:5432 |
TEST_BACKEND |
cargo or docker |
cargo |
CI_MINIMAL_FEATURES |
true for CI mode (zip-only features) |
unset |
EE_LICENSE_KEY |
Enterprise license for EE feature tests | unset |
TEST_FEATURES |
Additional cargo features (comma-separated) | unset |
TEST_CLI_RUNTIME |
node to test npm package |
unset |
UNIT_ONLY |
1 to skip backend setup in preload (used by test:unit) |
unset |
VERBOSE |
1 for backend process output |
unset |
Cleanup
Stale test databases (windmill_test_*) and orphaned backend processes from
previous crashed runs are automatically cleaned up when starting a new test run.
To manually check for leftovers:
# Check for stale test databases
psql postgres://postgres:changeme@localhost:5432/postgres -c \
"SELECT datname FROM pg_database WHERE datname LIKE 'windmill_test_%';"
# Check for orphaned backend processes
ps aux | grep "target/debug/windmill" | grep -v grep