Skip to content

Fix problems with pytest#45

Merged
dwhswenson merged 2 commits intomasterfrom
fix_pytest_cov_master
Jan 15, 2019
Merged

Fix problems with pytest#45
dwhswenson merged 2 commits intomasterfrom
fix_pytest_cov_master

Conversation

@dwhswenson
Copy link
Owner

Travis is currently installing old versions of pytest with a newer version of pytest-cov, causing errors in tests. This PR is intended to fix that.

@dwhswenson dwhswenson added bug upstream_changed PR in response to changes in dependencies misc PR and removed bug labels Jan 15, 2019
@dwhswenson
Copy link
Owner Author

Apparently the problem was that Travis already had pytest (3.3) installed. Recent versions of pytest-cov require pytest>3.6. Our request to install pytest as part of our testing requirements was met with "already installed" (at 3.3). The solution is to request up-to-date versions of all testing packages (add a -U to the pip command).

Better dependency resolution would help (e.g., using conda). But I want to use pip installs for the testing environment on Travis (we use conda on AppVeyor).

I assume this will pass the AppVeyor builds, since this problem doesn't exist there. Once it does, I'll merge. @sroet, you're welcome to take a look, but I don't think the 2-character change requires a review!

@dwhswenson dwhswenson changed the title [WIP] Fix problems with pytest Fix problems with pytest Jan 15, 2019
@dwhswenson dwhswenson merged commit 0225f39 into master Jan 15, 2019
@dwhswenson dwhswenson deleted the fix_pytest_cov_master branch January 15, 2019 13:15
@dwhswenson dwhswenson mentioned this pull request Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

misc PR upstream_changed PR in response to changes in dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant