Changeset f58522b0

Timestamp:

Sep 29, 2022, 3:16:51 PM (22 months ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, ast-experimental, master

Children:

2e94d94f

Parents:

77125cc

Message:

python test script now nice +5 the executables before running them.

Location:

tests

Files:

• pybin/tools.py (modified) (2 diffs)
• test.py (modified) (1 diff)

tests/pybin/tools.py

@@ -23 +23 @@
 
 # helper functions to run terminal commands
-def sh(*cmd, timeout = False, output_file = None, input_file = None, input_text = None, error = subprocess.STDOUT, ignore_dry_run = False, pass_fds = []):
+def sh(*cmd, timeout = False, output_file = None, input_file = None, input_text = None, error = subprocess.STDOUT, ignore_dry_run = False, pass_fds = [], nice = False):
         try:
                 cmd = list(cmd)
@@ -58 +58 @@
                         error = openfd(error, 'w', onexit, False)
 
+                        # prepare the parameters to the call
+                        popen_kwargs = {
+                                'stdout' : output_file,
+                                'stderr' : error,
+                                'pass_fds': pass_fds,
+                        }
+
+                        # depending on how we are passing inputs we need to set a different argument to popen
+                        if input_text:
+                                popen_kwargs['input'] = bytes(input_text, encoding='utf-8')
+                        else:
+                                popen_kwargs['stdin'] = input_file
+
+                        # we might want to nice this so it's not to obnixious to users
+                        if nice:
+                                popen_kwargs['preexec_fn'] = lambda: os.nice(5)
+
                         # run the desired command
                         # use with statement to make sure proc is cleaned
                         # don't use subprocess.run because we want to send SIGABRT on exit
-                        with subprocess.Popen(
-                                cmd,
-                                **({'input' : bytes(input_text, encoding='utf-8')} if input_text else {'stdin' : input_file}),
-                                stdout  = output_file,
-                                stderr  = error,
-                                pass_fds = pass_fds
-                        ) as proc:
-
+                        with subprocess.Popen( cmd, **popen_kwargs ) as proc:
                                 try:
                                         out, errout = proc.communicate(

tests/test.py

@@ -190 +190 @@
                                 if settings.dry_run or is_exe(exe_file):
                                         # run test
-                                        retcode, _, _ = sh(exe_file, output_file=out_file, input_file=in_file, timeout=True)
+                                        retcode, _, _ = sh(exe_file, output_file=out_file, input_file=in_file, timeout=True, nice=True)
                                 else :
                                         # simply cat the result into the output