Test ML Pipeline (router)
Where tests for an ML workspace live, what gets paired with what, and which subskill owns the body of each test category.
First action (every turn)
Before answering anything else:
- Confirm an approved design note exists for the test
the user is asking about. The pairing rule is hard:
tests/<category>/test_NN_<short_name>.pyonly exists ifjournal/NN_<short_name>.mdis at leastapprovedandexperiments/NN_<short_name>.pyis the matching script. If the design note doesn't exist, hand back toiterate-ml-experiment. - Emit the Pre-flight checklist (below) as visible text in your response, with each box marked.
- Use the Dispatch table to pick the subskill that owns the test category, then hand off.
Pre-flight — emit this checklist as visible text before any test work
Pre-flight (test-ml-pipeline):
- [ ] `journal/NN_<short_name>.md` exists and is at least `approved`
(or confirmed n/a — about to hand off to `iterate-ml-experiment`)
- [ ] `experiments/NN_<short_name>.py` exists with the matching stem
(or confirmed n/a — about to hand off to `organize-ml-workspace`)
- [ ] Test category picked: smoke | regression | distribution | …
- [ ] Subskill dispatched: `smoke-test-ml-pipeline` | …
- [ ] Test file stem decided: `tests/<category>/test_NN_<short_name>.py`
- [ ] pytest is on the project's dependency manifest (per
`data-science-python-stack` § Tier 1)
Stop conditions — read before anything else
- No test without an approved design note. Never create
tests/<category>/test_NN_*.pyif the matchingjournal/NN_*.mdisn't on disk and at leastapproved. The design note is the contract; the test asserts the contract holds. Reverse order is incoherent. - The stem rule is hard. Test file basename is
test_NN_<short_name>.py(with thetest_prefix that pytest expects); theNN_<short_name>portion matches the experiment exactly. One experiment → one test file per category. Notest_<NN>_v2.py, notest_NN_<short_name>_2.py. If a test needs to evolve, edit it in place; the pairing must stay 1:1. - One subskill per category. This skill only places the empty
test file and hands off. Don't write assertion bodies, fixture
construction, or test-specific logic in this skill — that belongs
to the matching subskill (
smoke-test-ml-pipeline, etc.). - pytest is the runner. Tests are pytest tests, not
jupytext-style
# %%scripts. The experiment scripts live inexperiments/and stay# %%-style for interactive iteration; the tests are binary pass/fail and benefit from pytest's reporting. Don't mix the two conventions.
Layout this skill owns
project/
└── tests/
├── smoke/
│ ├── test_01_baseline.py # ↔ experiments/01_baseline.py
│ ├── test_02_short_name.py
│ └── ...
└── (future: regression/, distribution/, …)
The pairing rule:
journal/NN_<short_name>.md
experiments/NN_<short_name>.py
tests/<category>/test_NN_<short_name>.py
— same NN_<short_name> stem in all three. The test_ prefix on
the test file basename is the pytest naming convention; everything
after it tracks the experiment.
The tests/<category>/ subfolder lets the workspace grow more
test types without renaming anything. tests/smoke/ is the only
required category for v1.
Test categories — Dispatch table
Use the user's signal first; fall back to the defaults at the bottom.
| Situation | Subskill |
|---|---|
| Brand-new experiment was just approved; need to wire its smoke test | smoke-test-ml-pipeline |
User says "write the smoke test for 02", "the smoke test is failing", "what should the smoke test for X assert?" |
smoke-test-ml-pipeline |
User says "the metrics drifted between 02 and 03, can we lock that in?", "regression test against last week's run" |
regression-test-ml-pipeline (future — not implemented in v1) |
| User says "predictions are out of range", "calibration looks off in production", "schema invariants" | distribution-test-ml-pipeline (future — not implemented in v1) |
| User asks an open-ended "should this experiment have a test?" | Default to smoke-test-ml-pipeline — the smoke test is required at every iteration; everything else is opt-in. |
The current implementation only ships smoke-test-ml-pipeline.
Future categories will land as sibling subskills; the dispatch
table is the contract for adding them.
The required-test-per-experiment rule
Every approved experiment must have a passing smoke test before
it can be marked done in JOURNAL.md. This is enforced by
iterate-ml-experiment § 3 (after design-note approval, before script
creation) and § 4 (before recording outcome): the test is part of
the iteration loop, not an afterthought. If the smoke test fails,
the iteration that follows is on the pipeline (re-enter
build-ml-pipeline), not on the model — a smoke-test failure is
almost always a structural problem with how the DataOps graph is
laid out, not a metric problem with the predictor.
The smoke test is non-optional for every experiment in the workspace. Other test categories (regression, distribution, …) are opt-in and added when the workspace's needs warrant them.
Decision flow
- Is there an approved
journal/NN_<short_name>.mdand a matchingexperiments/NN_<short_name>.py?- No → hand off to
iterate-ml-experiment(design note first) ororganize-ml-workspace(script first). Stop. - Yes → continue.
- No → hand off to
- Identify the test category from the user's signal (Dispatch
table). Default:
smoke. - Place the empty test file at
tests/<category>/test_NN_<short_name>.pywith the pytest scaffolding (onedef test_*():function, empty body, a# TODO: filled in by <subskill>marker). If the file already exists, do not overwrite. - Hand off to the subskill (
smoke-test-ml-pipelineetc.). The subskill writes the assertions, fixture construction, and any helpers it needs. - Confirm with the user that the subskill's draft is correct before running the test (the subskill owns this loop; this skill only does placement + dispatch).
What this skill does NOT do
- Run pytest. Test execution is the user's call (or CI's).
- Write assertion bodies. Each subskill owns the assertions for its category.
- Decide whether a test is required. The required-test-per-experiment rule (smoke at every iteration) is fixed by this skill; adding optional categories is a workspace-level decision the user makes, not this skill.
- Touch files outside
tests/. The companion-skill edits toiterate-ml-experiment/organize-ml-workspace/build-ml-pipeline/evaluate-ml-pipelineare on those skills, not this one.
Companion skills
smoke-test-ml-pipeline— owns the smoke test contract (fixture construction, the diagnostic-by-construction property, assertions, failure semantics). The only required test category in v1.organize-ml-workspace— scaffoldstests/<category>/alongsidejournal//experiments/at workspace creation time. The placeholder test files are placed by this skill, not byorganize-ml-workspace.iterate-ml-experiment— drives the iteration loop. After design-note approval, dispatches to this skill (which dispatches tosmoke-test-ml-pipeline) to draft the matching test. After the experiment runs, requires the smoke test to pass before the experiment can flip todone.build-ml-pipeline— pipeline declaration. The smoke test is the executable proof thatbuild-ml-pipeline's X-marker rule (mark X early, featurize after, history references via upstream nodes) was followed. Smoke test failure typically means the pipeline shape is wrong; route back here.evaluate-ml-pipeline— owns the CV protocol. CV is necessary but not sufficient when the pipeline has history-dependent features;smoke-test-ml-pipelinefills the gap CV doesn't.data-science-python-stack— declares pytest as a Tier 1 mandatory dependency for any workspace that uses this skill.