X Tutup
Skip to content

Convert mpl toolkits tests to pytest + minor cleanup#7887

Merged
phobson merged 2 commits intomatplotlib:masterfrom
QuLogic:pytest-mpl_toolkits
Jan 23, 2017
Merged

Convert mpl toolkits tests to pytest + minor cleanup#7887
phobson merged 2 commits intomatplotlib:masterfrom
QuLogic:pytest-mpl_toolkits

Conversation

@QuLogic
Copy link
Copy Markdown
Member

@QuLogic QuLogic commented Jan 20, 2017

Besides the pytest stuff, drop unnecessary setup and testing functions.

@QuLogic QuLogic added this to the 2.1 (next point release) milestone Jan 20, 2017
@NelleV NelleV changed the title Convert mpl toolkits tests to pytest + minor cleanup [MRG+1] Convert mpl toolkits tests to pytest + minor cleanup Jan 20, 2017
@jkseppan
Copy link
Copy Markdown
Member

Does the AppVeyor failure mean you should remove mpl_toolkits.tests.test_mplot3d from default_test_modules in lib/matplotlib/__init__.py?

@QuLogic
Copy link
Copy Markdown
Member Author

QuLogic commented Jan 20, 2017

Yes, I believe so; I didn't realize those builds don't install pytest at all. Though it doesn't technically break the nose build if it were installed, I'll just remove them from the list.

The setup is a) not done for normal matplotlib tests and b) mostly
implemented in the image_comparison decorator.
@QuLogic QuLogic force-pushed the pytest-mpl_toolkits branch from a3b2982 to af79d51 Compare January 20, 2017 22:05
@phobson phobson merged commit aabb6e8 into matplotlib:master Jan 23, 2017
@phobson phobson changed the title [MRG+1] Convert mpl toolkits tests to pytest + minor cleanup Convert mpl toolkits tests to pytest + minor cleanup Jan 23, 2017
@QuLogic QuLogic deleted the pytest-mpl_toolkits branch January 23, 2017 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup