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

more ETB test coverage (#2039) 

* test no pedal map

* test open loop

* open loop but not throttle

Co-authored-by: Matthew Kennedy <makenne@microsoft.com>
This commit is contained in:
Matthew Kennedy 2020-12-06 22:59:18 -06:00 committed by GitHub
parent 67269aa637
commit efef2bfe4f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 46 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
unit_tests/tests/test_etb.cpp
View File

@ -295,6 +295,15 @@ TEST(etb, setpointIdle) {
EXPECT_FLOAT_EQ(55, etb.getSetpoint().value_or(-1));
}
TEST(etb, setpointNoPedalMap) {
EtbController etb;
// Don't pass a pedal map
etb.init(ETB_Throttle1, nullptr, nullptr, nullptr);
EXPECT_EQ(etb.getSetpoint(), unexpected);
}
TEST(etb, setpointIdleValveController) {
EtbController etb;
@ -488,3 +497,40 @@ TEST(etb, closedLoopPid) {
EXPECT_FLOAT_EQ(etb.getClosedLoop(50, 70).value_or(-1), -60);
EXPECT_FLOAT_EQ(etb.getClosedLoop(50, 30).value_or(-1), 75);
}
TEST(etb, openLoopThrottle) {
WITH_ENGINE_TEST_HELPER(TEST_ENGINE);
EtbController etb;
INJECT_ENGINE_REFERENCE(&etb);
etb.init(ETB_Throttle1, nullptr, nullptr, nullptr);
// Map [0, 100] -> [-50, 50]
setLinearCurve(engineConfiguration->etbBiasBins, 0, 100);
setLinearCurve(engineConfiguration->etbBiasValues, -50, 50);
EXPECT_NEAR(-50, etb.getOpenLoop(0).value_or(-1), EPS4D);
EXPECT_NEAR(-25, etb.getOpenLoop(25).value_or(-1), EPS4D);
EXPECT_NEAR(0, etb.getOpenLoop(50).value_or(-1), EPS4D);
EXPECT_NEAR(25, etb.getOpenLoop(75).value_or(-1), EPS4D);
EXPECT_NEAR(50, etb.getOpenLoop(100).value_or(-1), EPS4D);
}
TEST(etb, openLoopNonThrottle) {
WITH_ENGINE_TEST_HELPER(TEST_ENGINE);
EtbController etb;
INJECT_ENGINE_REFERENCE(&etb);
etb.init(ETB_Wastegate, nullptr, nullptr, nullptr);
// Map [0, 100] -> [-50, 50]
setLinearCurve(engineConfiguration->etbBiasBins, 0, 100);
setLinearCurve(engineConfiguration->etbBiasValues, -50, 50);
// Should all return 0, as non-throttles don't use open loop table
EXPECT_EQ(0, etb.getOpenLoop(0).value_or(-1));
EXPECT_EQ(0, etb.getOpenLoop(25).value_or(-1));
EXPECT_EQ(0, etb.getOpenLoop(50).value_or(-1));
EXPECT_EQ(0, etb.getOpenLoop(75).value_or(-1));
EXPECT_EQ(0, etb.getOpenLoop(100).value_or(-1));
}