[merge] doc how to use new test features

Martin Pool mbp at sourcefrog.net
Wed May 2 15:08:56 BST 2007


On 5/2/07, Aaron Bentley <aaron.bentley at utoronto.ca> wrote:
> Martin Pool wrote:
> > silently return - this test just doesn't make sense in this case
>
> I don't like silent returns at all.  The test should not report that it
> passed unless it ran to completion.

Yes, that was my point in my later post -- and that this is commonly
how TestSkipped is currently used.  I don't mind if we have a
different status meaning "test not implemented" or "test prerequisite
not implemented yet."

> > This raises the question of when it's acceptable to have tests in any
> > of these cases.  Will we merge code with known failures?
>
> I think that has to be case-by-case.  We would certainly merge a test
> case that failed for already-present code.  Otherwise, it's a tradeoff
> based on the utility of the code vs the importance of the failure vs the
> difficulty in fixing the code.

Right.  Maybe we should just disclose it in reviews - "this adds one
more known failure for bug #123".

-- 
Martin



More information about the bazaar mailing list