andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Added a timeout for the UdpClient so that the test cannot hang.

Browse Source 
I noticed the build "hanging/halting" on the test when running the cobertura code coverage Eclipse plugin.
This commit is contained in:
Daniel Bevenius 2011-02-10 13:29:53 +01:00 committed by Trustin Lee
parent b26f13d3b9
commit 98db0d3908
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/test/java/org/jboss/netty/channel/socket/nio/NioDatagramChannelTest.java
View File

@ -112,7 +112,7 @@ public class NioDatagramChannelTest {
assertFalse("The payload should have been cleared", expectedPayload assertFalse("The payload should have been cleared", expectedPayload
.equals(new String(dp.getData()))); .equals(new String(dp.getData())));
udpClient.receive(dp); udpClient.receive(dp, 4000);
assertEquals(expectedPayload, new String(dp.getData())); assertEquals(expectedPayload, new String(dp.getData()));
udpClient.close(); udpClient.close();

3
src/test/java/org/jboss/netty/channel/socket/nio/UdpClient.java
View File

@ -48,7 +48,8 @@ public class UdpClient {
return dp; return dp;
} }
public void receive(final DatagramPacket dp) throws IOException { public void receive(final DatagramPacket dp, final int timeout) throws IOException {
clientSocket.setSoTimeout(timeout);
clientSocket.receive(dp); clientSocket.receive(dp);
} }