these lines just did NOTHING?

This commit is contained in:
Matthew Kennedy 2023-11-01 17:16:14 -07:00
parent c7864c601e
commit 83c6ac9f97
1 changed files with 0 additions and 6 deletions

View File

@ -187,14 +187,12 @@ void EngineTestHelper::smartFireFall(float delayMs) {
*/ */
void EngineTestHelper::firePrimaryTriggerRise() { void EngineTestHelper::firePrimaryTriggerRise() {
efitick_t nowNt = getTimeNowNt(); efitick_t nowNt = getTimeNowNt();
Engine *engine = &this->engine;
LogTriggerTooth(SHAFT_PRIMARY_RISING, nowNt); LogTriggerTooth(SHAFT_PRIMARY_RISING, nowNt);
handleShaftSignal(0, true, nowNt); handleShaftSignal(0, true, nowNt);
} }
void EngineTestHelper::firePrimaryTriggerFall() { void EngineTestHelper::firePrimaryTriggerFall() {
efitick_t nowNt = getTimeNowNt(); efitick_t nowNt = getTimeNowNt();
Engine *engine = &this->engine;
LogTriggerTooth(SHAFT_PRIMARY_FALLING, nowNt); LogTriggerTooth(SHAFT_PRIMARY_FALLING, nowNt);
handleShaftSignal(0, false, nowNt); handleShaftSignal(0, false, nowNt);
} }
@ -344,15 +342,11 @@ void EngineTestHelper::applyTriggerWaveform() {
// todo: open question if this is worth a helper method or should be inlined? // todo: open question if this is worth a helper method or should be inlined?
void EngineTestHelper::assertRpm(int expectedRpm, const char *msg) { void EngineTestHelper::assertRpm(int expectedRpm, const char *msg) {
Engine *engine = &this->engine;
EXPECT_EQ(expectedRpm, Sensor::getOrZero(SensorType::Rpm)) << msg; EXPECT_EQ(expectedRpm, Sensor::getOrZero(SensorType::Rpm)) << msg;
} }
void setupSimpleTestEngineWithMaf(EngineTestHelper *eth, injection_mode_e injectionMode, void setupSimpleTestEngineWithMaf(EngineTestHelper *eth, injection_mode_e injectionMode,
trigger_type_e trigger) { trigger_type_e trigger) {
Engine *engine = &eth->engine;
engineConfiguration->isIgnitionEnabled = false; // let's focus on injection engineConfiguration->isIgnitionEnabled = false; // let's focus on injection
engineConfiguration->cylindersCount = 4; engineConfiguration->cylindersCount = 4;
// a bit of flexibility - the mode may be changed by some tests // a bit of flexibility - the mode may be changed by some tests