Skip to content

test: add pytest suite alongside doctests - #21

Merged
m-messer merged 1 commit into
mainfrom
test-harness
Sep 15, 2026
Merged

m-messer merged 1 commit into
mainfrom
test-harness

Conversation

@m-messer

@m-messer m-messer commented Sep 1, 2026

Copy link
Copy Markdown
Member

Adds a real tests/ suite so the project no longer relies on doctests alone.

  • tests/test_runner.py runs each built-in filter over its own example.tex end to end, asserting on the returned Set and on the JSON/ZIP written to disk.
  • [tool.pytest.ini_options] collects both tests/ and the package doctests, so a bare pytest covers everything.
  • CI: black .black --check . (no longer silently reformats); isort/pydocstyle now also cover tests/.
  • Applies black to two pre-existing files (visibility_status.py, json_convert.py) that were not clean under black --check.

Stack (merge bottom-up): test-harness ← validator ← markdown-filter ← cli-group ← llm-client ← mathpix ← wizard-command
Base: main

🤖 Generated with Claude Code

Adds a real tests/ suite so the project no longer relies on doctests alone:

- tests/test_runner.py runs each built-in filter over its own example.tex
  end to end, asserting on the returned Set and on the JSON/ZIP written to
  disk.
- [tool.pytest.ini_options] collects both tests/ and the package doctests,
  so a bare `pytest` covers everything.
- CI: `black .` -> `black --check .` (no longer silently reformats), and
  isort/pydocstyle now also cover tests/.

Applies black to two pre-existing files (visibility_status.py,
json_convert.py) that were not clean under `black --check`.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017VXb8aZqgFBjoeuuddjW6r
@peterbjohnson

Copy link
Copy Markdown
Member

LGTM

@m-messer
m-messer merged commit 921ec12 into main Sep 15, 2026
4 checks passed
@m-messer
m-messer deleted the test-harness branch September 15, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants