[dmd-internals] [D-Programming-Language/dmd] 48ed15: Remove import of std.random:rand from test22

Brad Roberts braddr at puremagic.com
Tue Aug 23 11:14:14 PDT 2011


On Tue, 23 Aug 2011, Jason House wrote:
> On Aug 23, 2011, at 12:55 PM, Brad Roberts <braddr at puremagic.com> wrote:
> > 
> > For the time being, since this has been broken for a long time, can we 
> > revert/disable the failing test so that it doesn't hide other failures?
> 
> Personally, I hate the idea of completely disabling a test. I like when 
> test suites can handle expected failures or ignored tests. As an 
> example, cruise control will use yellow for ignored test, red for failed 
> tests, and green for passing tests. That's nicer, but still annoying 
> when trying to do test driven development.

In general, I greatly dislike disabling tests.  That said, when it's an 
OLD failure checking in the broken test before there's a fix for it, the 
value is questionable.

Additionally, the behavior your advocating is not what we have currently, 
so using what we have currently the best way we can is better than some 
theoretical it could be done differently.

If you'd like to invest in improving the infrastructure, please feel 
encouraged to do so.

Later,
Brad



More information about the dmd-internals mailing list