Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-06-21 01:01:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix race conditions in command tests by increasing the delay time (#1178)

Browse Source
This commit is contained in:
Austin Shalit
2018-07-08 15:39:15 -04:00
committed by Peter Johnson
parent ad3e2d7d3b
commit f5b1028b5a
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
wpilibj/src/test/java/edu/wpi/first/wpilibj/command/CommandSequentialGroupTest.java
View File

@@ -56,7 +56,7 @@ class CommandSequentialGroupTest extends AbstractCommandTest {
assertCommandState(command1, 1, 1, 1, 0, 0);
assertCommandState(command2, 0, 0, 0, 0, 0);
assertCommandState(command3, 0, 0, 0, 0, 0);
sleep(1000); // command 1 timeout
sleep(1250); // command 1 timeout
Scheduler.getInstance().run();
assertCommandState(command1, 1, 1, 1, 0, 1);
assertCommandState(command2, 1, 1, 1, 0, 0);
@@ -66,7 +66,7 @@ class CommandSequentialGroupTest extends AbstractCommandTest {
assertCommandState(command1, 1, 1, 1, 0, 1);
assertCommandState(command2, 1, 2, 2, 0, 0);
assertCommandState(command3, 0, 0, 0, 0, 0);
sleep(2000); // command 2 timeout
sleep(2500); // command 2 timeout
Scheduler.getInstance().run();
assertCommandState(command1, 1, 1, 1, 0, 1);
assertCommandState(command2, 1, 2, 2, 0, 1);

2
wpilibj/src/test/java/edu/wpi/first/wpilibj/command/CommandTimeoutTest.java
View File

@@ -38,7 +38,7 @@ class CommandTimeoutTest extends AbstractCommandTest {
assertCommandState(command, 1, 2, 2, 0, 0);
Scheduler.getInstance().run();
assertCommandState(command, 1, 3, 3, 0, 0);
sleep(2000);
sleep(2500);
Scheduler.getInstance().run();
assertCommandState(command, 1, 4, 4, 1, 0);
Scheduler.getInstance().run();