This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eric.snow
Recipients eric.araujo, eric.snow, larry, ned.deily, vinay.sajip
Date 2014-02-22.18:31:26
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
TL;DR new tests (improving coverage) uncovered existing "bugs".  We should probably disable the tests for now.

I'm glad you found this.  Out of curiosity, how often do you run the test suite against a clean checkout?  Typically I only run it out of the same "tainted" directory that I develop in and not very frequently with -w.  Have you seen related failures on any buildbots?

Anyway, it looks like the actual pydoc changes in the patch aren't at fault.  Rather, the 3 test_modules* tests I added are.  Specifically, something in pydoc.Helper (i.e. help() in the REPL) is the problem.  That's what I get for trying to add test coverage (where there was none) for code I'm fixing! <wink>

If I recall correctly, passing 'modules' to help() does something funny like actually loading *every* module it can find.  I can understand how this might have side effects, maybe expose bugs in other modules, and even cause weird failures when running the test suite! :P

I'm not sure what it will take to get this sorted out.  This may actually be the way that pydoc.Helper()('modules') is supposed to work.  In that case we'd need to fix the modules that are having an issue, namely disutils (and probably logging).  We should open separate issues for each module that needs fixing.  Until then we should probably disable those three tests, particularly for the upcoming 3.4 release (and rc2 if we can squeeze it in).

Any objections before I disable those 3 tests?
Date User Action Args
2014-02-22 18:31:28eric.snowsetrecipients: + eric.snow, vinay.sajip, larry, ned.deily, eric.araujo
2014-02-22 18:31:28eric.snowsetmessageid: <>
2014-02-22 18:31:28eric.snowlinkissue20484 messages
2014-02-22 18:31:26eric.snowcreate