| name | tests-improve |
| description | Comprehensively improve unit tests — invoke when a source file's coverage is below the 100%-per-file target, or its test file is missing or disorganized. Ensures test files exist, adds missing tests, removes unneeded ones, and restructures to reach full per-source-file coverage |
For every file in src/deephaven_mcp except _version.py:
- Make sure that there is a test file.
__init__.py files count. Every __init__.py — even one that only declares __all__ or re-exports from siblings — gets its own test_init.py. The package surface is part of the API contract; an untested __init__.py is a silent-refactor hazard.- What the
test_init.py must pin: the exact set of names in __all__, that each re-export is the same object as the internal definition, and that no _-prefixed names leak into the public surface.
- Canonical implementations:
tests/config/schema/test_init.py, tests/config/test_init.py, tests/auth/middleware/test_init.py.
- Exception — a test directory kept deliberately non-package cannot host a
test_init.py. tests/mcp_systems_server/_tools/ has no __init__.py because its modules import fixtures by bare name (from conftest import MockContext); adding one breaks collection there. Without __init__.py, a test_init.py in that directory resolves to the same module name as tests/test_init.py and pytest fails with "import file mismatch". That package's surface is pinned by tests/mcp_systems_server/_tools/test_tool_module_inventory.py instead. Do not create a test_init.py there.
- Make sure the test file is in the correct directory with a name that meets project standards.
- Unit tests:
test_<file>.py. Integration tests: test_<file>_integration.py. Both live in tests/<package>/ mirroring the source (see ref-python-coding-practices rule 5).
- For
__init__.py the test file is test_init.py (single underscore between test and init, matching tests/auth/middleware/test_init.py).
- A test whose subject is broader than one source file is out of scope for this step. Guardrail/contract tests and package-scope integration tests are named for their invariant, not for a module, and rule 5's broader than one source file clause governs them. Do not rename one to fit
test_<file>.py, and do not create a source-mirroring stub for one — tests/test_field_docs_contract.py tests an invariant across every schema in the package, not a source module of the same stem, and no such module exists.
- Analyze the test file and flag tests that are dead, redundant, test implementation detail rather than observable behavior, or are overly fragile (assert on incidental output, internal call order, or private state). List the flagged tests before changing anything.
- Also flag any threshold the author invented — a byte cap, a maximum entry count, a size budget with no source in the code or a documented requirement (
ref-python-coding-practices rule 21). Ask what defect it stands in for and assert that directly; if the property resists a test, delete the test rather than keep an arbitrary number that will fail someone else's legitimate work.
- Add tests for every uncovered branch and error path (coverage rises toward 100%); testing private functions is appropriate for simpler, more targeted tests.
- Remove the tests flagged dead or redundant in step 3; confirm coverage holds after removal.
- Restructure the tests flagged fragile in step 3 (those asserting on incidental output, call order, or private state) to assert on observable behavior instead.
- Run the individual test file. Test files must be run one-by-one to accurately assess per-file coverage.
- Target 100% coverage. If coverage is below 100%, add tests until it reaches 100%.
pytest is pre-configured for coverage — no extra flags needed. To run a single file: uv run pytest <file>. Do not add --cov= or --cov-report=; these are already configured.
Per-file runs are required because running the full suite together does not show whether a source file's own test file covers it completely. See the tests-run-file skill for the per-file invocation rules.
After per-file runs pass, run the tests-run skill to verify the full suite. For integration tests (test_<file>_integration.py), use the integration-tests-run skill — they have separate invocation rules (-m integration -s).
