Skip to content

Commit 38ce641

Browse files
authored
[CI] Fix unpacking in premerge advisor upload (#163944)
Seems like the function returns a dict, not a list of tuples like the other. Not sure of the schema or the design for this, so fixing the code without using the test-suite name. But this might have to be addressed once the owner can take a look.
1 parent fbbffc1 commit 38ce641

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

.ci/premerge_advisor_upload.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,11 @@ def main(commit_sha, workflow_run_number, build_log_files):
3232
"platform": current_platform,
3333
}
3434
if test_failures:
35-
for name, failure_message in test_failures:
36-
failure_info["failures"].append({"name": name, "message": failure_message})
35+
for _, failures in test_failures.items():
36+
for name, failure_message in failures:
37+
failure_info["failures"].append(
38+
{"name": name, "message": failure_message}
39+
)
3740
else:
3841
ninja_failures = generate_test_report_lib.find_failure_in_ninja_logs(ninja_logs)
3942
for name, failure_message in ninja_failures:

0 commit comments

Comments
 (0)