contest: backend: don't propagate total time to l2

L2 results start out as a copy to the top level result. This helps things like URL to get copied over, since sub-cases don't carry them. Propagating other keys, like 'time', make less sense. If we don't have per-case time we should show nothing. We had a case with MPTCP when only some cases failed to parse, and we'd show a mix of L2 times, and total times, which was super confusing. Signed-off-by: Jakub Kicinski <[email protected]>
linux-netdev · Sep 3, 2024 · aab6fac · aab6fac
1 parent a173c4f
commit aab6fac
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/contest/backend/query.py b/contest/backend/query.py
@@ -61,6 +61,8 @@ def result_as_l2(raw):
             for case in l1["results"]:
                 data = l1.copy()
                 del data["results"]
+                if "time" in data:
+                    del data["time"]
                 data |= case
                 data["test"] = l1["test"] + '.' + case["test"]
                 flat.append(data)