duplicate validation logic, bump pyright

Author: jakkdl

src/trio/_tests/test_timeouts.py

@@ -166,7 +166,7 @@ async def test_timeouts_raise_value_error() -> None:
     ):
         with pytest.raises(
             ValueError,
-            match="^(duration|deadline|timeout) must (not )*be (non-negative|NaN)$",
+            match="^(deadline|`seconds`) must (not )*be (non-negative|NaN)$",
         ):
             await fun(val)
 
@@ -180,7 +180,7 @@ async def test_timeouts_raise_value_error() -> None:
     ):
         with pytest.raises(
             ValueError,
-            match="^(duration|deadline|timeout|relative deadline) must (not )*be (non-negative|NaN)$",
+            match="^(deadline|`seconds`) must (not )*be (non-negative|NaN)$",
         ):
             with cm(val):
                 pass  # pragma: no cover

src/trio/_timeouts.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import math
 import sys
 from contextlib import contextmanager
 from typing import TYPE_CHECKING
@@ -23,6 +24,7 @@ def move_on_at(deadline: float, *, shield: bool = False) -> trio.CancelScope:
       ValueError: if deadline is NaN.
 
     """
+    # CancelScope validates that deadline isn't math.nan
     return trio.CancelScope(deadline=deadline, shield=shield)
 
 
@@ -42,9 +44,14 @@ def move_on_after(
           of the newly created cancel scope.
 
     Raises:
-      ValueError: if timeout is less than zero or NaN.
+      ValueError: if `seconds` is less than zero or NaN.
 
     """
+    # duplicate validation logic to have the correct parameter name
+    if seconds < 0:
+        raise ValueError("`seconds` must be non-negative")
+    if math.isnan(seconds):
+        raise ValueError("`seconds` must not be NaN")
     return trio.CancelScope(
         shield=shield,
         relative_deadline=seconds,
@@ -92,7 +99,7 @@ async def sleep(seconds: float) -> None:
 
     """
     if seconds < 0:
-        raise ValueError("duration must be non-negative")
+        raise ValueError("`seconds` must be non-negative")
     if seconds == 0:
         await trio.lowlevel.checkpoint()
     else:

test-requirements.txt

@@ -105,7 +105,7 @@ pylint==3.2.7
     # via -r test-requirements.in
 pyopenssl==24.2.1
     # via -r test-requirements.in
-pyright==1.1.378
+pyright==1.1.381
     # via -r test-requirements.in
 pytest==8.3.2
     # via -r test-requirements.in