Changeset a45fc7b for tests/test.py

Timestamp:

Mar 27, 2019, 2:44:34 PM (5 years ago)

Author:

tdelisle <tdelisle@…>

Branches:

ADT, arm-eh, ast-experimental, cleanup-dtors, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

1241851, ce3d305

Parents:

86fb8f2

Message:

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

File:

• tests/test.py (modified) (4 diffs)

tests/test.py

@@ -141 +141 @@
         # build, skipping to next test on error
         with Timed() as comp_dur:
-                make_ret, _ = make( test.target(),      redirects  = ("2> %s 1> /dev/null" % out_file), error_file = err_file )
+                make_ret, _ = make( test.target(), output=subprocess.DEVNULL, error=out_file, error_file = err_file )
 
         # if the make command succeds continue otherwise skip to diff
@@ -195 +195 @@
 
                 #print result with error if needed
-                text = name_txt + result_txt
+                text = '\t' + name_txt + result_txt
                 out = sys.stdout
                 if error :
@@ -212 +212 @@
 def run_tests(tests, jobs) :
         # clean the sandbox from previous commands
-        make('clean', redirects = '> /dev/null 2>&1')
+        make('clean', output=subprocess.DEVNULL, error=subprocess.DEVNULL)
 
         # create the executor for our jobs and handle the signal properly
@@ -230 +230 @@
 
         # clean the workspace
-        make('clean', redirects = '> /dev/null 2>&1')
+        make('clean', output=subprocess.DEVNULL, error=subprocess.DEVNULL)
 
         for failed in results: