Refactor test toggle logic.

pekkaklarck · pekkaklarck · commit 56a770c44931 · 2025-06-16T16:30:31.000+03:00
Avoids expanding failed/skipped tests when closing element. Fixes #5322. See also PR #5445 that provided the initial fix.
diff --git a/src/robot/htmldata/rebot/log.js b/src/robot/htmldata/rebot/log.js
@@ -8,11 +8,9 @@ function toggleTest(testId) {
     var test = window.testdata.findLoaded(testId);
     var autoExpand = test.status == "FAIL" || test.status == "SKIP";
     var closed = $('#' + testId).children('.element-header').hasClass('closed');
-
-    if (autoExpand)
+    if (autoExpand && closed)
         expandFailed(test);
-
-    if (!autoExpand || !closed)
+    else
         toggleElement(testId, ['keyword']);
 }
 
