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

Timestamp:

Jul 22, 2022 10:14:40 AM (2 years ago)

Author:

vboxsync

Message:

Validation Kit/tests/api/tdTreeDepth1.py: Added random selection of depth tests plus more logging in case something goes wrong (instead of just asserting).

File:

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

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

@@ -78 +78 @@
             assert oVM is not None
 
-            # create chain with 300 disk images (medium tree depth limit)
+            # create chain with up to 64 disk images (medium tree depth limit)
             fRc = True
             oSession = self.oTstDrv.openSession(oVM)
-            cImages = 38 #00
+            cImages = random.randrange(1, 64);
+            reporter.log('Creating chain with %d disk images' % (cImages))
             for i in range(1, cImages + 1):
                 sHddPath = os.path.join(self.oTstDrv.sScratchPath, 'Test' + str(i) + '.vdi')
@@ -121 +122 @@
             reporter.log('API reports %i base images' % (cBaseImages))
             fRc = fRc and cBaseImages == 0
+            if cBaseImages != 0:
+                reporter.error('Got %d initial base images, expected %d' % (cBaseImages, 0));
 
             # re-register to test loading of settings
@@ -138 +141 @@
             reporter.log('API reports %i base images' % (cBaseImages))
             fRc = fRc and cBaseImages == 0
-
-            assert fRc is True
+            if cBaseImages != 0:
+                reporter.error('Got %d base images after unregistering, expected %d' % (cBaseImages, 0));
+
         except:
             reporter.errorXcpt()
@@ -163 +167 @@
             fRc = fRc and oSession.saveSettings()
 
-            # take 250 snapshots (snapshot tree depth limit)
-            cSnapshots = 13 #00
+            # 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.
+            reporter.log('Taking %d snapshots' % (cSnapshots))
             for i in range(1, cSnapshots + 1):
                 fRc = fRc and oSession.takeSnapshot('Snapshot ' + str(i))
@@ -171 +176 @@
             reporter.log('API reports %i snapshots' % (oVM.snapshotCount))
             fRc = fRc and oVM.snapshotCount == cSnapshots
-
-            assert fRc is True
+            if oVM.snapshotCount != cSnapshots:
+                reporter.error('Got %d initial snapshots, expected %d' % (oVM.snapshotCount, cSnapshots));
 
             # unregister, making sure the images are closed
@@ -197 +202 @@
             reporter.log('API reports %i base images' % (cBaseImages))
             fRc = fRc and cBaseImages == 0
+            if cBaseImages != 0:
+                reporter.error('Got %d initial base images, expected %d' % (cBaseImages, 0));
 
             # re-register to test loading of settings
@@ -207 +214 @@
             reporter.log('API reports %i snapshots' % (oVM.snapshotCount))
             fRc = fRc and oVM.snapshotCount == cSnapshots
+            if oVM.snapshotCount != cSnapshots:
+                reporter.error('Got %d snapshots after re-registering, expected %d' % (oVM.snapshotCount, cSnapshots));
 
             reporter.log('unregistering VM')
@@ -215 +224 @@
             reporter.log('API reports %i base images' % (cBaseImages))
             fRc = fRc and cBaseImages == 0
-
-            assert fRc is True
+            if cBaseImages != 0:
+                reporter.error('Got %d base images after unregistering, expected %d' % (cBaseImages, 0));
         except:
             reporter.errorXcpt()