Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: upgrade pytest-cov #651

Closed
wants to merge 21 commits into from
Closed

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Sep 16, 2018

This causes coverage dropping on codecov at least.

Findings:

  • using just --cov (2nd commit) is very slow (10min / job), but restores more coverage already, unlike --cov=$PWD?!

This might speed up tests a bit.
The config appears to be missing in subprocesses?!
(https://travis-ci.org/pytest-dev/pytest-django/builds/429186955#L750)

    cwd is now '/tmp/pytest-of-travis/pytest-0/test_django_db_reset_sequences_fixture0'
     1963.5ece: New process: executable: /home/travis/build/pytest-dev/pytest-django/.tox/py36-dj20-postgres/bin/python
     1963.5ece: New process: cmd: /home/travis/build/pytest-dev/pytest-django/.tox/py36-dj20-postgres/lib/python3.6/site-packages/pytest.py --basetemp=/tmp/pytest-of-travis/pytest-0/test_django_db_reset_sequences_fixture0/runpytest-0 -v --reuse-db
     1963.5ece: New process: parent pid: 1951
     1963.5ece: -- config ----------------------------------------------------
     1963.5ece:                _include: None
     1963.5ece:                   _omit: None
     1963.5ece:  attempted_config_files: .coveragerc
     1963.5ece:                          setup.cfg
     1963.5ece:                          tox.ini
     1963.5ece:                  branch: False
     1963.5ece:             concurrency: None
     1963.5ece:            config_files: tox.ini
     1963.5ece:             cover_pylib: False
     1963.5ece:               data_file: /tmp/pytest-of-travis/pytest-0/test_django_db_reset_sequences_fixture0/.coverage

But not for the python process before that (also in this tmpdir
already)!
@blueyed
Copy link
Contributor Author

blueyed commented Sep 16, 2018

=> #656

@blueyed
Copy link
Contributor Author

blueyed commented Sep 17, 2018

Closing in favor of #657.

@blueyed blueyed closed this Sep 17, 2018
@blueyed blueyed deleted the pytest-cov branch September 17, 2018 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant