Fix enable_phase_switching_on_host_only config

- type seems mandatory for @DataClass
- added more tests

Author: stephanme

pvcontrol/chargecontroller.py

@@ -54,7 +54,9 @@ class ChargeControllerData(BaseData):
 class ChargeControllerConfig(BaseConfig):
     cycle_time: int = 30  # [s] control loop cycle time, used by scheduler
     enable_phase_switching: bool = True  # set to False of phase relay is not in operation
-    enable_phase_switching_on_host_only = ""  # if set, phase switching is only allowed when running on specified host (env var: HOSTNAME)
+    enable_phase_switching_on_host_only: str = (
+        ""  # if set, phase switching is only allowed when running on specified host (env var: HOSTNAME)
+    )
     enable_auto_phase_switching: bool = True  # automatic phase switching depending on available PV
     enable_charging_when_connecting_car: ChargeMode = ChargeMode.OFF
     line_voltage: float = 230  # [V]

tests/test_chargecontroller.py

@@ -296,6 +296,29 @@ def test_1P(self):
         self.assertEqual(1, self.wallbox.get_data().phases_in)
 
 
+class ChargeControllerDisabledPhaseSwitchingHostnameTest(unittest.TestCase):
+    def setUp(self) -> None:
+        self.wallbox = SimulatedWallbox(WallboxConfig())
+        self.meter = TestMeter(self.wallbox)
+        reset_controller_metrics()
+
+    def test_matching_hostname(self):
+        self.controller = ChargeController(
+            ChargeControllerConfig(enable_phase_switching_on_host_only="pi1"), self.meter, self.wallbox, "pi1"
+        )
+        self.assertTrue(self.controller._enable_phase_switching)
+        self.controller.run()  # init
+        self.assertEqual(PhaseMode.AUTO, self.controller.get_data().phase_mode)
+
+    def test_wrong_hostname(self):
+        self.controller = ChargeController(
+            ChargeControllerConfig(enable_phase_switching_on_host_only="pi1"), self.meter, self.wallbox, "pi2"
+        )
+        self.assertFalse(self.controller._enable_phase_switching)
+        self.controller.run()  # init
+        self.assertEqual(PhaseMode.DISABLED, self.controller.get_data().phase_mode)
+
+
 class ChargeControllerManualModeTest(unittest.TestCase):
     def setUp(self) -> None:
         self.wallbox = SimulatedWallbox(WallboxConfig())