From 4a65637c4a5a47afda56c77db991af72ccef0b25 Mon Sep 17 00:00:00 2001 From: David Arnold Date: Fri, 11 Oct 2024 13:20:27 +0200 Subject: [PATCH] fix: persistent per-site test record creation log file name (#28089) * fix: persistent per-site test record creation log file name * fix: persistent per-site test record creation log erasure --- frappe/tests/utils/generators.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/frappe/tests/utils/generators.py b/frappe/tests/utils/generators.py index d3da763cbc..3292de3446 100644 --- a/frappe/tests/utils/generators.py +++ b/frappe/tests/utils/generators.py @@ -263,9 +263,12 @@ def print_mandatory_fields(doctype): testing_logger.warning(" | ".join(msg.strip().splitlines())) +PERSISTENT_TEST_LOG_FILE = ".test_records.jsonl" + + class TestRecordLog: def __init__(self): - self.log_file = Path(frappe.get_site_path(".test_log.jsonl")) + self.log_file = Path(frappe.get_site_path(PERSISTENT_TEST_LOG_FILE)) self._log = None def get(self): @@ -303,8 +306,9 @@ class TestRecordLog: def _after_install_clear_test_log(): - with open(frappe.get_site_path(".test_log"), "w") as f: - f.write("{}") + log_file_path = frappe.get_site_path(PERSISTENT_TEST_LOG_FILE) + if os.path.exists(log_file_path): + os.remove(log_file_path) def make_test_records(doctype, force=False, commit=False):