SDN-IP unit test fix: Increased the timeout waiting for a message to be
received from 2s to 5s. Eventually, this should address some rare
test failures.
Change-Id: Ifbf024023580ac93f10e1b7fb1b1f5dd2fe361f9
diff --git a/apps/sdnip/src/test/java/org/onlab/onos/sdnip/bgp/BgpSessionManagerTest.java b/apps/sdnip/src/test/java/org/onlab/onos/sdnip/bgp/BgpSessionManagerTest.java
index 8905289..d9b3774 100644
--- a/apps/sdnip/src/test/java/org/onlab/onos/sdnip/bgp/BgpSessionManagerTest.java
+++ b/apps/sdnip/src/test/java/org/onlab/onos/sdnip/bgp/BgpSessionManagerTest.java
@@ -61,6 +61,9 @@
private static final long DEFAULT_LOCAL_PREF = 10;
private static final long DEFAULT_MULTI_EXIT_DISC = 20;
+ // Timeout waiting for a message to be received
+ private static final int MESSAGE_TIMEOUT_MS = 5000; // 5s
+
// The BGP Session Manager to test
private BgpSessionManager bgpSessionManager;
@@ -212,10 +215,10 @@
peerBootstrap.connect(connectToSocket);
// Wait until the OPEN message is received
- peerFrameDecoder.receivedOpenMessageLatch.await(2000,
+ peerFrameDecoder.receivedOpenMessageLatch.await(MESSAGE_TIMEOUT_MS,
TimeUnit.MILLISECONDS);
// Wait until the KEEPALIVE message is received
- peerFrameDecoder.receivedKeepaliveMessageLatch.await(2000,
+ peerFrameDecoder.receivedKeepaliveMessageLatch.await(MESSAGE_TIMEOUT_MS,
TimeUnit.MILLISECONDS);
//
@@ -256,10 +259,10 @@
peerBootstrap.connect(connectToSocket);
// Wait until the OPEN message is received
- peerFrameDecoder.receivedOpenMessageLatch.await(2000,
+ peerFrameDecoder.receivedOpenMessageLatch.await(MESSAGE_TIMEOUT_MS,
TimeUnit.MILLISECONDS);
// Wait until the KEEPALIVE message is received
- peerFrameDecoder.receivedKeepaliveMessageLatch.await(2000,
+ peerFrameDecoder.receivedKeepaliveMessageLatch.await(MESSAGE_TIMEOUT_MS,
TimeUnit.MILLISECONDS);
// Get the BGP Session handler