Changeset f806b61 for tests/test.py

Timestamp:

Mar 28, 2019, 5:19:52 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:

b611fc3

Parents:

ea62265

Message:

Tests are now run in temporary directory

File:

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

tests/test.py

@@ -8 +8 @@
 import re
 import sys
+import tempfile
 import time
 
@@ -143 +144 @@
                 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
         run_dur = None
-        if success(make_ret):
-                with Timed() as run_dur:
-                        if settings.dry_run or is_exe(exe_file):
-                                # run test
-                                retcode, _ = sh(exe_file, output=out_file, input=in_file, timeout=True)
+        # run everything in a temp directory to make sure core file are handled properly
+        with tempdir():
+                # if the make command succeds continue otherwise skip to diff
+                if success(make_ret):
+                        with Timed() as run_dur:
+                                if settings.dry_run or is_exe(exe_file):
+                                        # run test
+                                        retcode, _ = sh(exe_file, output=out_file, input=in_file, timeout=True)
+                                else :
+                                        # simply cat the result into the output
+                                        retcode = cat(exe_file, out_file)
+                else:
+                        retcode = mv(err_file, out_file)
+
+                if success(retcode):
+                        if settings.generating :
+                                # if we are ounly generating the output we still need to check that the test actually exists
+                                if no_rule(out_file, test.target()) :
+                                        retcode = 1
+                                        error = "\t\tNo make target for test %s!" % test.target()
+                                        rm(out_file)
+                                else:
+                                        error = None
                         else :
-                                # simply cat the result into the output
-                                retcode = cat(exe_file, out_file)
-        else:
-                retcode = mv(err_file, out_file)
-
-        if success(retcode):
-                if settings.generating :
-                        # if we are ounly generating the output we still need to check that the test actually exists
-                        if no_rule(out_file, test.target()) :
-                                retcode = 1
-                                error = "\t\tNo make target for test %s!" % test.target()
-                                rm(out_file)
-                        else:
-                                error = None
-                else :
-                        # fetch return code and error from the diff command
-                        retcode, error = diff(cmp_file, out_file)
-
-        else:
-                with open (out_file, "r") as myfile:
-                        error = myfile.read()
-
-                ret, info = core_info(exe_file)
-                error = error + info if error else info
+                                # fetch return code and error from the diff command
+                                retcode, error = diff(cmp_file, out_file)
+
+                else:
+                        with open (out_file, "r") as myfile:
+                                error = myfile.read()
+
+                        ret, info = core_info(exe_file)
+                        error = error + info if error else info