----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44020/#review120783 -----------------------------------------------------------
geode-web-api/build.gradle (line 64) <https://reviews.apache.org/r/44020/#comment182272> maybe checkout what the geode-assembly module is doing. The AgentUtilJunitTest in that module is checking the same thing distributedTest dependOnInstalledProduct integrationTest dependOnInstalledProduct this would make sure that if you run either integration tests or distributed tests, the war files are generated. - Jinmei Liao On Feb. 25, 2016, 9:41 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44020/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2016, 9:41 p.m.) > > > Review request for geode, Anthony Baker and Dan Smith. > > > Repository: geode > > > Description > ------- > > A number of unit tests were still sitting in the work queue to be moved to > open-source including all of the REST API tests. This change set shows the > build changes and AgentUtil changes needed to accomodate the new tests but > does not show the new tests. The tests are being added to the geode-web-api > project, and all of the support classes in geode-core's > com.gemstone.gemfire.rest.internal.web.controllers package are being moved to > that project. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/AgentUtil.java > 0310658d2182f90bc8e2041cd0f6b0f0b53c6568 > geode-web-api/build.gradle 51848c2b750373cfcc1c5ba9f0b24417ac38745d > gradle/dependency-versions.properties > 611ee7a413d70d9c72b5ca0a5384b4dee7c26c60 > > Diff: https://reviews.apache.org/r/44020/diff/ > > > Testing > ------- > > The new tests pass when run from gradle or eclipse > > > Thanks, > > Bruce Schuchardt > >