Changeset 98826 in vbox for trunk/src/VBox/ValidationKit/tests/api/tdTreeDepth1.py

Timestamp:

Mar 3, 2023 10:53:30 AM (2 years ago)

Author:

vboxsync

Message:

ValidationKit/tests/api/tdTreeDepth1.py: The test case calls
VirtualBox::openMachine() to supposedly re-register the VM but
this only opens the VM and does not register it. Add the missing
IVirtualBox::registerMachine() calls to register the opened VM.

File:

• trunk/src/VBox/ValidationKit/tests/api/tdTreeDepth1.py (modified) (3 diffs)

trunk/src/VBox/ValidationKit/tests/api/tdTreeDepth1.py

@@ -142 +142 @@
             else:
                 oVM = oVBox.openMachine(sSettingsFile)
+            oVBox.registerMachine(oVM);
             ## @todo r=klaus: count known hard disk images, should be cImages
 
@@ -177 +178 @@
             fRc = fRc and oSession.saveSettings()
 
-            # take up to 200 snapshots (255 is the snapshot tree depth limit)
-            cSnapshots = random.randrange(1, 200); ## @todo r=andy BUGBUG When specifying 254 here, it fails with object 251.
+            # take up to 200 snapshots (250 is the snapshot tree depth limit (settings.h:SETTINGS_SNAPSHOT_DEPTH_MAX))
+            cSnapshots = random.randrange(1, 200);
             reporter.log('Taking %d snapshots' % (cSnapshots))
             for i in range(1, cSnapshots + 1):
@@ -222 +223 @@
             else:
                 oVM = oVBox.openMachine(sSettingsFile)
+            oVBox.registerMachine(oVM);
             reporter.log('API reports %i snapshots' % (oVM.snapshotCount))
             fRc = fRc and oVM.snapshotCount == cSnapshots