core/tests/test_math_problem_graph.py
Shay d17fec6801 fix(math-graph): refuse contradictory initial possessions (wrong=0 hazard)
MathProblemGraph.__post_init__ now raises MathGraphError when two
InitialPossession entries share the same (entity, unit) key but
declare different quantity values.

Pre-fix behavior surfaced by 2026-05-28 ADR-0174 Phase 3 post-merge
diagnostic: math_solver.solve() line 207 used last-write-wins dict
assignment when consolidating initial state. Two contradictory
inputs would silently overwrite without trace:

  'Sam has 5 marbles. Sam has 3 marbles. How many marbles does Sam have?'
   → returned 3.0 (wrong=0 violation: definite answer from
     contradictory input)

Post-fix: same input refuses with 'no branch produced a solvable
graph' — refusal-preferring discipline as wrong=0 doctrine requires.

Identical duplicates (same value) are admitted as redundant (no
contradiction). Different units for same actor admitted. Different
actors for same unit admitted. Single-value cases (the dominant
real-world pattern) unchanged.

This is an extraction-layer hazard discovered while investigating
Phase 3b scope: Phase 3b compound-clause held hypotheses would
emit multiple CandidateInitial entries per sentence, exercising
exactly this consolidation path. Fixing the silent overwrite NOW
ensures Phase 3b admission doesn't silently produce wrong answers.

Acceptance:
- 4 new tests in TestContradictoryInitialPossessionsRefuse
- 165/165 test_math_problem_graph tests pass (was 161/161)
- Smoke 67/67, packs 141/141 unchanged
- train_sample 3/47/0 unchanged (no real case exercised the
  overwrite — but the hazard was latent)

References: CLAUDE.md §Lookback Review Discipline (the doctrine
that surfaced this), CLAUDE.md §Non-Negotiable Field Invariant
(make illegal states difficult to represent).
2026-05-28 09:51:14 -07:00

266 lines
9.5 KiB
Python

"""ADR-0115 Phase 1.1 — math problem graph schema invariants.
Pins:
1. The five seed cases in ``evals/gsm8k_parser_dev/cases.jsonl`` round-trip
through ``graph_from_dict`` → ``as_json`` without changing bytes.
2. ``MathProblemGraph.canonical_bytes()`` is deterministic: same logical
graph constructed twice produces identical bytes.
3. Construction-time validation refuses malformed graphs.
4. Pyhand-solving each seed case from its ground-truth graph reproduces the
``expected_answer`` — this catches mis-authored ground-truth graphs.
"""
from __future__ import annotations
import json
from pathlib import Path
import pytest
from generate.math_problem_graph import (
InitialPossession,
MathGraphError,
MathProblemGraph,
Operation,
Quantity,
Unknown,
graph_from_dict,
)
_REPO_ROOT = Path(__file__).resolve().parent.parent
_CASES = _REPO_ROOT / "evals" / "gsm8k_parser_dev" / "cases.jsonl"
def _load_cases() -> list[dict]:
return [json.loads(line) for line in _CASES.read_text().splitlines() if line.strip()]
class TestSeedCasesRoundTrip:
@pytest.mark.parametrize("case", _load_cases(), ids=lambda c: c["id"])
def test_graph_loads(self, case: dict) -> None:
graph = graph_from_dict(case["ground_truth_graph"])
assert isinstance(graph, MathProblemGraph)
@pytest.mark.parametrize("case", _load_cases(), ids=lambda c: c["id"])
def test_round_trip_byte_equal(self, case: dict) -> None:
graph = graph_from_dict(case["ground_truth_graph"])
reloaded = graph_from_dict(graph.as_json())
assert graph.canonical_bytes() == reloaded.canonical_bytes()
class TestCanonicalBytesDeterminism:
def test_two_identical_graphs_produce_identical_bytes(self) -> None:
g1 = MathProblemGraph(
entities=("Sam",),
initial_state=(
InitialPossession("Sam", Quantity(5, "apples")),
),
operations=(Operation("Sam", "add", Quantity(3, "apples")),),
unknown=Unknown("Sam", "apples"),
)
g2 = MathProblemGraph(
entities=("Sam",),
initial_state=(
InitialPossession("Sam", Quantity(5, "apples")),
),
operations=(Operation("Sam", "add", Quantity(3, "apples")),),
unknown=Unknown("Sam", "apples"),
)
assert g1.canonical_bytes() == g2.canonical_bytes()
assert g1 == g2
class TestSchemaRejectsMalformed:
def test_quantity_rejects_string_value(self) -> None:
with pytest.raises(MathGraphError):
Quantity("5", "apples") # type: ignore[arg-type]
def test_quantity_rejects_empty_unit(self) -> None:
with pytest.raises(MathGraphError):
Quantity(5, "")
def test_operation_rejects_unknown_kind(self) -> None:
with pytest.raises(MathGraphError):
Operation("Sam", "explode", Quantity(3, "apples"))
def test_transfer_requires_target(self) -> None:
with pytest.raises(MathGraphError):
Operation("Sam", "transfer", Quantity(3, "apples"))
def test_non_transfer_rejects_target(self) -> None:
with pytest.raises(MathGraphError):
Operation("Sam", "add", Quantity(3, "apples"), target="Tom")
def test_transfer_self_rejected(self) -> None:
with pytest.raises(MathGraphError):
Operation("Sam", "transfer", Quantity(3, "apples"), target="Sam")
def test_graph_rejects_duplicate_entities(self) -> None:
with pytest.raises(MathGraphError):
MathProblemGraph(
entities=("Sam", "Sam"),
initial_state=(),
operations=(),
unknown=Unknown("Sam", "apples"),
)
def test_graph_rejects_unknown_entity_in_initial(self) -> None:
with pytest.raises(MathGraphError):
MathProblemGraph(
entities=("Sam",),
initial_state=(InitialPossession("Tom", Quantity(5, "apples")),),
operations=(),
unknown=Unknown("Sam", "apples"),
)
def test_graph_rejects_unknown_entity_in_question(self) -> None:
with pytest.raises(MathGraphError):
MathProblemGraph(
entities=("Sam",),
initial_state=(),
operations=(),
unknown=Unknown("Tom", "apples"),
)
def _hand_solve(graph: MathProblemGraph) -> tuple[float, str]:
"""Reference solver — ADR-0116 supersedes this with a real solver.
Used here only to falsify mis-authored ground-truth graphs in the seed
set. Sufficient for the patterns Phase 1.1 covers.
"""
state: dict[tuple[str, str], float] = {}
for p in graph.initial_state:
state[(p.entity, p.quantity.unit)] = float(p.quantity.value)
for op in graph.operations:
key = (op.actor, op.operand.unit)
cur = state.get(key, 0.0)
v = float(op.operand.value)
if op.kind == "add":
state[key] = cur + v
elif op.kind == "subtract":
state[key] = cur - v
elif op.kind == "transfer":
assert op.target is not None
state[key] = cur - v
tgt_key = (op.target, op.operand.unit)
state[tgt_key] = state.get(tgt_key, 0.0) + v
elif op.kind == "multiply":
state[key] = cur * v
elif op.kind == "divide":
state[key] = cur / v
if graph.unknown.entity is None:
total = sum(
v for (_, unit), v in state.items() if unit == graph.unknown.unit
)
return total, graph.unknown.unit
return state[(graph.unknown.entity, graph.unknown.unit)], graph.unknown.unit
class TestGroundTruthGraphsAgreeWithExpectedAnswers:
"""Falsifies mis-authored seed cases.
For each seed case, hand-solving the ground-truth graph using the
documented operation semantics must reproduce ``expected_answer`` and
``expected_unit``.
"""
@pytest.mark.parametrize("case", _load_cases(), ids=lambda c: c["id"])
def test_hand_solve_matches_expected(self, case: dict) -> None:
graph = graph_from_dict(case["ground_truth_graph"])
computed, unit = _hand_solve(graph)
assert unit == case["expected_unit"], (
f"{case['id']}: unit mismatch — graph says {unit!r}, "
f"expected {case['expected_unit']!r}"
)
# Accept int/float equivalence; problems are integer-valued.
assert computed == case["expected_answer"], (
f"{case['id']}: hand-solve produced {computed} but case "
f"declared expected_answer={case['expected_answer']}"
)
class TestContradictoryInitialPossessionsRefuse:
"""ADR-0174 Phase 3 post-merge diagnostic — surfaced 2026-05-28.
Before this fix, two contradictory initial possessions for the
same (entity, unit) silently overwrote each other in
math_solver.solve()'s state dict (line 207: last-write-wins
semantics). 'Sam has 5 marbles. Sam has 3 marbles.' would return
3.0 — a wrong=0 violation (definite answer from genuinely
contradictory input).
Fix: MathProblemGraph.__post_init__ now raises MathGraphError on
contradictory (entity, unit) initial possessions. Identical
duplicates are admitted (redundant but not contradictory).
"""
def _ip(self, entity: str, value: int, unit: str) -> InitialPossession:
return InitialPossession(
entity=entity, quantity=Quantity(value=value, unit=unit)
)
def test_contradictory_initial_possessions_refused(self) -> None:
with pytest.raises(MathGraphError, match="contradictory possessions"):
MathProblemGraph(
entities=("Sam",),
initial_state=(
self._ip("Sam", 5, "marbles"),
self._ip("Sam", 3, "marbles"),
),
operations=(),
unknown=Unknown(entity="Sam", unit="marbles"),
)
def test_identical_duplicate_initial_admitted(self) -> None:
# Redundant but not contradictory — must admit.
g = MathProblemGraph(
entities=("Sam",),
initial_state=(
self._ip("Sam", 5, "marbles"),
self._ip("Sam", 5, "marbles"),
),
operations=(),
unknown=Unknown(entity="Sam", unit="marbles"),
)
assert len(g.initial_state) == 2
def test_different_units_same_actor_admitted(self) -> None:
# Sam has apples AND Sam has oranges — no contradiction.
g = MathProblemGraph(
entities=("Sam",),
initial_state=(
self._ip("Sam", 5, "apples"),
self._ip("Sam", 3, "oranges"),
),
operations=(),
unknown=Unknown(entity="Sam", unit="apples"),
)
assert len(g.initial_state) == 2
def test_different_actors_same_unit_admitted(self) -> None:
# Sam has marbles AND Tom has marbles — different keys.
g = MathProblemGraph(
entities=("Sam", "Tom"),
initial_state=(
self._ip("Sam", 5, "marbles"),
self._ip("Tom", 3, "marbles"),
),
operations=(),
unknown=Unknown(entity="Sam", unit="marbles"),
)
assert len(g.initial_state) == 2
class TestCaseIdsAreSequential:
def test_ids_are_gpd_zero_padded_sequential(self) -> None:
cases = _load_cases()
for i, c in enumerate(cases, start=1):
assert c["id"] == f"gpd-{i:03d}", (
f"case {i}: expected id 'gpd-{i:03d}', got {c['id']!r}"
)