Hash: SHA1

Jonathan Lange wrote:
> On Sat, Apr 10, 2010 at 7:16 PM, Tres Seaver <tsea...@palladion.com> wrote:
>> 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.

Thank you!

> 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 :)

We'll try to do better this time ;).

- --
Tres Seaver          +1 540-429-0999          tsea...@palladion.com
Palladion Software   "Excellence by Design"    http://palladion.com
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org


Zope-Dev maillist  -  Zope-Dev@zope.org
**  No cross posts or HTML encoding!  **
(Related lists - 
 https://mail.zope.org/mailman/listinfo/zope )

Reply via email to