improve new log messages

strtgbb · strtgbb · commit 573a73bf3ae0 · 2025-12-19T11:49:17.000-05:00
diff --git a/tests/clickhouse-test b/tests/clickhouse-test
@@ -167,7 +167,7 @@ def test_is_known_fail(test_name, test_logs, build_flags, broken_tests_file_path
                 ct in build_flags for ct in rule_data["check_types"]
             ):
                 log_file.write(
-                    f"Check types didn't match: '{rule_data['check_types']}' not in '{build_flags}'\n"
+                    f"Skip rule: Check types didn't match: '{rule_data['check_types']}' not in '{build_flags}'\n"
                 )
                 continue  # check_types didn't match → skip rule
 
@@ -185,7 +185,7 @@ def test_is_known_fail(test_name, test_logs, build_flags, broken_tests_file_path
                 )
                 continue
 
-            log_file.write(f"Test {test_name} matched rule: {rule_data}\n")
+            log_file.write(f"Matched rule: {rule_data}\n")
             return rule_data["reason"]
 
         return False

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,7 @@ def test_is_known_fail(test_name, test_logs, build_flags, broken_tests_file_path`
`167`	`167`	`ct in build_flags for ct in rule_data["check_types"]`
`168`	`168`	`):`
`169`	`169`	`log_file.write(`
`170`		`- f"Check types didn't match: '{rule_data['check_types']}' not in '{build_flags}'\n"`
	`170`	`+ f"Skip rule: Check types didn't match: '{rule_data['check_types']}' not in '{build_flags}'\n"`
`171`	`171`	`)`
`172`	`172`	`continue # check_types didn't match → skip rule`
`173`	`173`
`@@ -185,7 +185,7 @@ def test_is_known_fail(test_name, test_logs, build_flags, broken_tests_file_path`
`185`	`185`	`)`
`186`	`186`	`continue`
`187`	`187`
`188`		`- log_file.write(f"Test {test_name} matched rule: {rule_data}\n")`
	`188`	`+ log_file.write(f"Matched rule: {rule_data}\n")`
`189`	`189`	`return rule_data["reason"]`
`190`	`190`
`191`	`191`	`return False`