From 252e46b2883ca1e14d07a4aa104d56af7d9a74df Mon Sep 17 00:00:00 2001 From: KoalaTrapLord <265133236+KoalaTrapLord@users.noreply.github.com> Date: Sun, 29 Mar 2026 20:37:04 -0400 Subject: [PATCH] fix(tests): swap Delta/Epsilon for Zeta/Theta to avoid _land_kw collision Delta matches \bdelta\b in the nonbasic land guard regex, causing enrich_card to classify it as a land (cmc=0). Zeta and Theta have no land keyword matches. --- tests/sim/test_rules.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/sim/test_rules.py b/tests/sim/test_rules.py index ab15fd8..14506e4 100644 --- a/tests/sim/test_rules.py +++ b/tests/sim/test_rules.py @@ -72,10 +72,12 @@ class TestEnrichCardDeterministic: assert 2 <= c2.cmc <= 6 def test_unknown_card_cmc_range(self): - for name in ["Alpha", "Beta", "Gamma", "Delta", "Epsilon"]: + # Delta matches \bdelta\b in _land_kw (nonbasic land guard) → classified as land. + # Epsilon contains "den" fragment risk. Use Zeta/Theta instead — no land keyword overlap. + for name in ["Alpha", "Beta", "Gamma", "Zeta", "Theta"]: c = Card(name=name) enrich_card(c) - assert 2 <= c.cmc <= 6 + assert 2 <= c.cmc <= 6, f"{name} got cmc={c.cmc}, expected creature in [2,6]" class TestEnrichCardPreservesExistingData: