Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Test all #611

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

Merged
merged 2 commits into from
Dec 6, 2011
Merged

Test all #611

merged 2 commits into from
Dec 6, 2011

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented Dec 6, 2011

Closes #571.

Allows nosetests matplotlib to work. (I hope -- I was not able to reproduce @fperez's error).

Doesn't dump any files outside of the result_images directory.

@fperez
Copy link
Member

fperez commented Dec 6, 2011

Confirming that this removes the Cocoa problem reported in #571.

Also ran the full test suite and everything looks clean, no spurious images left outside the test directory.

Thanks for the quick response!

ps - on my system, merging this on top of current master I get:

Ran 1046 tests in 601.790s

FAILED (errors=21, failures=248)
ICE default IO error handler doing an exit(), pid = 18143, errno = 32

but I don't know if you make it a point of keeping master in passing condition...

mdboom added a commit that referenced this pull request Dec 6, 2011
@mdboom mdboom merged commit 9ae8771 into matplotlib:v1.1.x Dec 6, 2011
@mdboom
Copy link
Member Author

mdboom commented Dec 6, 2011

I'm only getting 2 failures on master at the moment. Can you file a new issue with the output from nose?

@fperez
Copy link
Member

fperez commented Dec 6, 2011

OK, I'll repeat the tests once more to make sure it wasn't a glitch in my system, and will file a ticket with all the nose output if I see similar results.

@fperez
Copy link
Member

fperez commented Dec 7, 2011

Done, #612. I did a fully clean build, nuking everything before installing, and now it's much better. Still 22 errors and 10 failures, but not hundreds.

@mdboom mdboom deleted the test_all branch March 3, 2015 18:44
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.

2 participants