Changeset a45fc7b for tests/test.py


Ignore:
Timestamp:
Mar 27, 2019, 2:44:34 PM (3 years ago)
Author:
tdelisle <tdelisle@…>
Branches:
arm-eh, cleanup-dtors, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr
Children:
1241851, ce3d305
Parents:
86fb8f2
Message:

Tests no longer use subprocess.run(shell=True), which solves leak process problems

File:
1 edited

Legend:

Unmodified
Added
Removed
  • tests/test.py

    r86fb8f2 ra45fc7b  
    141141        # build, skipping to next test on error
    142142        with Timed() as comp_dur:
    143                 make_ret, _ = make( test.target(),      redirects  = ("2> %s 1> /dev/null" % out_file), error_file = err_file )
     143                make_ret, _ = make( test.target(), output=subprocess.DEVNULL, error=out_file, error_file = err_file )
    144144
    145145        # if the make command succeds continue otherwise skip to diff
     
    195195
    196196                #print result with error if needed
    197                 text = name_txt + result_txt
     197                text = '\t' + name_txt + result_txt
    198198                out = sys.stdout
    199199                if error :
     
    212212def run_tests(tests, jobs) :
    213213        # clean the sandbox from previous commands
    214         make('clean', redirects = '> /dev/null 2>&1')
     214        make('clean', output=subprocess.DEVNULL, error=subprocess.DEVNULL)
    215215
    216216        # create the executor for our jobs and handle the signal properly
     
    230230
    231231        # clean the workspace
    232         make('clean', redirects = '> /dev/null 2>&1')
     232        make('clean', output=subprocess.DEVNULL, error=subprocess.DEVNULL)
    233233
    234234        for failed in results:
Note: See TracChangeset for help on using the changeset viewer.