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

Skip to content

Don't install the nose KnownFailure plugin as a pkg_resources entry_point #2139

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 21, 2013
Merged

Don't install the nose KnownFailure plugin as a pkg_resources entry_point #2139

merged 2 commits into from
Jun 21, 2013

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented Jun 19, 2013

As reported by @fperez on matplotlib-devel "Weird KnownFailure problem in IPython test suite with mpl master", this can interfere with the KnownFailure plugin in IPython.

@mdboom
Don't install the KnownFailure nosetest plugin as a pkg_resources ent…
cee81f8 
…ry_point, as it can, under some circumstances, conflict with the one in IPython. This means that the supported method to run the tests is from the tests.py script or `import matplotlib; matplotlib.test()`.
@mdboom
Copy link
Member Author

mdboom commented Jun 19, 2013

The plot thickens. If we don't install the KnownFailure plugin as an entry point, it isn't found when doing parallel testing. The following is the output of just doing the test_basic.py tests:

cheetah ~t/matplotlib › ~b/matplotlib/tests.py                                                                                                                                                       1
.K.
----------------------------------------------------------------------
Ran 3 tests in 0.023s

OK (KNOWNFAIL=1)
cheetah ~t/matplotlib › ~b/matplotlib/tests.py --processes=-1 --process-timeout=300
.E.
======================================================================
ERROR: matplotlib.tests.test_basic.test_simple_knownfail
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/mdboom/python/lib/python2.7/site-packages/matplotlib-1.3.0rc3-py2.7-linux-x86_64.egg/matplotlib/testing/decorators.py", line 48, in failer
    raise KnownFailureTest(msg) # An error here when running nose means that you don't have the matplotlib.testing.noseclasses:KnownFailure plugin in use.
KnownFailureTest: Test known to fail

----------------------------------------------------------------------
Ran 3 tests in 0.078s

mdboom added a commit that referenced this pull request Jun 21, 2013
@mdboom
Merge pull request #2139 from mdboom/dont-install-nose-plugin
d53e6b2 
Don't install the nose KnownFailure plugin as a pkg_resources entry_point
@mdboom mdboom merged commit d53e6b2 into matplotlib:v1.3.x Jun 21, 2013
@mdboom mdboom deleted the dont-install-nose-plugin branch August 7, 2014 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.3.x blocker
Development

Successfully merging this pull request may close these issues.
1 participant
@mdboom