Supress exception occuring in NIXNET _detect_available_configs() (hardbyte#1085)

felixdivo · mergify[bot] · web-flow · commit c64f16260465 · 2021-06-25T01:26:10.000+02:00
* Supress exception occuring in NIXNET _detect_available_configs()

* Format code with black

* fix error import

* Format code with black

Co-authored-by: mergify[bot] &lt;37929162+mergify[bot]@users.noreply.github.com&gt;
diff --git a/can/interfaces/nixnet.py b/can/interfaces/nixnet.py
@@ -19,7 +19,15 @@
 
 if sys.platform == "win32":
     try:
-        from nixnet import session, types, constants, errors, system, database
+        from nixnet import (
+            session,
+            types,
+            constants,
+            errors,
+            system,
+            database,
+            XnetError,
+        )
     except ImportError:
         logger.error("Error, NIXNET python module cannot be loaded.")
         raise ImportError()
@@ -225,18 +233,25 @@ def shutdown(self):
     @staticmethod
     def _detect_available_configs():
         configs = []
-        nixnet_system = system.System()
-        for can_intf in nixnet_system.intf_refs_can:
-            logger.info("Channel index %d: %s", can_intf.port_num, str(can_intf))
-            configs.append(
-                {
-                    "interface": "nixnet",
-                    "channel": str(can_intf),
-                    "can_term_available": can_intf.can_term_cap
-                    == constants.CanTermCap.YES,
-                }
-            )
-        nixnet_system.close()
+
+        try:
+            with system.System() as nixnet_system:
+                for interface in nixnet_system.intf_refs_can:
+                    cahnnel = str(interface)
+                    logger.debug(
+                        "Found channel index %d: %s", interface.port_num, cahnnel
+                    )
+                    configs.append(
+                        {
+                            "interface": "nixnet",
+                            "channel": cahnnel,
+                            "can_term_available": interface.can_term_cap
+                            == constants.CanTermCap.YES,
+                        }
+                    )
+        except XnetError as error:
+            logger.debug("An error occured while searching for configs: %s", str(error))
+
         return configs
 
 
