Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Small tweak on test

This commit is contained in:
Clebert Suconic 2015-10-26 14:06:20 -04:00
parent 32e6d9c61a
commit 992c34bc6a
1 changed files with 2 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/SessionCloseOnGCTest.java
View File

@ -216,20 +216,8 @@ public class SessionCloseOnGCTest extends ActiveMQTestBase {
ActiveMQTestBase.checkWeakReferences(ref1, ref2, ref3); ActiveMQTestBase.checkWeakReferences(ref1, ref2, ref3);
int count = 0; for (int i = 0; i < 1000 && sf.numSessions() != 0; i++) {
final int TOTAL_SLEEP_TIME = 400; forceGC();
final int MAX_COUNT = 20;
while (count++ < MAX_COUNT) {
/*
* The assertion is vulnerable to races, both in the session closing as well as the return
* value of the sessions.size() (i.e. HashSet.size()).
*/
synchronized (this) {
// synchronized block will (as a side effect) force sync all field values
if (sf.numSessions() == 0)
break;
Thread.sleep(TOTAL_SLEEP_TIME / MAX_COUNT);
}
} }
Assert.assertEquals("# sessions", 0, sf.numSessions()); Assert.assertEquals("# sessions", 0, sf.numSessions());
Assert.assertEquals("# connections", 1, sf.numConnections()); Assert.assertEquals("# connections", 1, sf.numConnections());