update config

Author: LKuemmel

packages/helpermodules/update_config.py

@@ -57,7 +57,7 @@
 
 class UpdateConfig:
 
-    DATASTORE_VERSION = 123
+    DATASTORE_VERSION = 124
 
     valid_topic = [
         "^openWB/bat/config/bat_control_activated$",
@@ -3120,3 +3120,15 @@ def upgrade(topic: str, payload) -> Optional[dict]:
                     return {topic: configuration_payload}
         self._loop_all_received_topics(upgrade)
         self._append_datastore_version(123)
+
+    def upgrade_datastore_124(self) -> None:
+        def upgrade(topic: str, payload) -> Optional[dict]:
+            if re.search("^openWB/vehicle/[0-9]+/soc_module/config$", topic) is not None:
+                configuration_payload = decode_payload(payload)
+                if configuration_payload.get("type") == "json":
+                    json_config = configuration_payload.get("configuration", {})
+                    if "odometer_pattern" not in json_config:
+                        json_config["odometer_pattern"] = None
+                        return {topic: configuration_payload}
+        self._loop_all_received_topics(upgrade)
+        self._append_datastore_version(124)

packages/helpermodules/update_config_test.py

@@ -58,6 +58,35 @@ def test_upgrade_datastore_94(index_test_template, expected_index):
     assert plan_ids == expected_index
 
 
+def test_upgrade_datastore_124_adds_missing_odometer_pattern_for_json_soc_module():
+    update_con = UpdateConfig()
+    update_con.all_received_topics = {
+        "openWB/system/datastore_version": list(range(124)),
+        "openWB/vehicle/0/soc_module/config": {
+            "type": "json",
+            "configuration": {
+                "url": "https://example.invalid/soc",
+                "soc_pattern": ".soc",
+            }
+        },
+        "openWB/vehicle/1/soc_module/config": {
+            "type": "homeassistant",
+            "configuration": {
+                "url": "http://ha.local"
+            }
+        }
+    }
+
+    update_con.upgrade_datastore_124()
+
+    json_config = update_con.all_received_topics["openWB/vehicle/0/soc_module/config"]["configuration"]
+    assert "odometer_pattern" in json_config
+    assert json_config["odometer_pattern"] is None
+
+    ha_config = update_con.all_received_topics["openWB/vehicle/1/soc_module/config"]["configuration"]
+    assert "odometer_pattern" not in ha_config
+
+
 @pytest.mark.parametrize("name", [
     "happy_path",
     "missing_prices_dict",