Merge pull request #101 from espressif/fix/serial_port_occupied_when_init_failed

Fix/serial port occupied when init failed

Author: hfudev

pytest-embedded-serial/pytest_embedded_serial/serial.py

@@ -51,19 +51,23 @@ def __init__(self, pexpect_proc: PexpectProcess, port: Union[str, pyserial.Seria
             self.port = self.proc.port
 
         self.pexpect_proc = pexpect_proc
-        self.occupied_ports[self.port] = None
-        logging.debug(f'occupied {self.port}')
 
         self._post_init()
 
         self._start()
 
+        self._finalize_init()
+
     def _post_init(self):
         pass
 
     def _start(self):
         pass
 
+    def _finalize_init(self):
+        self.occupied_ports[self.port] = None
+        logging.debug(f'occupied {self.port}')
+
     def _forward_io(self, pexpect_proc: PexpectProcess) -> None:
         while self.proc.is_open:
             try:

pytest-embedded/pytest_embedded/plugin.py

@@ -336,13 +336,18 @@ def wrapper(*args, **kwargs):
         def _close_or_terminate(obj):
             try:
                 obj.close()
-            except OSError:
+            except OSError as e:
+                logging.debug(str(e))
                 pass
             except AttributeError:
                 try:
                     obj.terminate()
-                except AttributeError:
+                except AttributeError as e:
+                    logging.debug(str(e))
                     pass
+            except Exception as e:
+                logging.debug(str(e))
+                pass  # swallow up all error
             finally:
                 del obj
 

requirements.txt

@@ -1,5 +1,6 @@
-# This is an requirements.txt for extra dependencies.
+# This is a requirements.txt for extra dependencies.
 # To install all packages, please run `bash foreach.sh install`
+cryptography==37.0.2
 
 # testing
 pytest-cov