Skip to content

Commit 4b98a79

Browse files
author
Myron Scott
authored
Merge pull request #1335 from myronkscott/fix_test
Fix test for slow systems
2 parents beca943 + bfc1b4c commit 4b98a79

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

tc-server/src/main/java/com/tc/objectserver/entity/ActiveToPassiveReplication.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -408,9 +408,9 @@ Map<SyncReplicationActivity.ActivityID, ActivePassiveAckWaiter> getWaiters() {
408408
return waiters;
409409
}
410410
// for test
411-
void finishPassiveSync(long timeout) throws InterruptedException {
411+
boolean finishPassiveSync(long timeout) throws InterruptedException {
412412
passiveSyncPool.shutdown();
413-
passiveSyncPool.awaitTermination(timeout, TimeUnit.MILLISECONDS);
413+
return passiveSyncPool.awaitTermination(timeout, TimeUnit.MILLISECONDS);
414414
}
415415

416416

tc-server/src/test/java/com/tc/objectserver/entity/ActiveToPassiveReplicationTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,10 @@ public void testNodeLeft() throws Exception {
109109
when(activity.getActivityID()).thenReturn(SyncReplicationActivity.ActivityID.getNextID());
110110
ActivePassiveAckWaiter ack2 = replication.replicateActivity(nowait, replication.passives());
111111
Assert.assertTrue(ack2.isCompleted());
112-
replication.finishPassiveSync(10000);
112+
int turns = 1;
113+
while (!replication.finishPassiveSync(10000)) {
114+
System.out.println("Still waiting for sync shutdown after " + turns++ + "0s");
115+
}
113116
removal.get();
114117
Assert.assertTrue(replication.getWaiters().isEmpty());
115118
}

0 commit comments

Comments
 (0)