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

Fix handling when tests change working directory. #95

Merged
merged 1 commit into from
Oct 2, 2015

Conversation

ionelmc
Copy link
Member

@ionelmc ionelmc commented Oct 1, 2015

Use absolute paths for coverage config file and sources. Add a new internal variable "COV_CORE_DATAFILE" to workaround the misplacement of .coverage.foobar.123 files.

Because needs to be an absolute path COV_CORE_CONFIG we need to do some special things: coverage treats config_file='.coveragerc' as if it was config_file=True. We need to keep that behavior.

Closes #94. Closes #77.

Use absolute paths for coverage config file and sources. Add a new internal variable "COV_CORE_DATAFILE" to workaround the misplacement of .coverage.foobar.123 files.

Because needs to be an absolute path COV_CORE_CONFIG we need to do some special things: coverage treats config_file='.coveragerc' as if it was config_file=True. We need to keep that behavior.

Closes #94. Closes #77.
@embray
Copy link

embray commented Oct 2, 2015

I can confirm that this PR fixes #94 for me. Thanks for the quick fix!

ionelmc added a commit that referenced this pull request Oct 2, 2015
Fix handling when tests change working directory.
@ionelmc ionelmc merged commit 8b627a6 into master Oct 2, 2015
@ionelmc ionelmc deleted the support-cwd-changes branch October 2, 2015 18:32
@schlamar
Copy link
Contributor

schlamar commented Oct 2, 2015

Great job! 🍰

@ionelmc
Copy link
Member Author

ionelmc commented Oct 3, 2015

The plan is to make PyPI release tomorrow.

@ionelmc
Copy link
Member Author

ionelmc commented Oct 4, 2015

Released in v2.2.0.

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.

3 participants