On Wed, Feb 21, 2018 at 6:13 PM, Uwe Schindler <[email protected]> wrote: >> > There are exactly three >> > BadApple annotations in the entire code base at present, is there >> > enough value in introducing another annotation to make it worthwhile? >> >> If we change BadApple tests to be executed by default for "ant test" >> (but not for the most frequent jenkins jobs), then that would be fine. >> Basically, add a -Dtests.disable-badapples and use that for the >> jenkins jobs that email the list all the time. > > No need for a new sysprop. It's already there, just inverted! Configuring > Jenkins to enable disable them is trivial.
The issue is that flakey tests should not be ignored by developers running unit tests before committing new changes. That's the most important point in time for test coverage. -Yonik --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
