[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