----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54801/#review159626 -----------------------------------------------------------
Ship it! Ship It! - Kevin Duling On Dec. 16, 2016, 2:11 p.m., Kirk Lund wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54801/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2016, 2:11 p.m.) > > > Review request for geode, Jinmei Liao, Jared Stewart, and Kevin Duling. > > > Bugs: GEODE-1662 and GEODE-2207 > https://issues.apache.org/jira/browse/GEODE-1662 > https://issues.apache.org/jira/browse/GEODE-2207 > > > Repository: geode > > > Description > ------- > > The point of this test originally was to start up a Manager and a non-Manager > with a SecurityManager and verify that init and close were correctly being > invoked in both. At some point the test was marked as Flaky and then > completed disabled with Ignore. > > I've renamed the test and some methods to make it more clear about what it's > doing and testing. I also tried to make it more stable and ran it 1000 times > to try to shake the flakiness. It seems stable now. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/security/IntegratedSecurityCacheLifecycleDistributedTest.java > 00a9276c51c45e5260a3795258d186f99edaa21f > geode-core/src/test/java/org/apache/geode/security/SpySecurityManager.java > 8704e934a04910907bbcda068629086d9c278143 > > Diff: https://reviews.apache.org/r/54801/diff/ > > > Testing > ------- > > targeted testing for flakiness > precheckin in progress > > > File Attachments > ---------------- > > With Rule > > https://reviews.apache.org/media/uploaded/files/2016/12/16/48f16ec0-e57a-4bc1-8183-649f1fde5fca__SecurityManagerCallbacksDistributedTest.java > Without Rule > > https://reviews.apache.org/media/uploaded/files/2016/12/16/739cb2d0-6798-45ab-863f-b400d4e814a3__SecurityManagerCallbacksDistributedTest.java > > > Thanks, > > Kirk Lund > >
