From a1b630b61c2ab08e4699e409911a4eb2c09ab082 Mon Sep 17 00:00:00 2001 From: Ashish Sharma Date: Wed, 1 Apr 2026 04:58:29 +0000 Subject: [PATCH] fix: normalize network boot fail reasons to lists -store pass/fail reasons as lists in network_boot parser -prevent waiver post-processing from iterating over characters -align network boot JSON structure with other parsers Signed-off-by: Ashish Sharma ashish.sharma2@arm.com Change-Id: I12d9d1a85cf7c040478dc234bcd18b7d5b316937 --- common/log_parser/standalone_tests/logs_to_json.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/common/log_parser/standalone_tests/logs_to_json.py b/common/log_parser/standalone_tests/logs_to_json.py index cf7abd05..e31a8d08 100755 --- a/common/log_parser/standalone_tests/logs_to_json.py +++ b/common/log_parser/standalone_tests/logs_to_json.py @@ -1227,7 +1227,7 @@ def parse_network_boot_log(log_data): status = match_explicit.group(2).strip() reason = match_explicit.group(3).strip() if match_explicit.group(3) else "" - # Create subtest with string reasons instead of arrays + # Create subtest with reason lists to match other parsers subtest = { "sub_Test_Number": str(subtest_number), "sub_Test_Description": test_name, @@ -1242,12 +1242,12 @@ def parse_network_boot_log(log_data): } } - # Add reason as string (not array) only if present + # Add reason as list to keep downstream waiver logic consistent if reason: if status == "PASSED": - subtest["sub_test_result"]["pass_reasons"] = reason + subtest["sub_test_result"]["pass_reasons"] = [reason] elif status == "FAILED": - subtest["sub_test_result"]["fail_reasons"] = reason + subtest["sub_test_result"]["fail_reasons"] = [reason] current_test["subtests"].append(subtest) current_test["test_suite_summary"][f"total_{status.lower()}"] += 1