> On May 11, 2016, 11:01 p.m., Kirk Lund wrote: > > gradle/test.gradle, line 112 > > <https://reviews.apache.org/r/47157/diff/1/?file=1377352#file1377352line112> > > > > 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
Unit we figure out GEODE-837 we will have a problem trying to run a separate test target for flaky tests. The tests run and results are reported in the html output but not in the TEST-* xml files used by Jenkins and other post-processors. - Anthony ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47157/#review132779 ----------------------------------------------------------- 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 > >
