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

Timestamp:

Jan 14, 2023 10:47:51 AM (23 months ago)

Author:

vboxsync

Message:

Validation Kit/tests/tdCloneMedium1: Various pylint fixes. bugref:10090

File:

• trunk/src/VBox/ValidationKit/tests/api/tdCloneMedium1.py (modified) (5 diffs)

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

@@ -43 +43 @@
 # Standard Python imports.
 import os
-from subprocess import call
 import sys
 
@@ -78 +77 @@
     #
 
-    def createTestMedium(self, oVM, sPathSuffix, sFmt = 'VDI', cbSize = 1024*1024, data = []):
+    def createTestMedium(self, oVM, sPathSuffix, sFmt = 'VDI', cbSize = 1024*1024, data):
         assert oVM is not None
 
@@ -101 +100 @@
         oMediumIOBaseHdd1 = oBaseHdd1.openForIO(True, "")
 
-        if(len(data) > 0):
+        if data and len(data):
             cbWritten = oMediumIOBaseHdd1.write(0, data)
 
@@ -134 +133 @@
             oProgressCom = oSrcHd.resizeAndCloneTo(oTgtHd, cbTgtSize, (vboxcon.MediumVariant_Standard, ), None);
         except:
-            reporter.errorXcpt('failed to resize and clone medium %s to %s and to size %d' % (oSrcHd.name, oTgtHd.name, cbTgtSize));
+            reporter.errorXcpt('failed to resize and clone medium %s to %s and to size %d' \
+                               % (oSrcHd.name, oTgtHd.name, cbTgtSize));
             return False;
         oProgress = vboxwrappers.ProgressWrapper(oProgressCom, self.oTstDrv.oVBoxMgr, self.oTstDrv,
-                                                 'resize and clone base disk %s to %s and to size %d' % (oSrcHd.name, oTgtHd.name, cbTgtSize));
+                                                 'resize and clone base disk %s to %s and to size %d' \
+                                                 % (oSrcHd.name, oTgtHd.name, cbTgtSize));
         oProgress.wait(cMsTimeout = 15*60*1000); # 15 min
         oProgress.logResult();
@@ -235 +236 @@
 
     def testAll(self):
-        return (self.testCloneOnly() & self.testResizeAndClone())
+        return self.testCloneOnly() & self.testResizeAndClone()
 
 if __name__ == '__main__':