diff --git a/java_console/autotest/src/com/rusefi/AutoTest.java b/java_console/autotest/src/com/rusefi/AutoTest.java index a881cfcab7..cdfd5680b3 100644 --- a/java_console/autotest/src/com/rusefi/AutoTest.java +++ b/java_console/autotest/src/com/rusefi/AutoTest.java @@ -75,24 +75,32 @@ public class AutoTest extends BaseTest { sendCommand("set " + CMD_ENGINESNIFFERRPMTHRESHOLD + " 100"); changeRpm(900); // first let's get to expected RPM - assertRpmDoesNotJump(20000, 15, 30, FAIL, commandQueue); + assertRpmDoesNotJump(20000, 5, 40, FAIL, commandQueue); } private void testV12() { setEngineType(ET_BMW_M73_F); changeRpm(700); // first let's get to expected RPM - assertRpmDoesNotJump(20000, 15, 30, FAIL, commandQueue); + assertRpmDoesNotJump(20000, 5, 40, FAIL, commandQueue); + testCaseBug1873(); + } + + private void testCaseBug1873() { + assertRpmDoesNotJump(60, 5, 110, FAIL, commandQueue); } public static void assertRpmDoesNotJump(int rpm, int settleTime, int testDuration, Function callback, CommandQueue commandQueue) { IoUtil.changeRpm(commandQueue, rpm); sleepSeconds(settleTime); AtomicReference result = new AtomicReference<>(); + long start = System.currentTimeMillis(); SensorCentral.SensorListener listener = value -> { double actualRpm = SensorCentral.getInstance().getValue(Sensor.RPM); - if (!isCloseEnough(rpm, actualRpm)) - result.set("Got " + actualRpm + " while trying to stay at " + rpm); + if (!isCloseEnough(rpm, actualRpm)) { + long seconds = (System.currentTimeMillis() - start) / 1000; + result.set("Got " + actualRpm + " while trying to stay at " + rpm + " after " + seconds + " seconds"); + } }; SensorCentral.getInstance().addListener(Sensor.RPM, listener); sleepSeconds(testDuration);