Changeset 67032 in vbox for trunk

Timestamp:

May 23, 2017 10:00:28 AM (8 years ago)

Author:

vboxsync

Message:

ValidationKit: Makes no sense to add task A and remove task B (and oSession is already added by starting the VM and will be removed when terminating it). Causes a leftover SessionWrapper, which keeps API objects alive.

File:

• trunk/src/VBox/ValidationKit/tests/additions/tdAddBasic1.py (modified) (4 diffs)

trunk/src/VBox/ValidationKit/tests/additions/tdAddBasic1.py

@@ -57 +57 @@
     """
     ## @todo
-    # - More of the settings stuff can e and need to be generalized!
+    # - More of the settings stuff can be and need to be generalized!
     #
 
@@ -128 +128 @@
                                                                   sFileCdWait = 'AUTORUN.INF');
         if oSession is not None:
-            self.addTask(oSession);
+            self.addTask(oTxsSession);
             # Do the testing.
             reporter.testStart('Install');
@@ -146 +146 @@
             reporter.testDone(fSkip);
 
-            ## @todo Save an restore test.
+            ## @todo Save and restore test.
 
             ## @todo Reset tests.
@@ -154 +154 @@
             # Cleanup.
             self.removeTask(oTxsSession);
-            #self.terminateVmBySession(oSession)
+            self.terminateVmBySession(oSession)
         return fRc;