> On Feb. 25, 2016, 7:21 p.m., Dan Smith wrote: > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/AgentUtil.java, > > line 57 > > <https://reviews.apache.org/r/44020/diff/1/?file=1271099#file1271099line57> > > > > It looks like you're setting things up so that running a test in > > eclipse will find a war file built by the command line build? I think > > that's going to be confusing. People will modify the code in eclipse and > > expect the modifications to have an effect when the run the test.
I don't see another way to allow the tests to run in Eclipse. They deploy the WAR into a web server. Do you think I should just disable the ability to run them in Eclipse? That'll make it harder for the people developing new tests for REST. - Bruce ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44020/#review120739 ----------------------------------------------------------- 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 > >
