Fix Custom mode for SWI68 SWI 165

Author: SliDeeN

app/src/main/java/com/mg4/control/model/CustomDriveScale.kt

@@ -10,9 +10,9 @@ package com.mg4.control.model
  *   la puissance y réutilise l'échelle des modes de conduite (Éco = 2) ;
  * - [Family.VSM_68] : méthodes `setElectricPowertrainLevel` / `setSteeringLevel` /
  *   `setBrakePedalLevel` du SDK SAIC, qui écrivent les propriétés `0x2140a18c` à `0x2140a18e`.
- *   Le service véhicule SWI68 borne ces trois valeurs à 0..3 ; l'échelle 1/2/3 retenue ici reste
- *   une hypothèse, que la sonde `MG4_CUSTOM` confirmera en voiture (surtout pour la pédale, qui
- *   vaut 1/0/2 sur les deux autres familles).
+ *   Échelles LUES dans l'écran d'origine (DrivingSettingsViewModel des Réglages véhicule SWI68 et
+ *   SWI165, position du sélecteur → valeur) : puissance et direction 1/2/3, pédale 1/0/2 — donc
+ *   les mêmes qu'en SWI133, seule la voie d'accès change.
  */
 object CustomDriveScale {
 
@@ -26,19 +26,18 @@ object CustomDriveScale {
         return when (setting) {
             Setting.POWER    -> index + if (family == Family.A9) 2 else 1
             Setting.STEERING -> index + 1
-            Setting.PEDAL    -> if (family == Family.VSM_68) index + 1 else PEDAL_VALUES[index]
+            Setting.PEDAL    -> PEDAL_VALUES[index]
         }
     }
 
     /** Index correspondant à une valeur lue, ou null si elle est hors échelle (équipement absent). */
     fun index(setting: Setting, value: Int, family: Family): Int? = when (setting) {
         Setting.POWER    -> (value - if (family == Family.A9) 2 else 1).takeIf { it in 0..2 }
         Setting.STEERING -> (value - 1).takeIf { it in 0..2 }
-        Setting.PEDAL    -> if (family == Family.VSM_68) (value - 1).takeIf { it in 0..2 }
-                            else PEDAL_VALUES.indexOf(value).takeIf { it >= 0 }
+        Setting.PEDAL    -> PEDAL_VALUES.indexOf(value).takeIf { it >= 0 }
     }
 
-    /** Pédale, familles VPM_133 et A9 : confort = 1, normal = 0, sport = 2. */
+    /** Pédale, les trois familles : confort = 1, normal = 0, sport = 2 (Normal vaut ZÉRO). */
     private val PEDAL_VALUES = intArrayOf(1, 0, 2)
 
     private fun IntArray.indexOf(value: Int): Int = indices.firstOrNull { this[it] == value } ?: -1

app/src/test/java/com/mg4/control/model/CustomDriveScaleTest.kt

@@ -17,7 +17,7 @@ class CustomDriveScaleTest {
     private val indexes = 0..2
 
     @Test
-    fun `puissance - ancien SDK 1-2-3, A9 2-3-4, SWI68 1-2-3`() {
+    fun `puissance - 1-2-3 partout sauf A9 en 2-3-4`() {
         assertEquals(listOf(1, 2, 3), indexes.map { CustomDriveScale.value(Setting.POWER, it, Family.VPM_133) })
         assertEquals(listOf(2, 3, 4), indexes.map { CustomDriveScale.value(Setting.POWER, it, Family.A9) })
         assertEquals(listOf(1, 2, 3), indexes.map { CustomDriveScale.value(Setting.POWER, it, Family.VSM_68) })
@@ -32,12 +32,12 @@ class CustomDriveScaleTest {
     }
 
     @Test
-    fun `pedale - Normal vaut ZERO sauf sur SWI68-165`() {
-        // Relevé dans le smali SWI133 et le dispatch A9 : confort=1, normal=0, sport=2.
-        assertEquals(listOf(1, 0, 2), indexes.map { CustomDriveScale.value(Setting.PEDAL, it, Family.VPM_133) })
-        assertEquals(listOf(1, 0, 2), indexes.map { CustomDriveScale.value(Setting.PEDAL, it, Family.A9) })
-        // SWI68/165 : service véhicule borné à 0..3 ; échelle supposée 1/2/3, à confirmer par la sonde.
-        assertEquals(listOf(1, 2, 3), indexes.map { CustomDriveScale.value(Setting.PEDAL, it, Family.VSM_68) })
+    fun `pedale - Normal vaut ZERO sur les trois familles`() {
+        // Relevé dans les écrans d'origine (SWI133, dispatch A9, DrivingSettingsViewModel SWI68/165).
+        Family.entries.forEach { family ->
+            assertEquals("famille $family", listOf(1, 0, 2),
+                indexes.map { CustomDriveScale.value(Setting.PEDAL, it, family) })
+        }
     }
 
     @Test
@@ -64,7 +64,7 @@ class CustomDriveScaleTest {
         assertNull(CustomDriveScale.index(Setting.POWER, -1, Family.VPM_133))
         assertNull(CustomDriveScale.index(Setting.POWER, 1, Family.A9))       // A9 commence à 2
         assertNull(CustomDriveScale.index(Setting.PEDAL, 3, Family.VPM_133))  // pédale : 1/0/2 seulement
-        assertNull(CustomDriveScale.index(Setting.PEDAL, 0, Family.VSM_68))   // SWI68 : 1/2/3
+        assertNull(CustomDriveScale.index(Setting.PEDAL, 4, Family.VSM_68))
         assertNull(CustomDriveScale.index(Setting.STEERING, 255, Family.VSM_68))
     }
 }