Changeset 92980 in vbox

Timestamp:

Dec 16, 2021 12:47:18 PM (3 years ago)

Author:

vboxsync

Message:

Validation Kit/testdriver: Handle error cases in addTestMachine() where either the API has not been initialized yet or the VM has not been found.

File:

• trunk/src/VBox/ValidationKit/testdriver/vbox.py (modified) (2 diffs)

trunk/src/VBox/ValidationKit/testdriver/vbox.py

@@ -2621 +2621 @@
         Adds an already existing (that is, configured) test VM to the
         test VM list.
+
+        Returns the VM object on success, None if failed.
         """
         # find + add the VM to the list.
+        oVM = None;
         try:
             if self.fpApiVer >= 4.0:
@@ -2630 +2633 @@
         except:
             reporter.errorXcpt('could not find vm "%s"' % (sNameOrId,));
-            return None;
-
-        self.aoVMs.append(oVM);
-        if not fQuiet:
-            reporter.log('Added "%s" with name "%s"' % (oVM.id, sNameOrId));
-            self.logVmInfo(oVM);
+
+        if oVM:
+            self.aoVMs.append(oVM);
+            if not fQuiet:
+                reporter.log('Added "%s" with name "%s"' % (oVM.id, sNameOrId));
+                self.logVmInfo(oVM);
         return oVM;