> On Feb. 25, 2016, 10:02 p.m., Jinmei Liao wrote: > > geode-web-api/build.gradle, line 64 > > <https://reviews.apache.org/r/44020/diff/2/?file=1271298#file1271298line64> > > > > 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.
Thanks Jinmei - I'll try that - Bruce ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44020/#review120783 ----------------------------------------------------------- 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 > >
