+1 on this. We should break away from extending base test class now. It makes it quite hard to understand how the test cluster is set up and what the test is doing.
On Fri, Nov 9, 2018 at 10:50 AM Galen O'Sullivan <gosulli...@pivotal.io> wrote: > I was looking at a test recently that extended JUnit4CacheTestCase and only > used a single server, and changed it to use ClusterStartupRule. > > JUnit4CacheTestCase adds additional complexity to JUnit4DistributedTestCase > and with the move to ClusterStartupRule for distributed tests, rather than > class inheritance, I think we should deprecate JUnit4CacheTestCase and > change the comment to imply that classes should inherit from it just > because they require a Cache. > > Is is worth deprecating JUnit4DistributedTestCase as well and encouraging > the use of ClusterStartupRule instead? > > Thanks, > Galen > -- Cheers Jinmei
