fix: uds log less data when sending message (#565)

Author: yannpoupon

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pykiso"
-version = "1.6.2"
+version = "1.6.3"
 description = "Embedded integration testing framework."
 authors = ["Sebastian Fischer <sebastian.fischer@de.bosch.com>"]
 license = "Eclipse Public License - v 2.0"

src/pykiso/lib/auxiliaries/udsaux/common/uds_response.py

@@ -53,14 +53,28 @@ def __init__(
         self.resp_time = resp_time
         self.pending_resp_times = pending_resp_times
 
-    def __repr__(self):
-        if self.data:
-            if self.is_negative:
-                return f"NegativeUdsResponse({bytes(self.data).hex(sep=' ')}, nrc={self.nrc.name})"
-            return f"{self.__class__.__name__}({bytes(self.data).hex(sep=' ')})"
-        else:
+    def __repr__(self, truncate=False):
+        """Return a string representation of the UDS response.
+
+        :param truncate: If True, only show first 6 bytes of data
+        :return: String representation of UdsResponse
+        """
+        if not self.data:
             return f"{self.__class__.__name__} no data."
 
+        data_to_show = self.data[:6] if truncate else self.data
+
+        if self.is_negative:
+            return f"NegativeUdsResponse({bytes(data_to_show).hex(sep=' ')}, nrc={self.nrc.name})"
+        return f"{self.__class__.__name__}({bytes(data_to_show).hex(sep=' ')})"
+
+    def get_truncated_representation(self) -> str:
+        """Return a truncated string representation of the UDS response.
+
+        :return: Truncated string representation showing only first 6 bytes
+        """
+        return self.__repr__(truncate=True)
+
 
 class NegativeResponseCode(IntEnum):
     """NRC code response"""

src/pykiso/lib/auxiliaries/udsaux/uds_auxiliary.py

@@ -135,7 +135,7 @@ def send_uds(
             if log.isEnabledFor(logging.getLogger().level):
                 log.internal_info(
                     "UDS request to send '%s'",
-                    ["0x{:02X}".format(i) for i in msg_to_send],
+                    ["0x{:02X}".format(i) for i in msg_to_send[:6]],
                 )
             resp = self.uds_config.send(
                 msg_to_send,
@@ -157,14 +157,12 @@ def send_uds(
             resp_time=self.uds_config.last_resp_time,
             pending_resp_times=self.uds_config.last_pending_resp_times,
         )
-        log.internal_info("UDS response received %s", resp)
+        if log.isEnabledFor(logging.getLogger().level):
+            log.internal_info("UDS response received %s", resp.get_truncated_representation())
         return resp
 
     def send_uds_raw(
-        self,
-        msg_to_send: Union[bytes, List[int], tuple],
-        timeout_in_s: float = 6,
-        response_required: bool = True,
+        self, msg_to_send: Union[bytes, List[int], tuple], timeout_in_s: float = 6, response_required: bool = True
     ) -> Union[UdsResponse, bool]:
         """Send a UDS diagnostic request to the target ECU and check response.
         Deprecated alias of `send_uds` that returns `False` on error,
@@ -182,6 +180,7 @@ def send_uds_raw(
             not expected and the command is properly sent otherwise
             False
         """
+
         try:
             return True if (response := self.send_uds(msg_to_send, response_required)) is None else response
         except self.errors.ResponseNotReceivedError as exc: