Fix client modbus function calls in remote by adding count as keyword argument (#2563)

Co-authored-by: phsete <philteb@gmx.de>

Author: 2 people

pymodbus/datastore/remote.py

@@ -74,16 +74,16 @@ def __build_mapping(self):
             params["slave"] = self.slave
         self.__get_callbacks = {
             "d": lambda a, c: self._client.read_discrete_inputs(
-                a, c, **params
+                a, count=c, **params
             ),
             "c": lambda a, c: self._client.read_coils(
-                a, c, **params
+                a, count=c, **params
             ),
             "h": lambda a, c: self._client.read_holding_registers(
-                a, c, **params
+                a, count=c, **params
             ),
             "i": lambda a, c: self._client.read_input_registers(
-                a, c, **params
+                a, count=c, **params
             ),
         }
         self.__set_callbacks = {

test/not_updated/test_remote_datastore.py

@@ -50,9 +50,11 @@ def test_remote_slave_get_values(self):
         """Test getting values from a remote slave context."""
         client = mock.MagicMock()
         pdu = ReadCoilsResponse(bits=[True] * 10)
-        client.read_coils = lambda a, b: pdu
-        client.read_input_registers = lambda a, b: ReadInputRegistersResponse(registers=[10] * 10)
-        client.read_holding_registers = lambda a, b: ExceptionResponse(0x15)
+        read_input_reg_res = ReadInputRegistersResponse(registers=[10] * 10)
+        exception_response = ExceptionResponse(0x15)
+        client.read_coils = lambda a, count=1: pdu
+        client.read_input_registers = lambda a, count=1: read_input_reg_res
+        client.read_holding_registers = lambda a, count=1: exception_response
 
         context = RemoteSlaveContext(client)
         context.validate(1, 0, 10)