FOME-Tech
/
fome-fw
mirror of https://github.com/FOME-Tech/fome-fw.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix sensor mocking bug (#956) 

* actually reset

* add test
This commit is contained in:
Matthew Kennedy 2019-09-27 17:21:50 -07:00 committed by rusefi
parent ba1d8eb512
commit 7989d59311
2 changed files with 26 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
firmware/controllers/sensors/sensor.cpp
View File

@ -35,6 +35,7 @@ bool Sensor::Register() {
auto &entry = s_sensorRegistry[i];
entry.sensor = nullptr;
entry.useMock = false;
entry.mockValue = 0.0f;
}
}

25
unit_tests/tests/sensor/mock_sensor.cpp
View File

@ -90,3 +90,28 @@ TEST_F(SensorMocking, ResetAll) {
EXPECT_FLOAT_EQ(result.Value, 46.0f);
}
}
TEST_F(SensorMocking, ResetWithResetRegistry) {
// Set a value on the "real" sensor
realSensor.set(46.0f);
// Now set a mock value
Sensor::setMockValue(SensorType::Clt, 33.0f);
// Expect to see the mock value
{
auto result = Sensor::get(SensorType::Clt);
EXPECT_TRUE(result.Valid);
EXPECT_FLOAT_EQ(result.Value, 33.0f);
}
// Reset registry - including mocks
Sensor::resetRegistry();
{
auto result = Sensor::get(SensorType::Clt);
EXPECT_TRUE(result.Valid);
EXPECT_FLOAT_EQ(result.Value, 46.0f);
}
}