← Back to team overview

launchpad-dev team mailing list archive

Re: warning: we will soon have much noise in the test results...

 

On Mon, Jul 26, 2010 at 10:10 AM, Abel Deuring
<abel.deuring@xxxxxxxxxxxxx> wrote:

> 1. I don't know why we raised errors for non-silenced warnings. In
> general, we should aim to get rid of all warnings in our code/tests. I
> would even consider the warnings introduced by my branch to be errors --
> the point is that they occur far too often to be fixed in a single
> branch, so I opted instead to annoy everbody who looks at the output of
> affected tests ;)

For future reference, see the patch James W put together to attach
oopses to tests; a similar approach would have let you gather all the
items without them showing up : we could then ratchet it down as a
form of lint without having the UI exploding in our faces.

Separately, I'm a little concerned we may spend a lot of time churning
between users to setup things now - and that that will have
performance implications for the test suite.

Lastly, and here I expose my ignorance of some subtleties in zope - I
thought security proxies only lived between view and model objects,
not between model objects?

-Rob



Follow ups

References