[GEODE-9850] flaky test: testGetOldestTombstoneTimeForReplicateTombstoneSweeper

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.13.5
• Fix Version/s: 1.15.0
• Component/s: tests
• Labels: GeodeOperationAPI pull-request-available
• Assignee: Xiaojian Zhou
• Reporter: Bill Burcham
• Created: 24/Nov/21 00:28
• Updated: 22/Jun/22 20:46
• Resolved: 25/Nov/21 04:21

Description

First saw this failure in PR pipeline on support/1.13 here: https://concourse.apachegeode-ci.info/builds/3912569

org.apache.geode.internal.cache.versions.TombstoneDUnitTest > testGetOldestTombstoneTimeForReplicateTombstoneSweeper FAILED
    org.apache.geode.test.dunit.RMIException: While invoking org.apache.geode.internal.cache.versions.TombstoneDUnitTest$$Lambda$42/2046302475.run in VM 0 running on Host 9a305b2d7db7 with 4 VMs
        at org.apache.geode.test.dunit.VM.executeMethodOnObject(VM.java:610)
        at org.apache.geode.test.dunit.VM.invoke(VM.java:437)
        at org.apache.geode.internal.cache.versions.TombstoneDUnitTest.testGetOldestTombstoneTimeForReplicateTombstoneSweeper(TombstoneDUnitTest.java:228)

        Caused by:
        java.lang.AssertionError: 
        Expecting:
         <-1637701703343L>
        to be greater than:
         <0L> 
            at org.apache.geode.internal.cache.versions.TombstoneDUnitTest.lambda$testGetOldestTombstoneTimeForReplicateTombstoneSweeper$bb17a952$3(TombstoneDUnitTest.java:237) 

I believe the fix is to wrap this assertion in an awaitility call.

Issue Links

• links to: GitHub Pull Request #7141
• links to: GitHub Pull Request #7198