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

Minimal fix on waiting time in TrafficShaping test

Browse Source 
Motivation:
Possibly due to very small time (< 100ms), the trafficShaping time might be a bit high in rare condition.

Modification:
Remove extra time (only stepms is kept, not minimalms).

Result:
Time shall be ok now.
This commit is contained in:
Frédéric Brégier 2014-08-24 11:25:12 +02:00
parent 0258772deb
commit 2a0d8818f1
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/test/java/org/jboss/netty/handler/traffic/TrafficShapingHandlerTest.java
View File

@ -176,7 +176,7 @@ public class TrafficShapingHandlerTest {
minimalWaitBetween[0] = 0; minimalWaitBetween[0] = 0;
for (int i = 0; i < multipleMessage.length; i++) { for (int i = 0; i < multipleMessage.length; i++) {
if (multipleMessage[i] > 1) { if (multipleMessage[i] > 1) {
minimalWaitBetween[i + 1] = (multipleMessage[i] - 1) * stepms + minimalms; minimalWaitBetween[i + 1] = (multipleMessage[i] - 1) * stepms;
} else { } else {
minimalWaitBetween[i + 1] = 10; minimalWaitBetween[i + 1] = 10;
} }
@ -188,7 +188,7 @@ public class TrafficShapingHandlerTest {
long[] minimalWaitBetween = new long[multipleMessage.length + 1]; long[] minimalWaitBetween = new long[multipleMessage.length + 1];
for (int i = 0; i < multipleMessage.length; i++) { for (int i = 0; i < multipleMessage.length; i++) {
if (multipleMessage[i] > 1) { if (multipleMessage[i] > 1) {
minimalWaitBetween[i] = (multipleMessage[i] - 1) * stepms + minimalms; minimalWaitBetween[i] = (multipleMessage[i] - 1) * stepms;
} else { } else {
minimalWaitBetween[i] = 10; minimalWaitBetween[i] = 10;
} }