VirtualBox

Ignore:
Timestamp:
Nov 28, 2022 11:53:56 AM (2 years ago)
Author:
vboxsync
Message:

Validation Kit/Guest Control: Made setting up the Guest Control testing environment a bit more resilient against VBoxSVC / client crashes and warn about it.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/ValidationKit/tests/additions/tdAddGuestCtrl.py

    r97688 r97690  
    111111        Applies credential defaults, based on the test VM (guest OS), if
    112112        no credentials were set yet.
     113
     114        Returns success status.
    113115        """
    114116        self.oTestVm = oTestVm;
    115         assert self.oTestVm is not None;
     117        if not self.oTestVm:
     118            reporter.log('VM object is invalid -- did VBoxSVC or a client crash?');
     119            return False;
    116120
    117121        if self.sUser is None:
    … …  
    123127        if self.sDomain is None:
    124128            self.sDomain   = '';
     129
     130        return True;
    125131
    126132class tdTestGuestCtrlBase(object):
    … …  
    141147        """
    142148        Sets the test environment required for this test.
     149
     150        Returns success status.
    143151        """
    144152        _ = oTxsSession;
    145153
     154        fRc = True;
    146155        try:
    147156            self.oGuest  = oSession.o.console.guest;
    148157            self.oTestVm = oTestVm;
    149158        except:
    150             reporter.errorXcpt();
     159            fRc = reporter.errorXcpt();
    151160
    152161        if self.oCreds is None:
    153162            self.oCreds = tdCtxCreds();
    154         self.oCreds.applyDefaultsIfNotSet(self.oTestVm);
    155 
    156         return True;
     163
     164        fRc = fRc and self.oCreds.applyDefaultsIfNotSet(self.oTestVm);
     165
     166        if not fRc:
     167            reporter.log('Error setting up Guest Control testing environment!');
     168
     169        return fRc;
    157170
    158171    def uploadLogData(self, oTstDrv, aData, sFileName, sDesc):
    … …  
    888901        assert self.enmUser is None; # For later.
    889902        self.oCreds = tdCtxCreds();
    890         self.setEnvironment(oVmSession, oTxsSession, oTestVm);
     903        fRc = self.setEnvironment(oVmSession, oTxsSession, oTestVm);
     904        if not fRc:
     905            return False;
    891906        reporter.log2('%s: %s steps' % (sMsgPrefix, len(self.aoSteps),));
    892907        fRc, oCurSession = self.createSession(sMsgPrefix);
    … …  
    25432558            oCurRes  = tTest[1] # type: tdTestResult
    25442559
    2545             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     2560            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     2561            if not fRc:
     2562                break;
    25462563            reporter.log('Testing #%d, user="%s", sPassword="%s", sDomain="%s" ...'
    25472564                         % (i, oCurTest.oCreds.sUser, oCurTest.oCreds.sPassword, oCurTest.oCreds.sDomain));
    … …  
    25942611        for i in xrange(cMaxGuestSessions + 1):
    25952612            aoMultiSessions[i] = tdTestSession(sSessionName = 'MultiSession #%d' % (i,));
    2596             aoMultiSessions[i].setEnvironment(oSession, oTxsSession, oTestVm);
     2613            fRc = aoMultiSessions[i].setEnvironment(oSession, oTxsSession, oTestVm);
     2614            if not fRc:
     2615                break;
    25972616
    25982617            cCurSessions = aoMultiSessions[i].getSessionCount(self.oTstDrv.oVBoxMgr);
    … …  
    31813200            oCurTest = tTest[0]  # type: tdTestExec
    31823201            oCurRes  = tTest[1]  # type: tdTestResultExec
    3183             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3202            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3203            if not fRc:
     3204                break;
    31843205            fRc2, oCurGuestSession = oCurTest.createSession('testGuestCtrlExec: Test #%d' % (i,));
    31853206            if fRc2 is not True:
    … …  
    32383259                    oCurRes  = tTest[1] # type: tdTestResultExec
    32393260
    3240                     oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3261                    fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3262                    if not fRc:
     3263                        break;
    32413264                    fRc = self.gctrlExecDoTest(i, oCurTest, oCurRes, oCurGuestSession);
    32423265                    if fRc is False:
    … …  
    36033626            reporter.log('Testing #%d, sDirectory="%s" ...' % (i, limitString(oCurTest.sDirectory)));
    36043627
    3605             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3628            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3629            if not fRc:
     3630                break;
    36063631            fRc, oCurGuestSession = oCurTest.createSession('testGuestCtrlDirCreate: Test #%d' % (i,));
    36073632            if fRc is False:
    … …  
    37013726                         (i, oCurTest.sTemplate, oCurTest.fMode, oCurTest.sDirectory, oCurTest.fSecure));
    37023727
    3703             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3728            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3729            if not fRc:
     3730                break;
    37043731            fRc, oCurGuestSession = oCurTest.createSession('testGuestCtrlDirCreateTemp: Test #%d' % (i,));
    37053732            if fRc is False:
    … …  
    37943821
    37953822            reporter.log('Testing #%d, dir="%s" ...' % (i, oCurTest.sDirectory));
    3796             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3823            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3824            if not fRc:
     3825                break;
    37973826            fRc, oCurGuestSession = oCurTest.createSession('testGuestCtrlDirRead: Test #%d' % (i,));
    37983827            if fRc is not True:
    … …  
    38363865        if fRc is True:
    38373866            oCurTest = tdTestDirRead();
    3838             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
    3839             fRc, oCurGuestSession = oCurTest.createSession('testGuestCtrlDirRead: gctrlReadDirTree2');
     3867            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3868            if fRc:
     3869                fRc, oCurGuestSession = oCurTest.createSession('testGuestCtrlDirRead: gctrlReadDirTree2');
    38403870            if fRc is True:
    38413871                for oDir in (self.oTestFiles.oEmptyDir, self.oTestFiles.oManyDir, self.oTestFiles.oTreeDir):
    … …  
    39293959        for (i, oTest) in enumerate(aoTests): # int, tdTestRemoveBase
    39303960            reporter.log('Testing #%d, path="%s" %s ...' % (i, oTest.sPath, oTest.__class__.__name__));
    3931             oTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3961            fRc = oTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3962            if not fRc:
     3963                break;
    39323964            fRc, _ = oTest.createSession('testGuestCtrlFileRemove: Test #%d' % (i,));
    39333965            if fRc is False:
    … …  
    39393971        if fRc is True:
    39403972            oCurTest = tdTestDirRead();
    3941             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
    3942             fRc, oCurGuestSession = oCurTest.createSession('remove final');
     3973            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     3974            if fRc:
     3975                fRc, oCurGuestSession = oCurTest.createSession('remove final');
    39433976            if fRc is True:
    39443977
    … …  
    40544087
    40554088        oTest = tdTestGuestCtrlBase();
    4056         oTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4089        fRc = oTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4090        if not fRc:
     4091            return (False, oTxsSession);
    40574092        fRc2, oGuestSession = oTest.createSession('FsStat on TestFileSet');
    40584093        if fRc2 is not True:
    … …  
    42644299
    42654300            oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4301            if not fRc:
     4302                break;
    42664303            fRc, _ = oCurTest.createSession('testGuestCtrlFileOpen: Test #%d' % (i,));
    42674304            if fRc is not True:
    … …  
    42904327        #
    42914328        oTest = tdTestGuestCtrlBase();
    4292         oTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4329        fRc = oTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4330        if not fRc:
     4331            return (False, oTxsSession);
    42934332        fRc2, oGuestSession = oTest.createSession('FsStat on TestFileSet');
    42944333        if fRc2 is not True:
    … …  
    47064745        for (i, oCurTest) in enumerate(aoTests):
    47074746            reporter.log('Testing #%d: %s ...' % (i, oCurTest.toString(),));
    4708 
    4709             oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4747            fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     4748            if not fRc:
     4749                break;
    47104750            fRc, _ = oCurTest.createSession('testGuestCtrlFileWrite: Test #%d' % (i,));
    47114751            if fRc is not True:
    … …  
    49635003
    49645004            oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     5005            if not fRc:
     5006                break;
    49655007            fRc, oCurGuestSession = oCurTest.createSession('testGuestCtrlCopyTo: Test #%d' % (i,));
    49665008            if fRc is not True:
    … …  
    51925234                fRc = oCurTest.execute(self.oTstDrv, oSession, oTxsSession, oTestVm, 'testing #%d' % (i,));
    51935235            else:
    5194                 oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     5236                fRc = oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     5237                if not fRc:
     5238                    break;
    51955239                fRc2, oCurGuestSession = oCurTest.createSession('testGuestCtrlCopyFrom: Test #%d' % (i,));
    51965240                if fRc2 is not True:
    … …  
    52835327
    52845328            oCurTest.setEnvironment(oSession, oTxsSession, oTestVm);
     5329            if not fRc:
     5330                break;
    52855331            fRc, _ = oCurTest.createSession('Test #%d' % (i,));
    52865332            if fRc is not True:
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