Strip trailing OK from ClickHouse EXPLAIN output in tests (#121)

ClickHouse sometimes appends "OK" as a success indicator after
EXPLAIN AST output. This is not part of the actual AST and should
be stripped when comparing expected vs actual output.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: kyleconroy

parser/parser_test.go

@@ -238,6 +238,10 @@ func TestParser(t *testing.T) {
 						if idx := strings.Index(expected, "\nThe query succeeded but the server error"); idx != -1 {
 							expected = strings.TrimSpace(expected[:idx])
 						}
+						// Strip trailing "OK" line (ClickHouse success indicator, not part of AST)
+						if strings.HasSuffix(expected, "\nOK") {
+							expected = strings.TrimSpace(expected[:len(expected)-3])
+						}
 						actual := strings.TrimSpace(parser.Explain(stmts[0]))
 						// Use case-insensitive comparison since ClickHouse EXPLAIN AST has inconsistent casing
 						if !strings.EqualFold(actual, expected) {

parser/testdata/01544_errorCodeToName/metadata.json

@@ -1 +1 @@
-{"explain_todo":{"stmt3":true}}
+{}