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

More updates to the Gyro test fixing potential null pointer exception

Browse Source 
also retuned the deviation over time test to make it pass more reliably.
Ran sucessfully 40/40 times run including several run with the entire wpilibj

Change-Id: I2e42c368fdb81f9206e15ce39878ea105da1e405
This commit is contained in:
Patrick
2016-02-08 15:03:51 -05:00
parent f17d27aacf
commit f436b33d79
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
wpilibjIntegrationTests/src/main/java/edu/wpi/first/wpilibj/GyroTest.java
View File

@@ -100,7 +100,7 @@ public class GyroTest extends AbstractComsSetup {
assertEquals(errorMessage(angle, 0), 0, angle, .5);
Timer.delay(5);
angle = gyro.getAngle();
assertEquals("After 5 seconds " + errorMessage(angle, 0), 0, angle, 1);
assertEquals("After 5 seconds " + errorMessage(angle, 0), 0, angle, 2.0);
}
/**

7
wpilibjIntegrationTests/src/main/java/edu/wpi/first/wpilibj/fixtures/TiltPanCameraFixture.java
View File

@@ -65,6 +65,7 @@ public abstract class TiltPanCameraFixture implements ITestFixture {
@Override
public boolean reset() {
if(gyro != null)
gyro.reset();
return true;
}
@@ -101,8 +102,14 @@ public abstract class TiltPanCameraFixture implements ITestFixture {
tilt = null;
pan.free();
pan = null;
if(gyro !=null){//in case not freed during gyro tests
gyro.free();
gyro = null;
}
if(gyroParam != null){//in case gyro tests failed before getting to this point
gyroParam.free();
gyroParam = null;
}
return true;
}