----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47157/#review132779 -----------------------------------------------------------
Fix it, then Ship it! Add "forkEvery 1" and review GEODE-837. Need to address policy about precheckin and flaky tests. And then Ship It! gradle/test.gradle <https://reviews.apache.org/r/47157/#comment197026> Dan added exclude *DUnitTest here because of GEODE-837 -- I don't really understand the problem well enough to know if it's ok to remove the exclusion gradle/test.gradle (line 134) <https://reviews.apache.org/r/47157/#comment197028> Please add "forkEvery 1" to this task - Kirk Lund On May 10, 2016, 4:49 a.m., Anthony Baker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47157/ > ----------------------------------------------------------- > > (Updated May 10, 2016, 4:49 a.m.) > > > Review request for geode, Dick Cavender, Kirk Lund, Mark Bretl, and Dan Smith. > > > Repository: geode > > > Description > ------- > > Add a new test task specifically for running tests annotated with > @FlakyTest. This allows us to isolate tests that exhibit > intermittent failures. Other changes include: > > - Exclude flaky tests from test, integrationTest, uiTest, and > distributedTest tasks > - Cleanup how tests are found using the minimum set of > include/exclude categories > > > Diffs > ----- > > gradle/test.gradle 2dd75210b38fe37120778e35a4f96dbd08de97fe > > Diff: https://reviews.apache.org/r/47157/diff/ > > > Testing > ------- > > Tested by running test, flakyTest, integrationTest. Will run further tests > before pushing. > > > Thanks, > > Anthony Baker > >
