Move BusState conversion to util.py (#1956)

Author: Greg Brooks

can/interfaces/pcan/pcan.py

@@ -268,12 +268,6 @@ def __init__(
 
         self.check_api_version()
 
-        if not isinstance(state, BusState):
-            try:
-                state = BusState[state]
-            except KeyError as e:
-                raise ValueError("State must be ACTIVE or PASSIVE") from e
-
         if state in [BusState.ACTIVE, BusState.PASSIVE]:
             self.state = state
         else:

can/util.py

@@ -249,6 +249,12 @@ def _create_bus_config(config: dict[str, Any]) -> typechecking.BusConfig:
     if "fd" in config:
         config["fd"] = config["fd"] not in (0, False)
 
+    if "state" in config:
+        try:
+            config["state"] = can.BusState[config["state"]]
+        except KeyError as e:
+            raise ValueError("State config not valid!") from e
+
     return cast("typechecking.BusConfig", config)