tests: drivers: uart: Fix serial connection errors in uart passtrough test

Enable HW flow control
Do not break on first error
Show send/receive log

Signed-off-by: Bartosz Miller <[email protected]>

Author: nordic-bami

tests/drivers/uart/uart_passtrough/boards/nrf5340dk_nrf5340_cpuapp.overlay

@@ -16,3 +16,7 @@
 	status = "okay";
 	hw-flow-control;
 };
+
+&uart0 {
+	hw-flow-control;
+};

tests/drivers/uart/uart_passtrough/boards/nrf54h20dk_nrf54h20_cpuapp.overlay

@@ -17,3 +17,7 @@
 	memory-regions = <&cpuapp_dma_region>;
 	hw-flow-control;
 };
+
+&uart136 {
+	hw-flow-control;
+};

tests/drivers/uart/uart_passtrough/pytest/serial_port.py

@@ -44,24 +44,22 @@ def _instanitate_serial_port(self) -> serial.Serial:
             logger.error(f"Opening '{self.port_name}' failed with error: {exc}!")
             raise exc
 
-    def close(
-        self,
-    ):
+    def close(self):
         """
         Close serial port
         """
         if self.serial_port:
-            logger.debug("Closing serial port connection")
+            logger.info("Closing serial port connection")
             self.serial_port.close()
-            logger.debug("Serial port connection closed")
+            logger.info("Serial port connection closed")
             self.serial_port = None
 
     def open(self) -> None:
         """
         Open serial port
         """
         if not self.serial_port.is_open:
-            logger.debug(f"Opening: '{self}'")
+            logger.info(f"Opening: '{self}'")
             self.serial_port.open()
             self.serial_port.reset_input_buffer()
             self.serial_port.reset_output_buffer()
@@ -79,7 +77,7 @@ def send(
         try:
             time.sleep(0.25)
             self.serial_port.write_timeout = self._timeout
-            logger.debug(f"[{self.serial_port.port}] Serial --> {message}")
+            logger.info(f"[{self.serial_port.port}] Serial --> {message}")
             line_termination: str = "\n" if add_line_termination else ""
             self.serial_port.write((message + line_termination).encode())
             if get_response:
@@ -90,7 +88,7 @@ def send(
                 while (self.serial_port.in_waiting > 0) and (time.time() - start < self._timeout):
                     time.sleep(0.1)
                     response += self.serial_port.read_all().decode()
-                logger.debug(f"Serial <-- {response}")
+                logger.info(f"Serial <-- {response}")
             return response
 
         except serial.SerialTimeoutException as exc:
@@ -114,8 +112,8 @@ def read_data_until_matched(self, message_to_find: str) -> bool:
         while time.perf_counter() - start < self._timeout:
             try:
                 if self.serial_port.in_waiting > 0:
-                    data += self.serial_port.read_all().decode()
-                    logger.debug(data)
+                    data += self.serial_port.read_all().decode(errors="replace")
+                    logger.info(data)
                     if message_to_find in data:
                         return True
                 time.sleep(0.1)

tests/drivers/uart/uart_passtrough/pytest/test_passtrough.py

@@ -62,13 +62,9 @@ def test_uart_passtrough(dut: DeviceAdapter):
 
     console_serial_port.open()
     passtrough_serial_port.open()
-    send_with_dead_time_between_characters(
-        console_serial_port.send, first_message, DEAD_TIME_S
-    )
+    send_with_dead_time_between_characters(console_serial_port.send, first_message, DEAD_TIME_S)
     first_search_result: bool = passtrough_serial_port.read_data_until_matched(first_message)
-    send_with_dead_time_between_characters(
-        passtrough_serial_port.send, second_message, DEAD_TIME_S
-    )
+    send_with_dead_time_between_characters(passtrough_serial_port.send, second_message, DEAD_TIME_S)
     second_search_result: bool = console_serial_port.read_data_until_matched(second_message)
     passtrough_serial_port.close()
     console_serial_port.close()