cov.

Author: janiversen

test/client/test_client.py

@@ -307,6 +307,12 @@ def test_client_mixin_convert_datatype_fail(self):
         with pytest.raises(TypeError):
             ModbusClientMixin.convert_from_registers([123], ("d", 4))  # type: ignore[arg-type]
 
+    def test_client_mixin_execute(self):
+        """Test mixin execute."""
+        a = ModbusClientMixin()
+        a.execute(False, cast(ModbusPDU, None))
+
+
 
 class TestClientBase:
     """Test client code."""

test/datastore/test_sequential.py

@@ -28,4 +28,5 @@ def test_datastore_Sequential_set(self):
     def test_datastore_Sequential_iter(self):
         """Test check frame."""
         block = ModbusSequentialDataBlock(0x01, [17])
+        str(block)
         _ = list(block)

test/pdu/test_pdu.py

@@ -164,6 +164,13 @@ def test_pdu_instance(self, pdutype):
         pdu = pdutype()
         assert pdu
 
+    def test_pdu_base_instance(self):
+        """Test that all PDU types can be created."""
+        pdu = ModbusPDU(4)
+        str(pdu)
+        assert pdu
+        assert pdu.encode() == b''
+
     @pytest.mark.parametrize(("pdutype", "args", "kwargs", "frame"), requests + responses)
     @pytest.mark.usefixtures("frame", "args")
     def test_pdu_instance_args(self, pdutype, kwargs):

test/server/test_server_asyncio.py

@@ -54,9 +54,8 @@ def connection_made(self, transport):
         if BasicClient.data is not None:
             _logger.debug("TEST Client write data")
             self.transport.write(BasicClient.data)  # type: ignore[union-attr]
-        if BasicClient.dataTo is not None:
-            _logger.debug("TEST Client sendTo data")
-            self.transport.sendto(BasicClient.dataTo)  # type: ignore[union-attr]
+        _logger.debug("TEST Client sendTo data")
+        self.transport.sendto(BasicClient.dataTo)  # type: ignore[union-attr]
 
     def data_received(self, data):
         """Get Data received."""

test/server/test_simulator.py

@@ -10,7 +10,7 @@
 from pymodbus.client import AsyncModbusTcpClient
 from pymodbus.datastore.simulator import Cell, CellType
 from pymodbus.server import ModbusSimulatorServer
-from pymodbus.server.simulator.main import run_main
+from pymodbus.server.simulator.main import get_commandline, run_main
 from pymodbus.transport import NULLMODEM_HOST
 
 
@@ -235,6 +235,11 @@ async def setup_simulator_server(self, server, device, unused_tcp_port, only_obj
                 await task.stop()
                 await task_future
 
+    def test_simulator_commandline(self):
+        """Test commandline."""
+        assert get_commandline()
+        assert get_commandline(["--json_file", "got it"])
+
     async def test_simulator_server_tcp(self, simulator_server):
         """Test init simulator server."""