diff mbox series

[v2,2/4] scripts/resulttool: rearrange regressions report order

Message ID 20231022174938.7012-3-alexis.lothore@bootlin.com
State Accepted, archived
Commit 599267467430e70fa4dc8ba6b2a8b126bf6da359
Headers show
Series Add a display limit for regression report generation | expand

Commit Message

Alexis Lothoré Oct. 22, 2023, 5:49 p.m. UTC 
From: Alexis Lothoré <alexis.lothore@bootlin.com>

Regressions reports currently reports matching pairs and improved pairs
first, then regressions.

Change order to print regressions first, which is the most valuable info in
the report, and then print improvements and matches at the bottom.

Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
---
 scripts/lib/resulttool/regression.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/scripts/lib/resulttool/regression.py b/scripts/lib/resulttool/regression.py
index d98504f4ced8..560d102e172c 100644
--- a/scripts/lib/resulttool/regression.py
+++ b/scripts/lib/resulttool/regression.py
@@ -80,6 +80,9 @@  STATUS_STRINGS = {
 
 REGRESSIONS_DISPLAY_LIMIT=50
 
+MISSING_TESTS_BANNER =   "-------------------------- Missing tests --------------------------"
+ADDITIONAL_DATA_BANNER = "--------------------- Matches and improvements --------------------"
+
 def test_has_at_least_one_matching_tag(test, tag_list):
     return "oetags" in test and any(oetag in tag_list for oetag in test["oetags"])
 
@@ -312,10 +315,11 @@  def regression_common(args, logger, base_results, target_results):
                         regressions.append(resstr)
         else:
             notfound.append("%s not found in target" % a)
-    print("\n".join(sorted(matches)))
-    print("\n")
     print("\n".join(sorted(regressions)))
+    print("\n" + MISSING_TESTS_BANNER + "\n")
     print("\n".join(sorted(notfound)))
+    print("\n" + ADDITIONAL_DATA_BANNER + "\n")
+    print("\n".join(sorted(matches)))
     return 0
 
 def regression_git(args, logger):