random test refactoring
This commit is contained in:
parent
51a708117a
commit
7c9a99fb72
|
@ -93,7 +93,7 @@ void EngineTestHelper::firePrimaryTriggerFall() {
|
|||
}
|
||||
|
||||
void EngineTestHelper::fireTriggerEventsWithDuration(int durationMs) {
|
||||
fireTriggerEvents2(1, durationMs);
|
||||
fireTriggerEvents2(/*count*/1, durationMs);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -326,7 +326,7 @@ void testRpmCalculator(void) {
|
|||
assertEquals(5, TRIGGER_SHAPE(triggerIndexByAngle[240]));
|
||||
assertEquals(5, TRIGGER_SHAPE(triggerIndexByAngle[241]));
|
||||
|
||||
eth.fireTriggerEvents(48);
|
||||
eth.fireTriggerEvents(/* count */ 48);
|
||||
|
||||
assertEqualsM("RPM", 1500, engine->rpmCalculator.getRpm(PASS_ENGINE_PARAMETER_SIGNATURE));
|
||||
assertEqualsM("index #1", 15, engine->triggerCentral.triggerState.getCurrentIndex());
|
||||
|
@ -343,9 +343,6 @@ void testRpmCalculator(void) {
|
|||
int start = eth.getTimeNowUs();
|
||||
assertEqualsM("start value", 485000, start);
|
||||
|
||||
// todo: why is this required here? we already have one 'prepareOutputSignals' in constructor, what's wrong with it?
|
||||
prepareOutputSignals(PASS_ENGINE_PARAMETER_SIGNATURE);
|
||||
|
||||
eth.engine.periodicFastCallback(PASS_ENGINE_PARAMETER_SIGNATURE);
|
||||
|
||||
assertEqualsM("fuel #1", 4.5450, engine->injectionDuration);
|
||||
|
|
Loading…
Reference in New Issue