Skip to content

Commit

Permalink
Travis and tox config improvements (home-assistant#28667)
Browse files Browse the repository at this point in the history
* Use travis_wait only with pylint

pylint is the only job that is expected to be silent for extended time.
For others such a silence is a sign of a problem and using travis_wait
just lengthens the wait, and makes things harder to follow and debug,
because it also suppresses output in the web UI.

* Use pytest-xdist in tox

Similarly as in Azure.
  • Loading branch information
scop authored and fabaff committed Nov 13, 2019
1 parent b4cec23 commit a48d426
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ matrix:
- python: "3.6.1"
env: TOXENV=lint
- python: "3.6.1"
env: TOXENV=pylint PYLINT_ARGS=--jobs=0
env: TOXENV=pylint PYLINT_ARGS=--jobs=0 TRAVIS_WAIT=30
- python: "3.6.1"
env: TOXENV=typing
- python: "3.6.1"
Expand All @@ -33,4 +33,4 @@ cache:
- $HOME/.cache/pre-commit
install: pip install -U tox
language: python
script: travis_wait 50 tox --develop
script: ${TRAVIS_WAIT:+travis_wait $TRAVIS_WAIT} tox --develop
6 changes: 4 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -5,19 +5,21 @@ skip_missing_interpreters = True
[testenv]
basepython = {env:PYTHON3_PATH:python3}
commands =
pytest --timeout=9 --durations=10 -qq -o console_output_style=count -p no:sugar {posargs}
pytest --timeout=9 --durations=10 -n auto --dist=loadfile -qq -o console_output_style=count -p no:sugar {posargs}
{toxinidir}/script/check_dirty
deps =
-r{toxinidir}/requirements_test_all.txt
-c{toxinidir}/homeassistant/package_constraints.txt
pytest-xdist

[testenv:cov]
commands =
pytest --timeout=9 --durations=10 -qq -o console_output_style=count -p no:sugar --cov --cov-report= {posargs}
pytest --timeout=9 --durations=10 -n auto --dist=loadfile -qq -o console_output_style=count -p no:sugar --cov --cov-report= {posargs}
{toxinidir}/script/check_dirty
deps =
-r{toxinidir}/requirements_test_all.txt
-c{toxinidir}/homeassistant/package_constraints.txt
pytest-xdist

[testenv:pylint]
ignore_errors = True
Expand Down

0 comments on commit a48d426

Please sign in to comment.