diff options
| author | Aiden Grossman <aidengrossman@google.com> | 2025-11-05 16:43:02 +0000 |
|---|---|---|
| committer | Aiden Grossman <aidengrossman@google.com> | 2025-11-05 16:43:02 +0000 |
| commit | 2fbb8cbf28990b806dd55cb8db3b0816c3749516 (patch) | |
| tree | e0afddbb1e8cd249ba579ff06d15050242eef064 | |
| parent | 0b5a00aab73967026292d0c1c84b87fabdcc2648 (diff) | |
[𝘀𝗽𝗿] changes to main this commit is based onusers/boomanaiden154/main.ci-add-ability-to-explain-failures
Created using spr 1.3.7
[skip ci]
| -rw-r--r-- | .ci/generate_test_report_lib.py | 23 |
1 files changed, 6 insertions, 17 deletions
diff --git a/.ci/generate_test_report_lib.py b/.ci/generate_test_report_lib.py index 0c025c561f6f..82752aae66ad 100644 --- a/.ci/generate_test_report_lib.py +++ b/.ci/generate_test_report_lib.py @@ -82,10 +82,10 @@ def find_failure_in_ninja_logs(ninja_logs: list[list[str]]) -> list[tuple[str, s return failures -def _format_ninja_failures(ninja_failures: list[tuple[str, str]]) -> list[str]: - """Formats ninja failures into summary views for the report.""" +def _format_failures(failures: list[tuple[str, str]]) -> list[str]: + """Formats failures into summary views for the report.""" output = [] - for build_failure in ninja_failures: + for build_failure in failures: failed_action, failure_message = build_failure output.extend( [ @@ -176,7 +176,7 @@ def generate_report( "", ] ) - report.extend(_format_ninja_failures(ninja_failures)) + report.extend(_format_failures(ninja_failures)) report.extend( [ "", @@ -212,18 +212,7 @@ def generate_report( for testsuite_name, failures in failures.items(): report.extend(["", f"### {testsuite_name}"]) - for name, output in failures: - report.extend( - [ - "<details>", - f"<summary>{name}</summary>", - "", - "```", - output, - "```", - "</details>", - ] - ) + report.extend(_format_failures(failures)) elif return_code != 0: # No tests failed but the build was in a failed state. Bring this to the user's # attention. @@ -248,7 +237,7 @@ def generate_report( "", ] ) - report.extend(_format_ninja_failures(ninja_failures)) + report.extend(_format_failures(ninja_failures)) if failures or return_code != 0: report.extend(["", UNRELATED_FAILURES_STR]) |