Changeset 77657 in vbox for trunk/src/VBox

Timestamp:

Mar 11, 2019 6:37:53 PM (6 years ago)

Author:

vboxsync

Message:

vsheriff: make the code checking the full result tree for failures indicating a bad testbox actually work (needed more effort to get the failure reason)

File:

• trunk/src/VBox/ValidationKit/testmanager/batch/virtual_test_sheriff.py (modified) (1 diff)

trunk/src/VBox/ValidationKit/testmanager/batch/virtual_test_sheriff.py

@@ -469 +469 @@
                         aoFailedResults = oTree.getListOfFailures();
                         for oFailedResult in aoFailedResults:
-                            if oFailedResult.idFailureReason in aidFailureReasons:
-                                oFailure = oFailedResult;
+                            oFailure = self.oTestResultFailureLogic.getById(oFailedResult.idTestResult);
+                            if oFailure is not None and oFailure.idFailureReason in aidFailureReasons:
                                 break;
+                            else:
+                                oFailure = None;
                     if oFailure is not None:
                         cBad += 1;