Changeset cc9b520 for tests/pybin

Timestamp:

Mar 9, 2022, 7:59:46 PM (3 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, ast-experimental, enum, master, pthread-emulation, qualifiedEnum

Children:

b053083

Parents:

5baa33c

Message:

Clean-up error handling in test scripts

Location:

tests/pybin

Files:

• settings.py (modified) (1 diff)
• tools.py (modified) (7 diffs)

tests/pybin/settings.py

@@ -192 +192 @@
         global distcc
         distcc       = "DISTCC_CFA_PATH=~/.cfadistcc/%s/cfa" % tools.config_hash()
-        errf = os.path.join(BUILDDIR, ".validate.err")
-        make_ret, out = tools.make( ".validate", error_file = errf, output_file=subprocess.DEVNULL, error=subprocess.DEVNULL )
+        make_ret, out, err = tools.make( ".validate", output_file=subprocess.PIPE, error=subprocess.PIPE )
         if make_ret != 0:
-                with open (errf, "r") as myfile:
-                        error=myfile.read()
                 print("ERROR: Invalid configuration %s:%s" % (arch.string, debug.string), file=sys.stderr)
-                print("       verify returned : \n%s" % error, file=sys.stderr)
-                tools.rm(errf)
+                print("       verify returned : \n%s" % err, file=sys.stderr)
                 sys.exit(1)
-
-        tools.rm(errf)
 
 def prep_output(tests):

tests/pybin/tools.py

@@ -70 +70 @@
 
                                 try:
-                                        out, _ = proc.communicate(
+                                        out, errout = proc.communicate(
                                                 timeout = settings.timeout.single if timeout else None
                                         )
 
-                                        return proc.returncode, out.decode("latin-1") if out else None
+                                        return proc.returncode, out.decode("latin-1") if out else None, errout.decode("latin-1") if errout else None
                                 except subprocess.TimeoutExpired:
                                         if settings.timeout2gdb:
                                                 print("Process {} timeout".format(proc.pid))
                                                 proc.communicate()
-                                                return 124, str(None)
+                                                return 124, str(None), "Subprocess Timeout 2 gdb"
                                         else:
                                                 proc.send_signal(signal.SIGABRT)
                                                 proc.communicate()
-                                                return 124, str(None)
+                                                return 124, str(None), "Subprocess Timeout 2 gdb"
 
         except Exception as ex:
@@ -106 +106 @@
                 return (False, "No file")
 
-        code, out = sh("file", fname, output_file=subprocess.PIPE)
+        code, out, err = sh("file", fname, output_file=subprocess.PIPE)
         if code != 0:
-                return (False, "'file EXPECT' failed with code {}".format(code))
+                return (False, "'file EXPECT' failed with code {} '{}'".format(code, err))
 
         match = re.search(".*: (.*)", out)
@@ -242 +242 @@
 # move a file
 def mv(source, dest):
-        ret, _ = sh("mv", source, dest)
+        ret, _, _ = sh("mv", source, dest)
         return ret
 
 # cat one file into the other
 def cat(source, dest):
-        ret, _ = sh("cat", source, output_file=dest)
+        ret, _, _ = sh("cat", source, output_file=dest)
         return ret
 
@@ -291 +291 @@
 ################################################################################
 def jobserver_version():
-        make_ret, out = sh('make', '.test_makeflags', '-j2', output_file=subprocess.PIPE)
+        make_ret, out, err = sh('make', '.test_makeflags', '-j2', output_file=subprocess.PIPE, error=subprocess.PIPE)
         if make_ret != 0:
-                with open (errf, "r") as myfile:
-                        error=myfile.read()
                 print("ERROR: cannot find Makefile jobserver version", file=sys.stderr)
-                print("       test returned : \n%s" % out, file=sys.stderr)
+                print("       test returned : {} '{}'".format(make_ret, err), file=sys.stderr)
                 sys.exit(1)
 
@@ -302 +300 @@
         if not re_jobs:
                 print("ERROR: cannot find Makefile jobserver version", file=sys.stderr)
-                print("       MAKEFLAGS are : \n%s" % out, file=sys.stderr)
+                print("       MAKEFLAGS are : '{}'".format(out), file=sys.stderr)
                 sys.exit(1)
 
@@ -344 +342 @@
                 # remote hardware is allowed
                 # how much do we have?
-                ret, jstr = sh("distcc", "-j", output_file=subprocess.PIPE, ignore_dry_run=True)
+                ret, jstr, _ = sh("distcc", "-j", output_file=subprocess.PIPE, ignore_dry_run=True)
                 return int(jstr.strip()) if ret == 0 else multiprocessing.cpu_count()
         else:
@@ -445 +443 @@
         distcc_hash = os.path.join(settings.SRCDIR, '../tools/build/distcc_hash')
         config = "%s-%s" % (settings.arch.target, settings.debug.path)
-        _, out = sh(distcc_hash, config, output_file=subprocess.PIPE, ignore_dry_run=True)
+        _, out, _ = sh(distcc_hash, config, output_file=subprocess.PIPE, ignore_dry_run=True)
         return out.strip()