----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42479/#review115213 -----------------------------------------------------------
gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SyncChunkStackJUnitTest.java (line 19) <https://reviews.apache.org/r/42479/#comment176117> I think we need to run all our tests with assertions. Rahter than adding here, can you please add `-ea` to all the test targets in gradle scripts? - Swapnil Bawaskar On Jan. 19, 2016, 12:46 a.m., Darrel Schneider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42479/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2016, 12:46 a.m.) > > > Review request for geode and Swapnil Bawaskar. > > > Bugs: GEODE-783 > https://issues.apache.org/jira/browse/GEODE-783 > > > Repository: geode > > > Description > ------- > > added unit test for SyncChunkStack > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/offheap/SyncChunkStack.java > a615af0a601a1e0bdbbc5236e6c802944ca57caa > > gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SyncChunkStackJUnitTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/42479/diff/ > > > Testing > ------- > > ran SyncChunkStackJUnitTest > > > Thanks, > > Darrel Schneider > >
