Eliminate implicit optional in simulator (#1841)

Author: alexrudd2

pymodbus/datastore/simulator.py

@@ -1,9 +1,11 @@
 """Pymodbus ModbusSimulatorContext."""
+from __future__ import annotations
+
 import dataclasses
 import random
 import struct
 from datetime import datetime
-from typing import Any, Callable, Dict, List
+from typing import Any, Callable
 
 
 WORD_SIZE = 16
@@ -30,7 +32,7 @@ class Cell:
     access: bool = False
     value: int = 0
     action: int = 0
-    action_kwargs: Dict[str, Any] = None
+    action_kwargs: dict[str, Any] | None = None
     count_read: int = 0
     count_write: int = 0
 
@@ -452,18 +454,18 @@ class ModbusSimulatorContext:
     start_time = int(datetime.now().timestamp())
 
     def __init__(
-        self, config: Dict[str, Any], custom_actions: Dict[str, Callable]
+        self, config: dict[str, Any], custom_actions: dict[str, Callable]
     ) -> None:
         """Initialize."""
-        self.registers: List[int] = []
-        self.fc_offset: Dict[int, int] = {}
+        self.registers: list[int] = []
+        self.fc_offset: dict[int, int] = {}
         self.register_count = 0
         self.type_exception = False
-        self.action_name_to_id: Dict[str, int] = {}
-        self.action_id_to_name: List[str] = []
-        self.action_methods: List[Callable] = []
-        self.registerType_name_to_id: Dict[str, int] = {}
-        self.registerType_id_to_name: List[str] = []
+        self.action_name_to_id: dict[str, int] = {}
+        self.action_id_to_name: list[str] = []
+        self.action_methods: list[Callable] = []
+        self.registerType_name_to_id: dict[str, int] = {}
+        self.registerType_id_to_name: list[str] = []
         Setup(self).setup(config, custom_actions)
 
     # --------------------------------------------

pymodbus/server/simulator/http_server.py

@@ -1,12 +1,13 @@
 """HTTP server for modbus simulator."""
+from __future__ import annotations
+
 import asyncio
 import contextlib
 import dataclasses
 import importlib
 import json
 import os
 from time import time
-from typing import List
 
 
 try:
@@ -118,7 +119,7 @@ def __init__(
         http_port: int = 8080,
         log_file: str = "server.log",
         json_file: str = "setup.json",
-        custom_actions_module: str = None,
+        custom_actions_module: str | None = None,
     ):
         """Initialize http interface."""
         if AIOHTTP_MISSING:
@@ -139,15 +140,15 @@ def __init__(
             actions_module = importlib.import_module(custom_actions_module)
             custom_actions_dict = actions_module.custom_actions_dict
         else:
-            custom_actions_dict = None
+            custom_actions_dict = {}
         server = setup["server_list"][modbus_server]
         if server["comm"] != "serial":
             server["address"] = (server["host"], server["port"])
             del server["host"]
             del server["port"]
         device = setup["device_list"][modbus_device]
         self.datastore_context = ModbusSimulatorContext(
-            device, custom_actions_dict or None
+            device, custom_actions_dict or {}
         )
         datastore = ModbusServerContext(slaves=self.datastore_context, single=True)
         comm = comm_class[server.pop("comm")]
@@ -200,8 +201,8 @@ def __init__(
             with open(html_file, encoding="utf-8") as handle:
                 self.generator_html[entry][0] = handle.read()
         self.refresh_rate = 0
-        self.register_filter: List[int] = []
-        self.call_list: List[tuple] = []
+        self.register_filter: list[int] = []
+        self.call_list: list[tuple] = []
         self.request_lookup = ServerDecoder.getFCdict()
         self.call_monitor = CallTypeMonitor()
         self.call_response = CallTypeResponse()