Add search self-tests (hash consistency, QSearch sanity) and a native EPD parser (`epd` command + ParseEpdRecord), fixing a silent-corruption bug in convert_epd.pl that broke every Black-to-move EPD conversion.

testsearch.c: TestSearchHashConsistency exercises the hash table through real search (repeat-with-warm-table and shallow-then-deep agreement) rather than synthetic store/probe calls. TestQSearchSanity checks termination and a node-count ceiling; its own header documents that the TEST=1 self-test build doesn't define PERF_COUNTERS, so it checks whole-search node count rather than the qsearch-only counter. A planned null-move zugzwang test was dropped in favor of tests/zugzwang.epd (best-move outcome, not an internal counter) after the three classic literature positions turned out not to structurally reach this engine's verification-search code path on this build.

fen.c/script.c/command.c: new `epd <file>` command reads raw EPD directly via ParseEpdRecord, reusing ScriptCommand's existing suite-reporting machinery (factored into _ProcessSuiteLine/_PrintSuiteReport). Replaces the need for tests/convert_epd.pl, which had a real bug: its bm/am regex's [^b]+ filler breaks on any Black-to-move record (side-to-move field is literally "b"), and unscoped $fen/$id/$solution globals meant a failed match silently duplicated the previous record instead of erroring.

tests/ep_to_epd.pl converts the existing setboard/solution/avoid/id/go suites back to real EPD; cross-checked ecm_ringers.ep_ via `script` against ecm_ringers.epd via `epd` -- byte-identical node counts and solve stats. Added .epd siblings for every genuine tactical suite in tests/.

Added src/TESTING_PLAN.md: full audit of test*.c coverage gaps (dead SEE oracle, unwired eval/recognizer tests, zero hash-table correctness coverage, thin search self-tests) with a prioritized plan for closing them.

Co-Authored-By: Claude Sonnet 5 <[email protected]>

1 files changed, 11 insertions, 0 deletions

diff --git a/tests/convert_epd.pl b/tests/convert_epd.pl
index cdd774c..65355ad 100755
--- a/tests/convert_epd.pl
+++ b/tests/convert_epd.pl
@@ -2,6 +2,17 @@
 
 # Yes, I never wrote real code to parse EPD files.  This script converts
 # "normal" style EPDs into what typhoon wants to eat.
+#
+# SUPERSEDED: the engine now parses raw EPD directly -- `epd <file>`
+# (see ParseEpdRecord in fen.c, EpdCommand in script.c) -- so this
+# preprocessing step shouldn't be needed anymore. Left here rather than
+# deleted since some workflow may still shell out to it, but it has a
+# known bug: the bm/am regex's `[^b]+` filler between the board and
+# "bm" breaks on every Black-to-move record (the side-to-move field is
+# literally the character "b"), and $fen/$id/$solution are unscoped
+# globals, so a failed match on one line silently carries the
+# *previous* line's fen/solution forward instead of erroring. Prefer
+# `epd` for any new suite.
 
 while(<>) {
   @lines = split(/;/);