Line | Branch | Decision | Exec | Source |
---|---|---|---|---|
1 | #include "pch.h" | |||
2 | ||||
3 | #include "fuel_math.h" | |||
4 | #include "limp_manager.h" | |||
5 | ||||
6 | 4 | TEST(limp, testFatalError) { | ||
7 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
8 | 1 | LimpManager dut; | ||
9 | ||||
10 | // Everything should work by default | |||
11 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowElectronicThrottle()); | |
12 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowIgnition()); | |
13 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
14 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowTriggerInput()); | |
15 | ||||
16 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.fatalError(); | |
17 | ||||
18 | // Fatal error should kill everything | |||
19 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_FALSE(dut.allowElectronicThrottle()); | |
20 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowIgnition()); | |
21 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
22 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_FALSE(dut.allowTriggerInput()); | |
23 | 1 | } | ||
24 | ||||
25 | 4 | TEST(limp, revLimit) { | ||
26 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
27 | ||||
28 | 1 | engineConfiguration->rpmHardLimit = 2500; | ||
29 | 1 | engineConfiguration->cutFuelOnHardLimit = true; | ||
30 | 1 | engineConfiguration->cutSparkOnHardLimit = true; | ||
31 | ||||
32 | 1 | LimpManager dut; | ||
33 | ||||
34 | // Under rev limit, inj/ign allowed | |||
35 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(2000, 0); | |
36 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
37 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
38 | ||||
39 | // Over rev limit, no injection | |||
40 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(3000, 0); | |
41 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowIgnition()); | |
42 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
43 | ||||
44 | // Now recover back to under limit | |||
45 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(2000, 0); | |
46 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
47 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
48 | 2 | } | ||
49 | ||||
50 | 4 | TEST(limp, revLimitCltBased) { | ||
51 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
52 | ||||
53 | 1 | engineConfiguration->rpmHardLimit = 2500; | ||
54 | 1 | engineConfiguration->cutFuelOnHardLimit = true; | ||
55 | 1 | engineConfiguration->cutSparkOnHardLimit = true; | ||
56 | ||||
57 | // Configure CLT-based rev limit curve | |||
58 | 1 | engineConfiguration->useCltBasedRpmLimit = true; | ||
59 |
1/1✓ Branch 2 taken 1 time.
|
1 | copyArray(config->cltRevLimitRpmBins, { 10, 20, 30, 40 }); | |
60 | 1 | copyArray(config->cltRevLimitRpm, { 1000, 2000, 3000, 4000 }); | ||
61 | ||||
62 | 1 | LimpManager dut; | ||
63 | ||||
64 | // Check low temperature first | |||
65 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Clt, 10); | |
66 | ||||
67 | // Under rev limit, inj/ign allowed | |||
68 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(900, 0); | |
69 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
70 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
71 | ||||
72 | // Over rev limit, no injection | |||
73 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1100, 0); | |
74 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowIgnition()); | |
75 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
76 | ||||
77 | // Now recover back to under limit | |||
78 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(900, 0); | |
79 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
80 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
81 | ||||
82 | ||||
83 | // Check middle temperature | |||
84 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Clt, 35); | |
85 | ||||
86 | // Under rev limit, inj/ign allowed | |||
87 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(3400, 0); | |
88 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
89 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
90 | ||||
91 | // Over rev limit, no injection | |||
92 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(3600, 0); | |
93 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowIgnition()); | |
94 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
95 | ||||
96 | // Now recover back to under limit | |||
97 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(3400, 0); | |
98 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
99 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
100 | 2 | } | ||
101 | ||||
102 | 4 | TEST(limp, revHardLimitHyst) { | ||
103 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
104 | ||||
105 | 1 | engineConfiguration->rpmHardLimit = 2500; | ||
106 | 1 | engineConfiguration->rpmHardLimitHyst = 200; | ||
107 | 1 | engineConfiguration->cutSparkOnHardLimit = true; | ||
108 | ||||
109 | 1 | LimpManager dut; | ||
110 | ||||
111 | // Under rev limit, inj/ign allowed | |||
112 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(2500, 0); | |
113 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
114 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
115 | ||||
116 | // Over rev limit, no injection or ignition | |||
117 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(2501, 0); | |
118 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowIgnition()); | |
119 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
120 | ||||
121 | // Now set back inside the limit window - still not allowed | |||
122 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(2300, 0); | |
123 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowIgnition()); | |
124 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
125 | ||||
126 | // Now recover back to under lower limit | |||
127 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(2299, 0); | |
128 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowIgnition()); | |
129 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
130 | 2 | } | ||
131 | ||||
132 | 4 | TEST(limp, revSoftLimit) { | ||
133 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
134 | ||||
135 | 1 | engineConfiguration->rpmHardLimit = 2500; | ||
136 | 1 | engineConfiguration->rpmHardLimitHyst = 200; | ||
137 | 1 | engineConfiguration->rpmSoftLimitTimingRetard = 10; // 10 deg | ||
138 | 1 | engineConfiguration->rpmSoftLimitFuelAdded = 20; // 20% | ||
139 | 1 | engineConfiguration->cutSparkOnHardLimit = true; | ||
140 | ||||
141 |
1/1✓ Branch 1 taken 1 time.
|
1 | eth.engine.updateSlowSensors(); | |
142 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Clt, 36.605f); | |
143 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Iat, 30.0f); | |
144 | ||||
145 | // this is 5ms base fuel with some default CLT/IAT corrections | |||
146 | static const float baseFuel = 5.0f; | |||
147 | static const float normalRunningFuel = 5.3679f; | |||
148 | ||||
149 | // Under rev limit, no inj/ign corrections | |||
150 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Rpm, 2300); | |
151 |
1/1✓ Branch 1 taken 1 time.
|
1 | eth.engine.periodicFastCallback(); | |
152 | ||||
153 |
4/8✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
|
1 | EXPECT_FLOAT_EQ(0, getLimpManager()->getLimitingTimingRetard()); | |
154 |
2/6✓ Branch 4 taken 1 time.
✗ Branch 7 not taken.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✗ Branch 16 not taken.
✗ Branch 19 not taken.
|
1 | EXPECT_FLOAT_EQ((float)eth.engine.ignitionState.correctedIgnitionAdvance - (float)eth.engine.ignitionState.baseIgnitionAdvance, 0); | |
155 |
4/8✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
|
1 | EXPECT_FLOAT_EQ(1, getLimpManager()->getLimitingFuelCorrection()); | |
156 | // this is normal injection mode, no limiting fuel corrections | |||
157 |
4/10✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
✓ Branch 30 taken 1 time.
✗ Branch 31 not taken.
|
1 | ASSERT_NEAR(normalRunningFuel, getRunningFuel(baseFuel), EPS4D) << "base fuel"; | |
158 | ||||
159 | // For upper rev limit, we expect maximum inj/ign corrections | |||
160 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Rpm, 2500); | |
161 |
1/1✓ Branch 1 taken 1 time.
|
1 | eth.engine.periodicFastCallback(); | |
162 | ||||
163 |
4/8✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
|
1 | EXPECT_FLOAT_EQ(10, getLimpManager()->getLimitingTimingRetard()); // 10 deg | |
164 |
2/6✓ Branch 4 taken 1 time.
✗ Branch 7 not taken.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✗ Branch 16 not taken.
✗ Branch 19 not taken.
|
1 | EXPECT_FLOAT_EQ((float)eth.engine.ignitionState.correctedIgnitionAdvance - (float)eth.engine.ignitionState.baseIgnitionAdvance, -10.0f); | |
165 |
4/8✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
|
1 | EXPECT_FLOAT_EQ(1.2f, getLimpManager()->getLimitingFuelCorrection()); // 20% | |
166 |
4/10✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
✓ Branch 30 taken 1 time.
✗ Branch 31 not taken.
|
1 | ASSERT_NEAR(normalRunningFuel * 1.2f, getRunningFuel(baseFuel), EPS4D) << "base fuel"; // 20% | |
167 | ||||
168 | // In the middle of the limit window, we expect 50% interpolated inj/ign corrections | |||
169 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Rpm, 2400); | |
170 |
1/1✓ Branch 1 taken 1 time.
|
1 | eth.engine.periodicFastCallback(); | |
171 |
4/8✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
|
1 | EXPECT_FLOAT_EQ(5, getLimpManager()->getLimitingTimingRetard()); // 5 deg | |
172 |
2/6✓ Branch 4 taken 1 time.
✗ Branch 7 not taken.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✗ Branch 16 not taken.
✗ Branch 19 not taken.
|
1 | EXPECT_FLOAT_EQ((float)eth.engine.ignitionState.correctedIgnitionAdvance - (float)eth.engine.ignitionState.baseIgnitionAdvance, -5.0f); | |
173 |
4/8✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✓ Branch 8 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
|
1 | EXPECT_FLOAT_EQ(1.1f, getLimpManager()->getLimitingFuelCorrection()); // 10% | |
174 |
4/10✓ Branch 2 taken 1 time.
✓ Branch 5 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 23 not taken.
✓ Branch 30 taken 1 time.
✗ Branch 31 not taken.
|
1 | ASSERT_NEAR(normalRunningFuel * 1.1f, getRunningFuel(baseFuel), EPS4D) << "base fuel"; // 10% | |
175 | 1 | } | ||
176 | ||||
177 | 4 | TEST(limp, boostCut) { | ||
178 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
179 | ||||
180 | // Cut above 100kPa | |||
181 | 1 | engineConfiguration->boostCutPressure = 100; | ||
182 | 1 | engineConfiguration->boostCutPressureHyst = 20; | ||
183 | ||||
184 | 1 | LimpManager dut; | ||
185 | ||||
186 | // Below threshold, injection allowed | |||
187 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 80); | |
188 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
189 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
190 | ||||
191 | // Above rising threshold, injection cut | |||
192 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 105); | |
193 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
194 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
195 | ||||
196 | // Below rising threshold, but should have hysteresis, so not cut yet | |||
197 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 95); | |
198 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
199 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
200 | ||||
201 | // Below falling threshold, fuel restored | |||
202 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 79); | |
203 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
204 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
205 | ||||
206 | // SPECIAL CASE: threshold of 0 means never boost cut | |||
207 | 1 | engineConfiguration->boostCutPressure = 0; | ||
208 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 500); | |
209 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
210 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
211 | 2 | } | ||
212 | ||||
213 | 4 | TEST(limp, boostCutUint8Overflow) { | ||
214 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
215 | ||||
216 | // Cut above 1500kPa | |||
217 | 1 | engineConfiguration->boostCutPressure = 1500; | ||
218 | 1 | engineConfiguration->boostCutPressureHyst = 20; | ||
219 | ||||
220 | 1 | LimpManager dut; | ||
221 | ||||
222 | // Below threshold, injection allowed | |||
223 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 80); | |
224 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
225 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
226 | ||||
227 | // Above rising threshold, injection cut | |||
228 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 1600); | |
229 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
230 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
231 | ||||
232 | // Below rising threshold, but should have hysteresis, so not cut yet | |||
233 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 1495); | |
234 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
235 |
2/7✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
|
1 | EXPECT_FALSE(dut.allowInjection()); | |
236 | ||||
237 | // Below falling threshold, fuel restored | |||
238 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::Map, 79); | |
239 |
1/1✓ Branch 1 taken 1 time.
|
1 | dut.updateState(1000, 0); | |
240 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
241 | 2 | } | ||
242 | ||||
243 | ||||
244 | 4 | TEST(limp, oilPressureStartupFailureCase) { | ||
245 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
246 | 1 | engineConfiguration->minOilPressureAfterStart = 200; | ||
247 | ||||
248 | 1 | LimpManager dut; | ||
249 | ||||
250 | // Low oil pressure! | |||
251 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 50); | |
252 | ||||
253 | // Start the engine | |||
254 |
1/1✓ Branch 1 taken 1 time.
|
1 | engine->rpmCalculator.setRpmValue(1000); | |
255 | ||||
256 | // update & check: injection should be allowed | |||
257 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
258 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
259 | ||||
260 | // 4.5 seconds later, should still be allowed (even though pressure is low) | |||
261 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(4.5e6); | |
262 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
263 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
264 | ||||
265 | // 1 second later (5.5 since start), injection should cut | |||
266 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(1.0e6); | |
267 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
268 |
3/9✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
✓ Branch 36 taken 1 time.
✗ Branch 37 not taken.
|
1 | ASSERT_FALSE(dut.allowInjection()); | |
269 | ||||
270 | // But then oil pressure arrives! | |||
271 | // Injection still isn't allowed, since now we're late. | |||
272 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 250); | |
273 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
274 |
3/9✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
✓ Branch 36 taken 1 time.
✗ Branch 37 not taken.
|
1 | ASSERT_FALSE(dut.allowInjection()); | |
275 | 1 | } | ||
276 | ||||
277 | 4 | TEST(limp, oilPressureStartupSuccessCase) { | ||
278 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
279 | 1 | engineConfiguration->minOilPressureAfterStart = 200; | ||
280 | ||||
281 | 1 | LimpManager dut; | ||
282 | ||||
283 | // Low oil pressure! | |||
284 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 50); | |
285 | ||||
286 | // Start the engine | |||
287 |
1/1✓ Branch 1 taken 1 time.
|
1 | engine->rpmCalculator.setRpmValue(1000); | |
288 | ||||
289 | // update & check: injection should be allowed | |||
290 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
291 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
292 | ||||
293 | // 4.5 seconds later, should still be allowed (even though pressure is low) | |||
294 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(4.5e6); | |
295 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
296 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
297 | ||||
298 | // But then oil pressure arrives! | |||
299 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 250); | |
300 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
301 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
302 | ||||
303 | // 1 second later (5.5 since start), injection should be allowed since we saw pressure before the timeout | |||
304 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(1.0e6); | |
305 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
306 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
307 | ||||
308 | // Later, we lose oil pressure, but engine should stay running | |||
309 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(10e6); | |
310 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 10); | |
311 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
312 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
313 | 1 | } | ||
314 | ||||
315 | 4 | TEST(limp, oilPressureRunning) { | ||
316 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
317 | 1 | engineConfiguration->enableOilPressureProtect = true; | ||
318 | 1 | engineConfiguration->minimumOilPressureTimeout = 1.0f; | ||
319 | 1 | setArrayValues(config->minimumOilPressureValues, 100); | ||
320 | ||||
321 | 1 | LimpManager dut; | ||
322 | ||||
323 | // Oil pressure starts OK | |||
324 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 110); | |
325 | ||||
326 | // Start the engine | |||
327 |
1/1✓ Branch 1 taken 1 time.
|
1 | engine->rpmCalculator.setRpmValue(1000); | |
328 | ||||
329 | // update & check: injection should be allowed | |||
330 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
331 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
332 | ||||
333 | // A long time later, everything should still be OK | |||
334 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(60e6); | |
335 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
336 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
337 | ||||
338 | // Now oil pressure drops below threshold | |||
339 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 90); | |
340 | ||||
341 | // 0.9 second later, injection should continue as timeout isn't hit yet | |||
342 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(0.9e6); | |
343 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
344 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
345 | ||||
346 | // 0.2 second later (1.1s since low pressure starts), injection should cut | |||
347 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(1.0e6); | |
348 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
349 |
3/9✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
✓ Branch 36 taken 1 time.
✗ Branch 37 not taken.
|
1 | ASSERT_FALSE(dut.allowInjection()); | |
350 | ||||
351 | // Oil pressure is restored, and fuel should be restored too | |||
352 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 110); | |
353 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
354 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
355 | 1 | } | ||
356 | ||||
357 | 4 | TEST(limp, oilPressureMaxLimit) { | ||
358 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
359 | ||||
360 | 1 | config->maximumOilPressureValues[0] = 200; // kPa | ||
361 | 1 | config->maximumOilPressureValues[1] = 200; | ||
362 | 1 | config->maximumOilPressureValues[2] = 800; | ||
363 | 1 | config->maximumOilPressureValues[3] = 800; | ||
364 |
1/1✓ Branch 1 taken 1 time.
|
1 | setRpmTableBin(config->maximumOilPressureBins); // 700 800 3800 7000 | |
365 | ||||
366 | 1 | engineConfiguration->maxOilPressureTimeout = 1.0f; // sec | ||
367 | ||||
368 | 1 | LimpManager dut; | ||
369 | ||||
370 | // Oil pressure starts OK | |||
371 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 300); | |
372 | ||||
373 | // Start the engine | |||
374 | 1 | float rpm = 4000; | ||
375 |
1/1✓ Branch 1 taken 1 time.
|
1 | engine->rpmCalculator.setRpmValue(rpm); | |
376 | ||||
377 | // update & check: injection should be allowed | |||
378 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(rpm, getTimeNowNt()); | |
379 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
380 | ||||
381 | // after 5 sec - still allowed | |||
382 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(5.0e6); | |
383 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(rpm, getTimeNowNt()); | |
384 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
385 | ||||
386 | // On lower RPMs oil pressure goes above threshold | |||
387 | 1 | rpm = 800; | ||
388 |
1/1✓ Branch 1 taken 1 time.
|
1 | engine->rpmCalculator.setRpmValue(rpm); | |
389 | ||||
390 | // 0.9 second later, injection should continue as timeout isn't hit yet | |||
391 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(0.9e6); | |
392 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(rpm, getTimeNowNt()); | |
393 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
394 | ||||
395 | // 0.2 second later (1.1s since non-valid pressure), injection should cut | |||
396 |
1/1✓ Branch 1 taken 1 time.
|
1 | advanceTimeUs(1.0e6); | |
397 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(rpm, getTimeNowNt()); | |
398 |
3/9✓ Branch 4 taken 1 time.
✗ Branch 11 not taken.
✓ Branch 12 taken 1 time.
✗ Branch 15 not taken.
✗ Branch 20 not taken.
✗ Branch 24 not taken.
✗ Branch 27 not taken.
✓ Branch 36 taken 1 time.
✗ Branch 37 not taken.
|
1 | ASSERT_FALSE(dut.allowInjection()); | |
399 | ||||
400 | // Oil pressure is restored, and fuel should be restored too | |||
401 |
1/1✓ Branch 1 taken 1 time.
|
1 | Sensor::setMockValue(SensorType::OilPressure, 110); | |
402 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(rpm, getTimeNowNt()); | |
403 |
3/9✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 33 taken 1 time.
✗ Branch 34 not taken.
|
1 | ASSERT_TRUE(dut.allowInjection()); | |
404 | 1 | } | ||
405 | ||||
406 | ||||
407 | 4 | TEST(limp, gdiFuelCut) { | ||
408 |
1/1✓ Branch 2 taken 1 time.
|
1 | EngineTestHelper eth(engine_type_e::TEST_ENGINE); | |
409 | ||||
410 | // configure GDI engine for tests: | |||
411 | 1 | engineConfiguration->hpfpCam = HPFP_CAM_NONE; | ||
412 | 1 | engineConfiguration->hpfpCamLobes = 4; | ||
413 | 1 | engineConfiguration->mc33_t_hold_tot = 3.0f; | ||
414 |
1/1✓ Branch 1 taken 1 time.
|
1 | engine->rpmCalculator.setRpmValue(1000); | |
415 | ||||
416 | // below limits: | |||
417 | 1 | engine->engineState.injectionDuration = 1.8; | ||
418 | ||||
419 | 1 | LimpManager dut; | ||
420 | ||||
421 | // update & check: injection should be allowed | |||
422 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
423 |
2/7✓ Branch 3 taken 1 time.
✗ Branch 8 not taken.
✓ Branch 9 taken 1 time.
✗ Branch 12 not taken.
✗ Branch 17 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
|
1 | EXPECT_TRUE(dut.allowInjection()); | |
424 | ||||
425 | 1 | engine->engineState.injectionDuration = 3.1f; | ||
426 | // update & check: injection should cut | |||
427 |
2/2✓ Branch 1 taken 1 time.
✓ Branch 4 taken 1 time.
|
1 | dut.updateState(1000, getTimeNowNt()); | |
428 |
4/9✓ Branch 3 taken 1 time.
✓ Branch 7 taken 1 time.
✗ Branch 12 not taken.
✓ Branch 13 taken 1 time.
✗ Branch 16 not taken.
✗ Branch 21 not taken.
✗ Branch 24 not taken.
✓ Branch 31 taken 1 time.
✗ Branch 32 not taken.
|
1 | ASSERT_EQ(ClearReason::GdiLimits, dut.allowInjection().reason); | |
429 | 1 | } | ||
430 | ||||
431 | struct Mockhpfp : public MockHpfpController { | |||
432 | bool isHpfpActive; | |||
433 | angle_t m_deadangle; | |||
434 | }; | |||
435 |