Skip to content

Commit 35d5eba

Browse files
committed
Fix broken unit tests
1 parent 8c298d1 commit 35d5eba

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

core/src/main/java/org/strykeforce/thirdcoast/talon/StatusFrameRate.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,9 @@
2121
@ParametersAreNonnullByDefault
2222
public final class StatusFrameRate {
2323

24-
/**
25-
* Sets Talon to default frame rates.
26-
*/
24+
/** Sets Talon to default frame rates. */
2725
public static final StatusFrameRate DEFAULT = StatusFrameRate.builder().build();
28-
/**
29-
* Sets Talon to 5 ms rate for all frames.
30-
*/
26+
/** Sets Talon to 5 ms rate for all frames. */
3127
public static final StatusFrameRate GRAPHER;
3228

3329
static {

core/src/test/groovy/org/strykeforce/thirdcoast/talon/TalonFactoryTest.groovy

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,12 @@ class TalonFactoryTest extends Specification {
1818

1919
then:
2020
1 * wrapperFactory.createWrapper(27, TalonFactory.CONTROL_FRAME_MS) >> wrapper
21+
1 * wrapper.setStatusFrameRateMs(CANTalon.StatusFrameRate.AnalogTempVbat, 100)
2122
1 * wrapper.changeControlMode(CANTalon.TalonControlMode.Voltage)
23+
1 * wrapper.setStatusFrameRateMs(CANTalon.StatusFrameRate.Feedback, 20)
24+
1 * wrapper.setStatusFrameRateMs(CANTalon.StatusFrameRate.General, 10)
25+
1 * wrapper.setStatusFrameRateMs(CANTalon.StatusFrameRate.PulseWidth, 100)
26+
1 * wrapper.setStatusFrameRateMs(CANTalon.StatusFrameRate.QuadEncoder, 100)
2227
1 * wrapper.clearIAccum()
2328
1 * wrapper.ClearIaccum()
2429
1 * wrapper.clearMotionProfileHasUnderrun()

core/src/test/groovy/org/strykeforce/thirdcoast/telemetry/TelemetryServiceTest.groovy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ class TelemetryServiceTest extends Specification {
2626
telemetry.register(talon) // 4
2727

2828
then:
29-
2 * talon.setStatusFrameRateMs(CANTalon.StatusFrameRate.General, 10)
30-
1 * target.setStatusFrameRateMs(CANTalon.StatusFrameRate.General, 10)
29+
0 * talon.setStatusFrameRateMs(_)
30+
0 * target.setStatusFrameRateMs(_)
3131

3232
when:
3333
talon.getDeviceID() >>> [1, 3, 5, 4] // configureStatusFrameRates causes calls

tct/src/main/java/org/strykeforce/thirdcoast/telemetry/tct/talon/TalonSet.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import javax.inject.Singleton;
1212
import org.slf4j.Logger;
1313
import org.slf4j.LoggerFactory;
14-
import org.strykeforce.thirdcoast.talon.StatusFrameRate;
1514
import org.strykeforce.thirdcoast.talon.TalonConfiguration;
1615
import org.strykeforce.thirdcoast.talon.TalonConfigurationBuilder;
1716
import org.strykeforce.thirdcoast.telemetry.TelemetryService;

0 commit comments

Comments
 (0)