Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added hiding of docstrings by default. #1

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

babbageclunk commented Nov 13, 2011

Turn-off-able with --show-docstrings.

I had to rejig the BetterLoader code to get it to work correctly - with it returning a new loader, rather than patching the existing one, I could see that prepareTestResult was never called. (Not sure exactly why that was - let's have a look tomorrow.)

babbageclunk added some commits Nov 13, 2011

Stop prepareTestLoader from clobbering prepareTestResult.
Returning a new TestLoader from prepareTestLoader prevented nose from
calling prepareTestResult, so docstrings would still appear. (I'm not
sure exactly why this occurs - I haven't found the cause in the nose
code, although there are a few parts that look to contribute to it.)
Changed prepareTestLoader to patch the passed-in loader instead.
Owner

cjw296 commented Dec 13, 2011

A variant of this has been merged and released for a while :-)

@cjw296 cjw296 closed this Dec 13, 2011

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