Spotted pyright fixes

src/trio/__init__.py

@@ -111,7 +111,8 @@
 
 # Not imported by default, but mentioned here so static analysis tools like
 # pylint will know that it exists.
-if False:
+from typing import TYPE_CHECKING
+if TYPE_CHECKING:
     from . import testing
 
 from . import _deprecate as _deprecate
@@ -154,3 +155,4 @@
 fixup_module_metadata(from_thread.__name__, from_thread.__dict__)
 fixup_module_metadata(to_thread.__name__, to_thread.__dict__)
 del fixup_module_metadata
+del TYPE_CHECKING

src/trio/_tests/test_exports.py

@@ -198,7 +198,8 @@ def no_underscores(symbols: Iterable[str]) -> set[str]:
             )
     elif tool == "pyright_verifytypes":
         if not RUN_SLOW:  # pragma: no cover
-            pytest.skip("use --run-slow to check against mypy")
+            pytest.skip("use --run-slow to check against pyright")
+
         try:
             import pyright  # noqa: F401
         except ImportError as error:
@@ -217,10 +218,6 @@ def no_underscores(symbols: Iterable[str]) -> set[str]:
             if x["name"].startswith(modname)
         }
 
-        # pyright ignores the symbol defined behind `if False`
-        if modname == "trio":
-            static_names.add("testing")
-
         # these are hidden behind `if sys.platform != "win32" or not TYPE_CHECKING`
         # so presumably pyright is parsing that if statement, in which case we don't
         # care about them being missing.