On Sunday 02 December 2007 18:51, Eric Wilhelm wrote: > # from Ovid > > # on Sunday 02 December 2007 07:47: > >--- nadim khemir <[EMAIL PROTECTED]> wrote: > >> The subject says it all. IE: > > ... > > >> t/20_policies.t 15 2 578 583 > > > >It just means "you need to investigate this further". Personally, I > >would like to see it optionally mean failure,... > > How is "extra credit" *ever* any sort of failure?
Because a TODO means that it is not done not: it might happend to be done but I'm not really sure, maybe I get lucky. Either one removes the TODO and all is fine. Or it might just be a side effect that you haven't planned that makes the test pass. Calling "unexpected things" "features" does not make me feel more sure about quality. This is exactely why I complained loud about Test::Exception::dies_ok. It died but not for the problem I expected. Other may be better or have more luck than me though. I really would like to have -Werror for my tests. Cheers, Nadim.
