----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40871/#review108681 -----------------------------------------------------------
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunctionDUnitTest.java (line 35) <https://reviews.apache.org/r/40871/#comment168140> Consider making this test a JUnitTest. It runs in one jvm. Since you currently need a real PartitionedRegion it will need to be marked as an integration test. Also should you add a comment that it currently only has test coverage for GEODE-184 and the other methods on DataCommandFunction need coverage? gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunctionDUnitTest.java (line 86) <https://reviews.apache.org/r/40871/#comment168139> why is this property set? Remove it if this test does not need it. - Darrel Schneider On Dec. 2, 2015, 9:40 a.m., Jens Deppe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40871/ > ----------------------------------------------------------- > > (Updated Dec. 2, 2015, 9:40 a.m.) > > > Review request for geode, Barry Oglesby and Darrel Schneider. > > > Repository: geode > > > Description > ------- > > GEODE-184: The gfsh 'locate entry' command fails to find the entry on > partitioned regions if the key is not a string > > The bug report states exactly what the fix is. I'm also adding a test. Any > ideas on how to make this a less heavy test would be good. > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunction.java > d4dac05d7cfe9f775bdc1918c8d4621b1ae1bbb6 > > gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunctionDUnitTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/40871/diff/ > > > Testing > ------- > > Ran distributedTest task > > > Thanks, > > Jens Deppe > >
