Skip to content

test: add preprocessor drift fixtures#164

Merged
rlippmann merged 1 commit into
mainfrom
codex/preprocessor-fixture-drift
May 31, 2026
Merged

test: add preprocessor drift fixtures#164
rlippmann merged 1 commit into
mainfrom
codex/preprocessor-fixture-drift

Conversation

@rlippmann
Copy link
Copy Markdown
Owner

What changed

  • Added 10 shared preprocessor heuristic fixtures under tests/fixtures/preprocessor/ to cover regular-test-only Python cases.
  • Added fixture coverage for mixed-intent, discourse-marker/meta-prefix, near-miss, modal-prefix, and no-directive question behavior currently asserted in Python tests.
  • Included the confirmed drift case: use docker, actually prohibit docker -> unknown.

Why

  • Shared preprocessor conformance fixtures had drifted behind Python regular tests for several explicitly asserted behaviors.
  • These fixture additions make those behaviors part of shared source-of-truth fixture coverage, reducing parity gaps across ports.

Checklist

  • pre-commit run (uv run pre-commit run --all-files)
  • tests pass (uv run pytest)

@rlippmann rlippmann merged commit b0e6f1b into main May 31, 2026
14 checks passed
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.

1 participant