updates 33.1 139
This commit is contained in:
29
CLAUDE.md
29
CLAUDE.md
@@ -221,8 +221,27 @@ framework, and never something a test reaches into.
|
||||
|
||||
## Known Broken (found, not yet fixed)
|
||||
|
||||
- `tester/tests/_dev/test_health.py` imports `..endpoints`, which does not exist in
|
||||
core — the module is unimportable.
|
||||
- Client leaks remain in `station/monitors/databrowse/` (README + `index.html`) and
|
||||
`atlas/main.py` (`PAWPRINT_URL`, role labels). Core should carry no client
|
||||
vocabulary; it belongs in `cfg/<room>/`.
|
||||
- `tester/tests/example/test_health.py` imports `pytest`, which is neither a
|
||||
dependency nor allowed here — `tests/README.md` says stdlib `unittest` and
|
||||
`httpx`, no pytest. It fails to import, so `python -m tester discover` reports
|
||||
it as `_FailedTest`. It also duplicates `test_template.py`, which is the
|
||||
sanctioned example. Removing it is probably the fix; core ships no tests.
|
||||
- Old vocabulary survives in prose — `album`, `larder`, `ward`, `nest` and
|
||||
`pawprint` still appear in READMEs, `docs/`, and this file's history. Live code
|
||||
is clean; the docs lag.
|
||||
|
||||
## Fixed
|
||||
|
||||
- **A real API key was tracked** in `station/tools/tester/.env`. Untracked, `.env`
|
||||
added to `.gitignore`, `.env.example` added. **The key is still in git history
|
||||
and must be rotated** — untracking does not unpublish it.
|
||||
- Client vocabulary is out of live code. `atlas/main.py` was written entirely as
|
||||
`Album`/`larder`/`PAWPRINT_URL`; `databrowse` docs described a `larder/` the
|
||||
code stopped using; `get-api-key.sh` defaulted to the client's database name.
|
||||
- Two dead back-links, found while renaming: `atlas/main.py` passed
|
||||
`pawprint_url` and `artery/index.html` read `pawprint_url`, while `run.py`
|
||||
passes `soleprint_url`. Neither "← Soleprint" link had ever rendered.
|
||||
- `atlas/main.py` fetched `/api/data/album`; `main.py` serves `/api/data/atlas`.
|
||||
`get_data()` had been failing and returning empty lists.
|
||||
- `tester/tests/_dev/test_health.py` imported a non-existent `..endpoints`. The
|
||||
import was unused; removing it makes the module discoverable (2 tests).
|
||||
|
||||
Reference in New Issue
Block a user