PYTHON-5784 Fix update_interval type annotation to float

update_interval() stores into _interval (float) and is called with float
values, but was annotated new_interval: int. Correct the annotation on both
executors and use a small positive interval in the coverage test.

Author: aclark4life

pymongo/periodic_executor.py

@@ -87,7 +87,7 @@ def wake(self) -> None:
         """Execute the target function soon."""
         self._event = True
 
-    def update_interval(self, new_interval: int) -> None:
+    def update_interval(self, new_interval: float) -> None:
         self._interval = new_interval
 
     def skip_sleep(self) -> None:
@@ -217,7 +217,7 @@ def wake(self) -> None:
         """Execute the target function soon."""
         self._event = True
 
-    def update_interval(self, new_interval: int) -> None:
+    def update_interval(self, new_interval: float) -> None:
         self._interval = new_interval
 
     def skip_sleep(self) -> None:

test/asynchronous/test_periodic_executor.py

@@ -117,7 +117,7 @@ async def target():
             call_times.append(time.monotonic())
             if len(call_times) == 1:
                 # Shorten the interval from 30s so the next run happens promptly.
-                executor.update_interval(0)
+                executor.update_interval(0.05)
                 return True
             return False
 

test/test_periodic_executor.py

@@ -117,7 +117,7 @@ def target():
             call_times.append(time.monotonic())
             if len(call_times) == 1:
                 # Shorten the interval from 30s so the next run happens promptly.
-                executor.update_interval(0)
+                executor.update_interval(0.05)
                 return True
             return False