rusefi
/
rusefi
mirror of https://github.com/rusefi/rusefi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

random test refactoring

This commit is contained in:
rusefi 2019-01-10 23:33:50 -05:00
parent e7ee023d2d
commit 704e7112f4
2 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
unit_tests/engine_test_helper.cpp
View File

@ -93,7 +93,7 @@ void EngineTestHelper::firePrimaryTriggerFall() {
}
void EngineTestHelper::fireTriggerEventsWithDuration(int durationMs) {
fireTriggerEvents2(1, durationMs);
fireTriggerEvents2(/*count*/1, durationMs);
}
/**

5
unit_tests/tests/test_trigger_decoder.cpp
View File

@ -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);