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:36:59 UTC

[allura] branch pytest-finalize updated (bdd7df7a5 -> 3d68b00fc)

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

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


    from bdd7df7a5 [#8455] misc fixes to tests from pytest conversion
     new b346a06ff [#8455] change pytest from dev dep to full dep, add pytest-sugar dep
     new d16f8ac68 fixup! [#8455] update test runner to use pytest and pytest-xdist for parallelization
     new 3d68b00fc fixup! [#8455] added pytest.ini

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 pytest.ini           | 2 ++
 requirements-dev.txt | 2 --
 requirements.in      | 1 +
 requirements.txt     | 9 ++++++++-
 run_tests            | 6 +++---
 5 files changed, 14 insertions(+), 6 deletions(-)


[allura] 03/03: fixup! [#8455] added pytest.ini

Posted by di...@apache.org.
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 3d68b00fc7399532efe8b6c4bc3285fedc93c639
Author: Dillon Walls <di...@slashdotmedia.com>
AuthorDate: Mon Nov 7 20:36:41 2022 +0000

    fixup! [#8455] added pytest.ini
---
 pytest.ini | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/pytest.ini b/pytest.ini
index 3d38df179..92478cd93 100644
--- a/pytest.ini
+++ b/pytest.ini
@@ -5,6 +5,8 @@
 filterwarnings =
     ignore::DeprecationWarning
 
+addopts = --pyargs -p no:flaky
+
 # our patterns are listed first:  then ".*" and following are defaults from https://github.com/pytest-dev/pytest/blob/main/src/_pytest/main.py#L52
 norecursedirs = templates_responsive resources images js data docs public *.egg-info __pycache__ .* *.egg _darcs build CVS dist node_modules venv {arch}
 


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

Posted by di...@apache.org.
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 ''
 


[allura] 01/03: [#8455] change pytest from dev dep to full dep, add pytest-sugar dep

Posted by di...@apache.org.
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 b346a06ffc044b43c5cc6d4814adc92c2392ec23
Author: Dillon Walls <di...@slashdotmedia.com>
AuthorDate: Mon Nov 7 20:35:44 2022 +0000

    [#8455] change pytest from dev dep to full dep, add pytest-sugar dep
---
 requirements-dev.txt | 2 --
 requirements.in      | 1 +
 requirements.txt     | 9 ++++++++-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/requirements-dev.txt b/requirements-dev.txt
index 71ace7eda..ec1568631 100644
--- a/requirements-dev.txt
+++ b/requirements-dev.txt
@@ -7,7 +7,5 @@ sphinx-argparse
 sphinx-rtd-theme
 sphinxcontrib-programoutput
 coverage
-pytest
-pytest-xdist
 pycodestyle
 pyflakes
diff --git a/requirements.in b/requirements.in
index 2fbf36b84..50e6726e7 100644
--- a/requirements.in
+++ b/requirements.in
@@ -56,6 +56,7 @@ testfixtures
 WebTest
 pytest
 pytest-xdist
+pytest-sugar
 
 # deployment
 gunicorn
diff --git a/requirements.txt b/requirements.txt
index 4d7392610..0d4820e57 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -102,7 +102,9 @@ oauthlib==3.2.2
     #   -r requirements.in
     #   requests-oauthlib
 packaging==21.3
-    # via pytest
+    # via
+    #   pytest
+    #   pytest-sugar
 paginate==0.5.6
     # via -r requirements.in
 paste==3.5.2
@@ -144,7 +146,10 @@ pysolr==3.9.0
 pytest==7.1.3
     # via
     #   -r requirements.in
+    #   pytest-sugar
     #   pytest-xdist
+pytest-sugar==0.9.5
+    # via -r requirements.in
 pytest-xdist==3.0.2
     # via -r requirements.in
 python-dateutil==2.8.2
@@ -198,6 +203,8 @@ smmap==5.0.0
     # via gitdb
 soupsieve==2.3.2.post1
     # via beautifulsoup4
+termcolor==2.1.0
+    # via pytest-sugar
 testfixtures==7.0.2
     # via -r requirements.in
 textile==4.0.2