On Sat, Apr 10, 2010 at 7:16 PM, Tres Seaver <tsea...@palladion.com> wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Jonathan Lange wrote: >> Hello, >> >> The subunit output formatter that was added recently currently raises >> NotImplementedError when it's given a generic "error". The error() >> method on formatters is used to communicate user error (e.g. when the >> user selects incompatible options), internal framework errors (afaict, >> only when there's a subprocess read error) and layer setUp errors. >> >> I find the idea of masking layer setUp errors to be unconscionably >> awful, so I've attached the following patch, which changes the subunit >> formatter's error() method to print out whatever it gets. It doesn't >> have any tests, but it's simple enough, and none of the other >> formatters have tests for this method either. >> >> Ideally, I think there should be a new method on OutputFormatter to >> handle layer setup failure. Probably teardown failure too. If someone >> wants to push back on this simple patch in favour of a new method, I >> could probably be persuaded to do the work, provided I had clear >> backwards compatibility guidelines. > > Your patches look pretty reasonable to me: can you kick them into > Launchpad so they don't get lost in the mailing list archives? There is > a project page for zope.app.testing, but it doesn't (as of today) have > the bug tracker turned on. > > https://launchpad.net/zope.app.testing >
Branches pushed, merge proposals created, bugs filed, branches linked. FWIW, I sent to the mailing list first because last time I tried this I filed bugs, pushed up branches etc. and didn't get any response until I posted patches to the ML :) jml _______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )