Changeset 93037 in vbox for trunk/src/VBox/ValidationKit/tests

Timestamp:

Dec 21, 2021 10:40:28 AM (3 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

149040

Message:

Validation Kit/Guest Control: Added more checks for testGuestCtrlSessionReboot / "session_reboot" testcase.

File:

• trunk/src/VBox/ValidationKit/tests/additions/tdAddGuestCtrl.py (modified) (3 diffs)

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

@@ -3243 +3243 @@
         # Create a process.
         #
+        # That process will also be used later to see if the session cleanup worked upon reboot.
+        #
         sImage = self.oTstDrv.getGuestSystemShell(oTestVm);
         asArgs  = [ sImage, ];
@@ -3279 +3281 @@
                         oThreadReboot.start();
 
-                        # Not sure why this fudge is needed...
-                        reporter.log('5 second wait fudge before triggering reboot ...');
-                        self.oTstDrv.sleep(5);
-
                         # Do the reboot.
                         reporter.log('Rebooting guest and reconnecting TXS ...');
@@ -3295 +3293 @@
                         oThreadReboot.join();
 
+                        # Check that the guest session now still has the formerly guest process object created above,
+                        # but with the "down" status now (because of guest reboot).
+                        try:
+                            aoGuestProcs = self.oTstDrv.oVBoxMgr.getArray(oGuestSession, 'processes');
+                            if len(aoGuestProcs) == 1:
+                                enmProcSts = aoGuestProcs[0].status;
+                                if enmProcSts != vboxcon.ProcessStatus_Down:
+                                    fRc = reporter.error('Old guest process (before reboot) has status %d, expected %s' \
+                                                         % enmProcSts, vboxcon.ProcessStatus_Down);
+                            else:
+                                fRc = reporter.error('Old guest session (before reboot) has %d processes registered, expected 1' \
+                                                     % (len(aoGuestProcs)));
+                        except:
+                            fRc = reporter.errorXcpt();
             #
             # Try make sure we don't leave with a stale process on failure.