Specify more types in base framer (#2005)

Co-authored-by: jan iversen <[email protected]>

pymodbus/framer/base.py

@@ -27,7 +27,7 @@ class ModbusFramer:
     def __init__(
         self,
         decoder: ClientDecoder | ServerDecoder,
-        client=None,
+        client,
     ) -> None:
         """Initialize a new instance of the framer.
 
@@ -140,10 +140,10 @@ def processIncomingPacket(self, data, callback, slave, **kwargs):
 
     def frameProcessIncomingPacket(
         self, _single, _callback, _slave, _tid=None, **kwargs
-    ):
+    ) -> None:
         """Process new packet pattern."""
 
-    def buildPacket(self, message):
+    def buildPacket(self, message) -> bytes:  # type:ignore[empty-body]
         """Create a ready to send modbus packet.
 
         :param message: The populated request/response to send

pymodbus/framer/rtu_framer.py

@@ -304,7 +304,7 @@ def _process(self, callback, error=False):
         if (result := self.decoder.decode(data)) is None:
             raise ModbusIOException("Unable to decode request")
         if error and result.function_code < 0x80:
-            raise InvalidMessageReceivedException(result)
+            raise InvalidMessageReceivedException(str(result))
         self.populateResult(result)
         self.advanceFrame()
         callback(result)  # defer or push to a thread?

pymodbus/framer/socket_framer.py

@@ -147,7 +147,7 @@ def _process(self, callback, tid, error=False):
             self.resetFrame()
             raise ModbusIOException("Unable to decode request")
         if error and result.function_code < 0x80:
-            raise InvalidMessageReceivedException(result)
+            raise InvalidMessageReceivedException(str(result))
         self.populateResult(result)
         self.advanceFrame()
         if tid and tid != result.transaction_id:

pymodbus/framer/tls_framer.py

@@ -108,7 +108,7 @@ def _process(self, callback, error=False):
         if (result := self.decoder.decode(data)) is None:
             raise ModbusIOException("Unable to decode request")
         if error and result.function_code < 0x80:
-            raise InvalidMessageReceivedException(result)
+            raise InvalidMessageReceivedException(str(result))
         self.populateResult(result)
         self.advanceFrame()
         callback(result)  # defer or push to a thread?