VirtualBox

Changeset 65967 in vbox


Ignore:
Timestamp:
Mar 7, 2017 10:54:16 AM (8 years ago)
Author:
vboxsync
Message:

ValidationKit/testdriver: pylint 2.0.0 fixes.

Location:
trunk/src/VBox/ValidationKit/testdriver
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/ValidationKit/testdriver/base.py

    r65732 r65967  
    306306                return False;
    307307            sCurName = sCurName.strip();
    308             if sCurName is '':
     308            if sCurName == '':
    309309                return False;
    310310
    … …  
    10941094                return self.pollTasks();
    10951095
    1096             if len(self.aoTasks) == 0:
     1096            if not self.aoTasks:
    10971097                return None;
    10981098
    … …  
    15441544                        del dPids[iPid];
    15451545
    1546                 if len(dPids) == 0:
     1546                if not dPids:
    15471547                    reporter.log('All done.');
    15481548                    return True;

  • trunk/src/VBox/ValidationKit/testdriver/btresolver.py

    r65378 r65967  
    9999                asMembers = utils.unpackFile(sDbgArchive, self.sScratchPath, self.fnLog,
    100100                                             self.fnLog);
    101                 if asMembers is not None and len(asMembers) > 0:
     101                if asMembers:
    102102                    # Populate the list of debug files.
    103103                    for sMember in asMembers:
    … …  
    155155        for sLine in asReport[iLine:]:
    156156            asCandidate = sLine.split();
    157             if     len(asCandidate) is 5 \
     157            if     len(asCandidate) == 5 \
    158158               and asCandidate[0].startswith('0x') \
    159159               and asCandidate[1].startswith('0x') \
    … …  
    300300            while iLine < len(asReport):
    301301                asMatches = oRegExpPath.findall(asReport[iLine]);
    302                 if len(asMatches) > 0:
     302                if asMatches:
    303303                    # Line contains the path, extract start address and path to binary
    304304                    sAddr = oRegExpAddr.findall(asReport[iLine]);
    305305                    sPath = oRegExpBinPath.findall(asReport[iLine]);
    306306
    307                     if len(sAddr) > 0 and len(sPath) > 0:
     307                    if sAddr and sPath:
    308308                        # Construct the path in into the build cache containing the debug symbols
    309309                        oRegExp = re.compile(r'\w+\.{0,1}\w*$');
    … …  
    338338            # and the first one is a number.
    339339            if     len(asStackTrace) == 6 and asStackTrace[0].isdigit() \
    340                and (asStackTrace[1].find('VBox') is not -1 or asStackTrace[1].find('VirtualBox') is not -1) \
     340               and (asStackTrace[1].find('VBox') != -1 or asStackTrace[1].find('VirtualBox') != -1) \
    341341               and asStackTrace[3].startswith('0x'):
    342342
    … …  
    404404                asMembers = utils.unpackFile(sDbgArchive, self.sScratchPath, self.fnLog,
    405405                                             self.fnLog);
    406                 if asMembers is not None and len(asMembers) > 0:
     406                if asMembers:
    407407                    # Populate the list of debug files.
    408408                    for sMember in asMembers:
    … …  
    589589            asListBinaries = self.oResolverOs.getBinaryListWithLoadAddrFromReport(sReport.split('\n'));
    590590
    591             if len(asListBinaries) > 0:
     591            if asListBinaries:
    592592                asArgs = [self.sRTLdrFltPath, ];
    593593

  • trunk/src/VBox/ValidationKit/testdriver/reporter.py

    r65857 r65967  
    325325
    326326        # Flush buffers when reaching the last test.
    327         if len(self.atTests) == 0:
     327        if not self.atTests:
    328328            self.xmlFlush(fRetry = True);
    329329
    … …  
    344344        Returns True if no open tests, False if there were open tests.
    345345        """
    346         if len(self.atTests) == 0:
     346        if not self.atTests:
    347347            return True;
    348348        for _ in range(len(self.atTests)):
    … …  
    450450        if self.oXmlFile is not None:
    451451            # pop the test stack
    452             while len(self.atTests) > 0:
     452            while self.atTests:
    453453                sName, cErrorsStart, self.fTimedOut = self.atTests.pop();
    454454                self._xmlWrite([ '<End timestamp="%s" errors="%d"/>' % (sTsIso, self.cErrors - cErrorsStart,),
    … …  
    538538                        self.log(0, 'error writing %s: %s' % (sDstFilename, oXcpt), sCaller, sTsPrf);
    539539                    else:
    540                         if len(abBuf) > 0:
     540                        if abBuf:
    541541                            continue;
    542542                break;
    … …  
    693693    def __del__(self):
    694694        """Flush pending log messages?"""
    695         if len(self._asXml) > 0:
     695        if self._asXml:
    696696            self._xmlDoFlush(self._asXml, fRetry = True, fDtor = True);
    697697
    … …  
    955955            asXml = self._asXml;
    956956            self._asXml = [];
    957             if len(asXml) > 0  or fForce is True:
     957            if asXml or fForce is True:
    958958                self._fXmlFlushing = True;
    959959
    … …  
    10301030
    10311031    def doPollWork(self, sDebug = None):
    1032         if len(self._asXml) > 0:
     1032        if self._asXml:
    10331033            g_oLock.acquire();
    10341034            try:
    … …  
    10831083                    g_oReporter.log(0, '** internal-error: Hit exception #2! %s' % (traceback.format_exc()), sCaller, sTsPrf);
    10841084
    1085                 if len(asInfo) > 0:
     1085                if asInfo:
    10861086                    # Do the logging.
    10871087                    for sItem in asInfo:
    … …  
    11931193            sText.strip();
    11941194            idxText = 0;
    1195             while len(sText) > 0:
     1195            while sText:
    11961196                if self.sTagBuffer is None:
    11971197                    # Look for the start of a tag.

  • trunk/src/VBox/ValidationKit/testdriver/txsclient.py

    r62484 r65967  
    257257                                       ord(sOpcode[6]), \
    258258                                       ord(sOpcode[7]) ) ) );
    259             if len(abPayload) > 0:
     259            if abPayload:
    260260                abMsg.extend(abPayload);
    261261        except:
    … …  
    282282
    283283        # Read the header.
    284         if len(self.abReadAheadHdr) > 0:
     284        if self.abReadAheadHdr:
    285285            assert(len(self.abReadAheadHdr) == 16);
    286286            abHdr = self.abReadAheadHdr;
    … …  
    756756                        rc = None;
    757757                        break;
    758                     if len(sInput) > 0:
     758                    if sInput:
    759759                        oStdIn.uTxsClientCrc32 = zlib.crc32(sInput, oStdIn.uTxsClientCrc32);
    760760                        # Convert to a byte array before handing it of to sendMsg or the string
    … …  
    10901090                # Update the file stream CRC and send it off.
    10911091                uMyCrc32 = zlib.crc32(abBuf, uMyCrc32);
    1092                 if len(abBuf) == 0:
     1092                if not abBuf:
    10931093                    rc = self.sendMsg('DATA EOF', (long(uMyCrc32 & 0xffffffff), ));
    10941094                else:
    … …  
    11081108
    11091109                # EOF?
    1110                 if len(abBuf) == 0:
     1110                if not abBuf:
    11111111                    break;
    11121112
    … …  
    11491149        rc = self.taskDownloadCommon(sRemoteFile, oLocalString);
    11501150        if rc is True:
    1151             if len(oLocalString.asContent) == 0:
     1151            if not oLocalString.asContent:
    11521152                rc = '';
    11531153            else:
    … …  
    19301930                self.oSocket.setblocking(0);    # just in case it's not set.
    19311931                sData = "1";
    1932                 while len(sData) > 0:
     1932                while sData:
    19331933                    sData = self.oSocket.recv(16384);
    19341934            except:
    … …  
    19461946        self.oCv.release();
    19471947
    1948     def sendBytes(self, abMsg, cMsTimeout):
     1948    def sendBytes(self, abBuf, cMsTimeout):
    19491949        if self.oSocket is None:
    19501950            reporter.error('TransportTcp.sendBytes: No connection.');
    … …  
    19531953        # Try send it all.
    19541954        try:
    1955             cbSent = self.oSocket.send(abMsg);
    1956             if cbSent == len(abMsg):
     1955            cbSent = self.oSocket.send(abBuf);
     1956            if cbSent == len(abBuf):
    19571957                return True;
    19581958        except Exception, oXcpt:
    19591959            if not self.__isWouldBlockXcpt(oXcpt):
    1960                 reporter.errorXcpt('TranportTcp.sendBytes: %s bytes' % (len(abMsg)));
     1960                reporter.errorXcpt('TranportTcp.sendBytes: %s bytes' % (len(abBuf)));
    19611961                return False;
    19621962            cbSent = 0;
    … …  
    19671967            cMsElapsed = base.timestampMilli() - msStart;
    19681968            if cMsElapsed > cMsTimeout:
    1969                 reporter.error('TranportTcp.sendBytes: %s bytes timed out (1)' % (len(abMsg)));
     1969                reporter.error('TranportTcp.sendBytes: %s bytes timed out (1)' % (len(abBuf)));
    19701970                break;
    19711971
    … …  
    19731973            try:
    19741974                ttRc = select.select([], [self.oSocket], [self.oSocket], (cMsTimeout - cMsElapsed) / 1000.0);
    1975                 if len(ttRc[2]) > 0 and len(ttRc[1]) == 0:
     1975                if ttRc[2] and not ttRc[1]:
    19761976                    reporter.error('TranportTcp.sendBytes: select returned with exception');
    19771977                    break;
    1978                 if len(ttRc[1]) == 0:
    1979                     reporter.error('TranportTcp.sendBytes: %s bytes timed out (2)' % (len(abMsg)));
     1978                if not ttRc[1]:
     1979                    reporter.error('TranportTcp.sendBytes: %s bytes timed out (2)' % (len(abBuf)));
    19801980                    break;
    19811981            except:
    … …  
    19851985            # Try send more.
    19861986            try:
    1987                 cbSent += self.oSocket.send(abMsg[cbSent:]);
    1988                 if cbSent == len(abMsg):
     1987                cbSent += self.oSocket.send(abBuf[cbSent:]);
     1988                if cbSent == len(abBuf):
    19891989                    return True;
    19901990            except Exception, oXcpt:
    19911991                if not self.__isWouldBlockXcpt(oXcpt):
    1992                     reporter.errorXcpt('TranportTcp.sendBytes: %s bytes' % (len(abMsg)));
     1992                    reporter.errorXcpt('TranportTcp.sendBytes: %s bytes' % (len(abBuf)));
    19931993                    break;
    19941994
    … …  
    20112011            try:
    20122012                abBuf = self.oSocket.recv(cb - len(self.abReadAhead));
    2013                 if len(abBuf) > 0:
     2013                if abBuf:
    20142014                    self.abReadAhead.extend(array.array('B', abBuf));
    20152015            except Exception, oXcpt:
    … …  
    20262026            cMsElapsed = base.timestampMilli() - msStart;
    20272027            if cMsElapsed > cMsTimeout:
    2028                 if not fNoDataOk or len(self.abReadAhead) > 0:
     2028                if not fNoDataOk or self.abReadAhead:
    20292029                    reporter.error('TranportTcp.recvBytes: %s/%s bytes timed out (1)' % (len(self.abReadAhead), cb));
    20302030                break;
    … …  
    20332033            try:
    20342034                ttRc = select.select([self.oSocket], [], [self.oSocket], (cMsTimeout - cMsElapsed) / 1000.0);
    2035                 if len(ttRc[2]) > 0 and len(ttRc[0]) == 0:
     2035                if ttRc[2] and not ttRc[0]:
    20362036                    reporter.error('TranportTcp.recvBytes: select returned with exception');
    20372037                    break;
    2038                 if len(ttRc[0]) == 0:
    2039                     if not fNoDataOk or len(self.abReadAhead) > 0:
     2038                if not ttRc[0]:
     2039                    if not fNoDataOk or self.abReadAhead:
    20402040                        reporter.error('TranportTcp.recvBytes: %s/%s bytes timed out (2) fNoDataOk=%s'
    20412041                                       % (len(self.abReadAhead), cb, fNoDataOk));
    … …  
    20482048            try:
    20492049                abBuf = self.oSocket.recv(cb - len(self.abReadAhead));
    2050                 if len(abBuf) == 0:
     2050                if not abBuf:
    20512051                    reporter.error('TranportTcp.recvBytes: %s/%s bytes (%s) - connection has been shut down'
    20522052                                   % (len(self.abReadAhead), cb, fNoDataOk));
    … …  
    20592059                reporter.log('recv => exception %s' % (oXcpt,));
    20602060                if not self.__isWouldBlockXcpt(oXcpt):
    2061                     if not fNoDataOk  or  not self.__isConnectionReset(oXcpt)  or  len(self.abReadAhead) > 0:
     2061                    if not fNoDataOk  or  not self.__isConnectionReset(oXcpt)  or  self.abReadAhead:
    20622062                        reporter.errorXcpt('TranportTcp.recvBytes: %s/%s bytes (%s)' % (len(self.abReadAhead), cb, fNoDataOk));
    20632063                    break;
    … …  
    20752075        try:
    20762076            ttRc = select.select([], [], [self.oSocket], 0.0);
    2077             if len(ttRc[2]) > 0:
     2077            if ttRc[2]:
    20782078                return False;
    20792079
    … …  
    20862086        try:
    20872087            ttRc = select.select([self.oSocket], [], [], cMsTimeout / 1000.0);
    2088             if len(ttRc[0]) == 0:
     2088            if not ttRc[0]:
    20892089                return False;
    20902090        except:

  • trunk/src/VBox/ValidationKit/testdriver/vbox.py

    r65513 r65967  
    24162416        ## @todo Do this elsewhere.
    24172417        # Hack alert. Disables all annoying GUI popups.
    2418         if sType == 'gui' and len(self.aoRemoteSessions) == 0:
     2418        if sType == 'gui' and not self.aoRemoteSessions:
    24192419            try:
    24202420                self.oVBox.setExtraData('GUI/Input/AutoCapture', 'false');
    … …  
    24562456        if sType == 'gui':
    24572457            sEnv += '\nVBOX_GUI_DBG_ENABLED=1'
    2458         if asEnv is not None and len(asEnv) > 0:
     2458        if asEnv is not None and asEnv:
    24592459            sEnv += '\n' + ('\n'.join(asEnv));
    24602460
    … …  
    26472647                    for iCpu in xrange(0, cCpus):
    26482648                        sThis = oSession.queryDbgGuestStack(iCpu);
    2649                         if sThis is not None and len(sThis) > 0:
     2649                        if sThis:
    26502650                            asMiscInfos += [
    26512651                                '================ start guest stack VCPU %s ================\n' % (iCpu,),
    … …  
    26712671                        continue;
    26722672                    sThis = oSession.queryDbgInfo(sInfo, sArg);
    2673                     if sThis is not None and len(sThis) > 0:
     2673                    if sThis:
    26742674                        if sThis[-1] != '\n':
    26752675                            sThis += '\n';
    … …  
    27672767
    27682768        # Add the "info xxxx" items if we've got any.
    2769         if len(asMiscInfos) > 0:
     2769        if asMiscInfos:
    27702770            reporter.addLogString(u''.join(asMiscInfos), 'info.txt', 'info/collection', 'A bunch of info items.');
    27712771

  • trunk/src/VBox/ValidationKit/testdriver/vboxinstaller.py

    r65313 r65967  
    102102            # End of our parameters and start of the sub driver invocation.
    103103            iArg = self.requireMoreArgs(1, asArgs, iArg);
    104             assert len(self._asSubDriver) == 0;
     104            assert not self._asSubDriver;
    105105            self._asSubDriver = asArgs[iArg:];
    106106            self._asSubDriver[0] = self._asSubDriver[0].replace('/', os.path.sep);
    … …  
    122122        # Check that we've got what we need.
    123123        #
    124         if len(self._asBuildUrls) == 0:
     124        if not self._asBuildUrls:
    125125            reporter.error('No build files specfiied ("--vbox-build file1[,file2[...]]")');
    126126            return False;
    127         if len(self._asSubDriver) == 0:
     127        if not self._asSubDriver:
    128128            reporter.error('No sub testdriver specified. (" -- test/stuff/tdStuff1.py args")');
    129129            return False;
    … …  
    228228        try:
    229229            oFile = open(sFull, 'w');
    230             if len(sValue) > 0:
     230            if sValue:
    231231                oFile.write(sValue.encode('utf-8'));
    232232            oFile.close();
    … …  
    314314                if iIteration in [0, 21]  and  sBase in [ 'windbg', 'gdb', 'gdb-i386-apple-darwin', ]:
    315315                    reporter.log('Warning: debugger running: %s (%s)' % (oProcess.iPid, sBase,));
    316             if len(aoTodo) == 0:
     316            if not aoTodo:
    317317                return True;
    318318

  • trunk/src/VBox/ValidationKit/testdriver/vboxtestvms.py

    r65835 r65967  
    626626                    raise base.InvalidOption('The "--paravirt-modes" value "%s" is not valid; valid values are: %s'
    627627                                             % (sPvMode, ', '.join(g_kasParavirtProviders),));
    628             if len(self.asParavirtModes) == 0:
     628            if not self.asParavirtModes:
    629629                self.asParavirtModes = None;
    630630
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette