Code review comment for lp:~lifeless/launchpad/test

Revision history for this message
Robert Collins (lifeless) wrote :

Thats an Attribute Docstring - see PEP257.

I've expanded the audit comment - callers need to cleanUp if setUp fails (or we need to decide that the fixture must not need a cleanUp if setUp fails)).

That is, infact, probably how we want to play it, so I'll look at an enhancement to fixtures to make that easy to do.

-Rob

« Back to merge proposal