Go back to single core for test suite on CI (#2305)

The random asyncio bug is just too frequent and annoying to be
worth the speed improvements. Our test suite is already quite fast.
Random test failures hurt for 3 reasons, 1) they are discouraging for
new contributors who won't understand it's out of their control, 2)
it's annoying and time consuming to rerun the workflow, and 3) it
makes single job failures feel less important (even they should be
treated as important!).
This commit is contained in:
Richard Si 2021-06-03 21:26:21 -04:00 committed by GitHub
parent a2b5ba2a3a
commit c53b3ad8fa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 4 deletions

View File

@ -42,7 +42,7 @@ jobs:
- name: Unit tests - name: Unit tests
run: | run: |
tox -e py -- -v --color=yes tox -e ci-py -- -v --color=yes
- name: Publish coverage to Coveralls - name: Publish coverage to Coveralls
# If pushed / is a pull request against main repo AND # If pushed / is a pull request against main repo AND

12
tox.ini
View File

@ -1,17 +1,23 @@
[tox] [tox]
envlist = py{36,37,38,39},fuzz envlist = {,ci-}py{36,37,38,39},fuzz
[testenv] [testenv]
setenv = PYTHONPATH = {toxinidir}/src setenv = PYTHONPATH = {toxinidir}/src
skip_install = True skip_install = True
deps = deps =
-r{toxinidir}/test_requirements.txt -r{toxinidir}/test_requirements.txt
; parallelization is disabled on CI because pytest-dev/pytest-xdist#620 occurs too frequently
; local runs can stay parallelized since they aren't rolling the dice so many times as like on CI
commands = commands =
pip install -e .[d] pip install -e .[d]
coverage erase coverage erase
pytest tests --run-optional no_python2 --numprocesses auto --cov {posargs} pytest tests --run-optional no_python2 \
!ci: --numprocesses auto \
--cov {posargs}
pip install -e .[d,python2] pip install -e .[d,python2]
pytest tests --run-optional python2 --numprocesses auto --cov --cov-append {posargs} pytest tests --run-optional python2 \
!ci: --numprocesses auto \
--cov --cov-append {posargs}
coverage report coverage report
[testenv:fuzz] [testenv:fuzz]