Remove setup.py tests and adapt docs to use tests.py #5434

Merged
merged 3 commits into from Dec 15, 2015

Conversation

Projects
None yet
4 participants
Owner

jenshnielsen commented Nov 8, 2015

Given the number of issues that we have been seeing with setup.py test
#4720 and #5315 among others I suggest removing it and recommending python tests.py

We already have 2/3 other ways of running the tests and the only thing that setup.py test brings is the ability to install the test dependencies which is done inside the build enviroment with setuptools only available to the test process. I think it is better just to recommend manually installing the test dependencies.

Owner

jenshnielsen commented Nov 8, 2015

Related to #5428 and may conflict with #5325

Owner

tacaswell commented Nov 11, 2015

👍 in principle but want to be careful to not conflict with #5405 too badly.

@pizzathief Your branch needs a resbase anyway. How much of a hassle would it be for us to merge this as-is and than have you re-base on top of the new master?

Owner

jenshnielsen commented Nov 11, 2015

Im ok with rebasing this if need be. I think we should change the docs as soon as possible for the next release. Actually removing the method can better wait

Owner

tacaswell commented Nov 11, 2015

Actually, could you split this up in to a documentation only commit? That
the docs say to use setup.py test as the primary method of testing is a
mistake on my part, i thought we had reverted ask of those.

I am +1 on merging the docs and rebuilding what we have deployed for 1.5

On Wed, Nov 11, 2015, 08:25 Jens Hedegaard Nielsen notifications@github.com
wrote:

Im ok with rebasing this if need be. I think we should change the docs as
soon as possible for the next release. Actually removing the method can
better wait


Reply to this email directly or view it on GitHub
#5434 (comment)
.

Owner

jenshnielsen commented Nov 11, 2015

I will cherry-pick to a new branch with the docs changes only and target the 1.5 docs branch

Contributor

pizzathief commented Nov 11, 2015

reasonably hassle free, I think.

Owner

tacaswell commented Nov 24, 2015

@jenshnielsen What is the state of this PR? I think this needs to be re-bassed because the docs only PR was merged.

jenshnielsen added some commits Nov 8, 2015

@jenshnielsen jenshnielsen Remove get tests requires 7820787
@jenshnielsen jenshnielsen Change setup.py test to a noop telling users to run tests.py 1c5c21c
@jenshnielsen jenshnielsen Clarifty how to install test dependencies
d9caef3
Owner

jenshnielsen commented Nov 24, 2015

@tacaswell Yes you are right. Rebase done

@tacaswell tacaswell added a commit that referenced this pull request Dec 15, 2015

@tacaswell tacaswell Merge pull request #5434 from jenshnielsen/nosetuptest
Remove setup.py tests and adapt docs to use tests.py
e789a70

@tacaswell tacaswell merged commit e789a70 into matplotlib:master Dec 15, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 67.386%
Details

tacaswell removed the needs_review label Dec 15, 2015

Owner

tacaswell commented Dec 15, 2015

@mdboom any problem porting this all they way back to v1.5.0-doc?

Owner

jenshnielsen commented Dec 15, 2015

The docs part was merged in #5462 so I guess that this can just be backported to 1.5.x ?

Owner

tacaswell commented Dec 15, 2015

Great, thanks for remembering that.

@tacaswell tacaswell added a commit that referenced this pull request Dec 15, 2015

@tacaswell tacaswell Merge pull request #5434 from jenshnielsen/nosetuptest
Remove setup.py tests and adapt docs to use tests.py

conflicts in setup.py due to parts of #5604 being back-ported
as well.  Removed the windows bdist patch code to resolve.
4a705d1
Owner

tacaswell commented Dec 15, 2015

backported to v1.5.x as 4a705d1

jenshnielsen deleted the jenshnielsen:nosetuptest branch Jan 5, 2016

jcfr referenced this pull request in scikit-build/scikit-build Jul 20, 2016

Closed

Ensure `python setup.py test` calls `nosetests` #90

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment