emil
d32bb40cbd
Post-merge / detect-type (push) Successful in 53s
Post-merge / release (push) Successful in 59s
Post-merge / publish (push) Has been skipped
Post-merge / validate-commit-msg (push) Successful in 1m32s
Post-merge / configure-repo (push) Successful in 1m34s
Post-merge / vikunja (push) Successful in 1m38s
Post-merge / badges (push) Successful in 1m46s
Post-merge / sync-wiki (push) Successful in 2m34s
72 lines
2.2 KiB
Markdown
72 lines
2.2 KiB
Markdown
# testing-and-debugging
|
|
|
|
Make targets for testing, debugging, and CI investigation. **Use these
|
|
instead of raw `pytest`, `ruff`, or `molecule` commands.**
|
|
|
|
## Why Make Targets
|
|
|
|
Make targets encapsulate the correct venv activation, PYTHONPATH, env
|
|
vars, and flags. Running raw commands bypasses venv activation and
|
|
produces false failures (missing dependencies, wrong Python version).
|
|
|
|
## Unit Tests
|
|
|
|
| Task | Command | Notes |
|
|
|------|---------|-------|
|
|
| Run all unit tests | `make test-unit` | Fast, no coverage |
|
|
| Run with coverage | `make pytest-cov` | **Required before push** — enforces 100% |
|
|
| Run single test | `make pytest-cov TEST=tests/test_foo.py::test_bar` | |
|
|
|
|
## Linting
|
|
|
|
| Task | Command | Notes |
|
|
|------|---------|-------|
|
|
| Full lint | `make lint-all` | ruff + pyright + bandit + ansible-lint + checkmake + actionlint |
|
|
| Ruff only | `make lint-ruff` | |
|
|
| Type check | `make typecheck` | pyright |
|
|
| Bandit | `make lint-bandit` | Security linter |
|
|
| Workflow lint | `make workflow-check` | actionlint + act_runner dry-run |
|
|
|
|
## Molecule Tests
|
|
|
|
| Task | Command | Notes |
|
|
|------|---------|-------|
|
|
| All scenarios | `make molecule` | All 6 scenarios on Ubuntu 22.04 |
|
|
| All platforms | `make molecule-all` | All 6 scenarios on all 4 OSes |
|
|
| Parallel | `make molecule-all-parallel` | MOLECULE_JOBS=4 |
|
|
|
|
## Pre-Push Verification
|
|
|
|
**Before pushing any branch:**
|
|
|
|
```bash
|
|
make pre-push
|
|
```
|
|
|
|
This runs `lint-all` + `pytest-cov`. The pre-push git hook only
|
|
validates the Vikunja task exists — it does NOT run tests. You must
|
|
run `make pre-push` manually.
|
|
|
|
## CI Failure Investigation
|
|
|
|
When investigating a CI failure:
|
|
|
|
1. **Fetch logs via MCP** — use `mcp_call_tool` with gitea server,
|
|
`actions_run_read` method, `download_job_log` tool
|
|
2. **Reproduce locally** — use `make pytest-cov` or `make lint-ci`
|
|
depending on which CI job failed
|
|
3. **Never run raw pytest** — always use the make target
|
|
|
|
## Virtual Environment
|
|
|
|
All commands run inside `.venv`. `make` targets handle activation
|
|
automatically. For raw commands (rare), activate first:
|
|
|
|
```bash
|
|
source activate.sh # bash/zsh
|
|
source activate.fish # fish
|
|
source activate.zsh # zsh
|
|
```
|
|
|
|
If `.venv` doesn't exist, run `make setup` first.
|