You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by di...@apache.org on 2022/11/07 20:37:01 UTC

[allura] 02/03: fixup! [#8455] update test runner to use pytest and pytest-xdist for parallelization

This is an automated email from the ASF dual-hosted git repository.

dill0wn pushed a commit to branch pytest-finalize
in repository https://gitbox.apache.org/repos/asf/allura.git

commit d16f8ac68f9b75a6babde4e925f4e6d60719ee82
Author: Dillon Walls <di...@slashdotmedia.com>
AuthorDate: Mon Nov 7 20:36:14 2022 +0000

    fixup! [#8455] update test runner to use pytest and pytest-xdist for parallelization
---
 run_tests | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/run_tests b/run_tests
index 98c39f88c..813314b20 100755
--- a/run_tests
+++ b/run_tests
@@ -67,7 +67,7 @@ def run_one(cmd, **popen_kwargs):
     out_remainder, _ = proc.communicate()
     sys.stdout.write(print_ensured(out_remainder))
     sys.stdout.flush()
-    print('finished {}'.format(cmd_to_show))
+    print('finished {}, with returncode: {}'.format(cmd_to_show, proc.returncode))
     sys.stdout.flush()
     return proc
 
@@ -118,7 +118,7 @@ def check_packages(packages):
 
 def run_tests_in_parallel(options, runner_args):
     default_args = [
-        '-c /dev/null',  # pytest's equivalent of nose's NOSE_IGNORE_CONFIG_FILES='1' is '-c /dev/null/'
+        # '-c /dev/null',  # pytest's equivalent of nose's NOSE_IGNORE_CONFIG_FILES='1' is '-c /dev/null/'
         '--disable-warnings',
     ]
 
@@ -128,7 +128,7 @@ def run_tests_in_parallel(options, runner_args):
     def get_multiproc_args(pkg):
         if options.concurrent_tests == 1:
             return ''
-        return '-n {procs_per_suite}'.format(
+        return '-n {procs_per_suite} --dist loadfile'.format(
             procs_per_suite=options.concurrent_tests
         ) if pkg not in NOT_MULTIPROC_SAFE else ''