[ 
https://issues.apache.org/jira/browse/GEODE-5175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16983946#comment-16983946
 ] 

ASF subversion and git services commented on GEODE-5175:
--------------------------------------------------------

Commit fc20f2fa07fc2e6534bad0046b05dd7f0941e06d in geode's branch 
refs/heads/develop from Kirk Lund
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=fc20f2f ]

GEODE-4187: Fix PersistentColocatedPartitionedRegionDistributedTest

Fix flakiness in PersistentColocatedPartitionedRegionDistributedTest:
* GEODE-4187 testRebalanceDuringRecovery hang
* GEODE-5175 inline PersistentPartitionedRegionTestBase
* GEODE-6499 AsyncInvocation timeout
* GEODE-7082 testReplaceOfflineMemberAndRestartCreateColocatedPRLate
  hang

Changes to fix flakiness:
* Clean up uses of AsyncInvocation
* Replace sleep with Mockito.verify using timeout()
* Replace sleep with CountDownLatch
* Use GeodeAwaitility timeout consistently

Extract integration tests:
* PersistentColocationIntegrationTest
* ModifyColocationIntegrationTest

Changes to product code:
* Move ColocationLogger to internal.cache.partitioned.colocation
* Extract factory and interfaces from ColocationLogger
* Create unit tests for ColocationLogger and factory
* Pass ColocationLoggerFactory into PartitionedRegion constructor
* Use ColocationLogger via interfaces instead of implementations
* Add getLogger(Class) to LogService

Clean up PersistentColocatedPartitionedRegionDistributedTest
* Fix uses of ArgumentCaptor and Mockito timeout
* Improve readability
* User TemporaryFolder Rule for all files
* Use RestoreSystemProperties Rule instead of try-finally blocks
* Use tearDown instead of try-finally blocks
* Update to non-deprecated APIs
* Replace try-catch blocks with AssertJ catchThrowable
* Use lambdas instead of SerializableRunnable instances

Other test file changes:
* Clean up PartitionedRegionTest
* Delete MockAppender


> Remove PersistentPartitionedRegionTestBase
> ------------------------------------------
>
>                 Key: GEODE-5175
>                 URL: https://issues.apache.org/jira/browse/GEODE-5175
>             Project: Geode
>          Issue Type: Improvement
>          Components: tests
>            Reporter: Kirk Lund
>            Assignee: Kirk Lund
>            Priority: Minor
>
> Remove PersistentPartitionedRegionTestBase and overhaul the tests that extend 
> it:
> {noformat}
> JUnit4DistributedTestCase (org.apache.geode.test.dunit.internal)
>     JUnit4CacheTestCase (org.apache.geode.test.dunit.cache.internal)
>         PersistentPartitionedRegionTestBase 
> (org.apache.geode.internal.cache.partitioned)
>             PersistentPartitionedRegionWithTransactionDUnitTest 
> (org.apache.geode.internal.cache.partitioned)
>                 PersistentPartitionedRegionWithDistTXDUnitTest 
> (org.apache.geode.disttx)
>             PersistentColocatedPartitionedRegionDUnitTest 
> (org.apache.geode.internal.cache.partitioned)
>             PersistentPartitionedRegionDUnitTest 
> (org.apache.geode.internal.cache.partitioned)
>                 PersistentPartitionedRegionOldConfigDUnitTest 
> (org.apache.geode.internal.cache.partitioned)
>             BackupDistributedTest (org.apache.geode.internal.cache.backup)
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to