Changeset 79087 in vbox for trunk/src/VBox/ValidationKit/testmanager/core/testboxcontroller.py

Timestamp:

Jun 11, 2019 11:58:28 AM (6 years ago)

Author:

vboxsync

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

131247

Message:

ValKit: New pylint version - cleanup in progress.

File:

• trunk/src/VBox/ValidationKit/testmanager/core/testboxcontroller.py (modified) (7 diffs)

trunk/src/VBox/ValidationKit/testmanager/core/testboxcontroller.py

@@ -33 +33 @@
 import re;
 import os;
-import string;                          # pylint: disable=W0402
+import string;                          # pylint: disable=deprecated-module
 import sys;
 import uuid;
@@ -53 +53 @@
 # Python 3 hacks:
 if sys.version_info[0] >= 3:
-    long = int;     # pylint: disable=W0622,C0103
+    long = int;     # pylint: disable=redefined-builtin,invalid-name
 
 
@@ -60 +60 @@
     Exception class for TestBoxController.
     """
-    pass;
-
-
-class TestBoxController(object): # pylint: disable=R0903
+    pass;                               # pylint: disable=unnecessary-pass
+
+
+class TestBoxController(object): # pylint: disable=too-few-public-methods
     """
     TestBox Controller class.
@@ -190 +190 @@
         """
         sValue = self._getStringParam(sName, [ 'True', 'true', '1', 'False', 'false', '0'], sDefValue = str(fDefValue));
-        return sValue == 'True' or sValue == 'true' or sValue == '1';
+        return sValue in ('True', 'true', '1',);
 
     def _getIntParam(self, sName, iMin = None, iMax = None):
@@ -346 +346 @@
         return fSizeOk;
 
-    def _actionSignOn(self):        # pylint: disable=R0914
+    def _actionSignOn(self):        # pylint: disable=too-many-locals
         """ Implement sign-on """
 
@@ -401 +401 @@
             cPctScratchDiff = 100;
 
-        # pylint: disable=R0916
+        # pylint: disable=too-many-boolean-expressions
         if   self._sTestBoxAddr != oTestBox.ip \
           or sOs                != oTestBox.sOs \
@@ -738 +738 @@
 
             abBuf = oSrcFile.read(cbToRead);
-            oDstFile.write(abBuf); # pylint: disable=E1103
+            oDstFile.write(abBuf); # pylint: disable=maybe-no-member
             del abBuf;
 
-        oDstFile.close(); # pylint: disable=E1103
+        oDstFile.close(); # pylint: disable=maybe-no-member
 
         # Done.