Code review comment for lp:~salgado/lazr.restful/extension-interfaces

Revision history for this message
Guilherme Salgado (salgado) wrote :

On Fri, 2010-07-16 at 15:36 +0000, Leonard Richardson wrote:
> This branch is slow going but I have one suggestion. Can you test this assertion?
>
> + assert name in orig_iface, (
> + "Could not find interface where %s is defined" % name)
>
> I don't see how it could be triggered. But maye the point is that it never is triggered?

As we discussed at the Epic, I don't think we can trigger that assertion
without introducing a bug somewhere.

--
Guilherme Salgado <https://launchpad.net/~salgado>

« Back to merge proposal