Skip to content

Commit 52dde11

Browse files
malysclaude
andcommitted
chore(evhardware): track master b4767da, charge-limit write held back
EVHardware marks SET_CHARGE_LIMIT writeProven = false: SWI68 reads the limit back as 5, a code, while the write sends a percent. The diagnostic test now uses SET_CHARGING_ENABLED for the vendor-service binding check and pins the limit as WRITE_UNPROVEN. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent ca4e73f commit 52dde11

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

‎app/src/test/java/com/evsuite/tasker/debug/DiagnosticsTest.kt‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ class DiagnosticsTest {
245245
val none = Diagnostics.actions(allowed, FirmwareGen.SWI68)
246246

247247
assertEquals(Diagnostics.Reason.NO_VENDOR_SERVICE, entry(none, "SET_CABIN_TEMP").reason)
248-
assertEquals(Diagnostics.Reason.NO_VENDOR_SERVICE, entry(none, "SET_CHARGE_LIMIT").reason)
248+
assertEquals(Diagnostics.Reason.NO_VENDOR_SERVICE, entry(none, "SET_CHARGING_ENABLED").reason)
249249
assertEquals(Diagnostics.Reason.NO_VENDOR_SERVICE, entry(none, "PLAY_RADIO").reason)
250250
assertEquals(Diagnostics.Reason.NO_VENDOR_SERVICE, entry(none, "CALL_NUMBER").reason)
251251

@@ -257,7 +257,9 @@ class DiagnosticsTest {
257257
FirmwareGen.SWI68
258258
)
259259
assertEquals(Diagnostics.Status.OK, entry(bound, "SET_CABIN_TEMP").status)
260-
assertEquals(Diagnostics.Status.OK, entry(bound, "SET_CHARGE_LIMIT").status)
260+
assertEquals(Diagnostics.Status.OK, entry(bound, "SET_CHARGING_ENABLED").status)
261+
// Bound or not, the limit stays out: SWI68 reads it back as a code, the write sends a percent.
262+
assertEquals(Diagnostics.Reason.WRITE_UNPROVEN, entry(bound, "SET_CHARGE_LIMIT").reason)
261263
assertEquals(Diagnostics.Status.OK, entry(bound, "PLAY_RADIO").status)
262264
assertEquals(Diagnostics.Status.OK, entry(bound, "CALL_NUMBER").status)
263265
}

0 commit comments

Comments
 (0)