Address pre-commit findings concerning the use of tuple vs Tuple and List vs list in type hings, favoring list and tuple.

Author: vkruglik-aka

src/xdist/scheduler/isoscope.py

@@ -144,7 +144,7 @@ def __init__(self, config: pytest.Config, log: xdist.remote.Producer):
         # is performed once the number of registered node collections reaches
         # `_expected_num_workers`. It is initialized to None and then updated
         # after validation succeeds.
-        self._official_test_collection: Tuple[str, ...] | None = None
+        self._official_test_collection: tuple[str, ...] | None = None
         # Remote worker node having `_official_test_collection` as its test
         # collection (for reporting failed collection validations)
         self._official_test_collection_node: WorkerController | None = None
@@ -184,7 +184,7 @@ def __init__(self, config: pytest.Config, log: xdist.remote.Producer):
         )
 
     @property
-    def nodes(self) -> List[WorkerController]:
+    def nodes(self) -> list[WorkerController]:
         """A new list of all active `WorkerController` nodes.
 
         Called by xdist `DSession`.
@@ -536,7 +536,7 @@ def _reschedule_workers(self) -> None:
         """Distribute work to workers if needed at this time."""
         assert self._state is not None
 
-        traversed_states: List[IsoScopeScheduling._State] = []
+        traversed_states: list[IsoScopeScheduling._State] = []
         previous_state = None
         while self._state != previous_state:
             # NOTE: This loop will terminate because completion of tests and
@@ -764,7 +764,7 @@ def _handle_state_fence(self) -> None:
         self._log(f"Transitioned from {previous_state!s} to " f"{self._state!s}")
 
     def _distribute_workset(
-        self, workset: _ScopeWorkset, workers: List[_WorkerProxy]
+        self, workset: _ScopeWorkset, workers: list[_WorkerProxy]
     ) -> None:
         """Distribute the tests in the given workset to the given workers.
 
@@ -979,7 +979,7 @@ def _get_max_workers_for_num_tests(num_tests: int) -> int:
 
     def _get_workers_available_for_distribution(
         self, scope_id: str
-    ) -> List[_WorkerProxy]:
+    ) -> list[_WorkerProxy]:
         """Return workers available for distribution of the given Scope.
 
         Available workers are non-shutting-down workers that either
@@ -1003,7 +1003,7 @@ def _get_workers_available_for_distribution(
             )
         ]
 
-    def _get_workers_ready_for_fencing(self, scope_id: str) -> List[_WorkerProxy]:
+    def _get_workers_ready_for_fencing(self, scope_id: str) -> list[_WorkerProxy]:
         """Return workers that are ready to be Fenced for the given test Scope.
 
         A worker that needs to be Fenced satisfies all the following conditions:
@@ -1028,9 +1028,9 @@ def _get_workers_ready_for_fencing(self, scope_id: str) -> List[_WorkerProxy]:
     def _do_two_nodes_have_same_collection(
         self,
         reference_node: WorkerController,
-        reference_collection: Tuple[str, ...],
+        reference_collection: tuple[str, ...],
         node: WorkerController,
-        collection: Tuple[str, ...],
+        collection: tuple[str, ...],
     ) -> bool:
         """
         If collections differ, this method returns False while logging
@@ -1081,7 +1081,7 @@ def __init__(self, node: WorkerController):
 
         # An ordered collection of test IDs collected by the remote worker.
         # Initially None, until assigned by the Scheduler
-        self._collection: Tuple[str, ...] | None = None
+        self._collection: tuple[str, ...] | None = None
 
         self._pending_test_by_index: OrderedDict[int, _TestProxy] = OrderedDict()
 
@@ -1094,15 +1094,15 @@ def node(self) -> WorkerController:
         return self._node
 
     @property
-    def collection(self) -> Tuple[str, ...] | None:
+    def collection(self) -> tuple[str, ...] | None:
         """
         :return: An ordered collection of test IDs collected by the remote
             worker; `None` if the collection is not available yet.
         """
         return self._collection
 
     @collection.setter
-    def collection(self, collection: Tuple[str, ...]) -> None:
+    def collection(self, collection: tuple[str, ...]) -> None:
         """
         :param collection: An ordered collection of test IDs collected by the
             remote worker. Must not be `None`. Also, MUST NOT be set already.
@@ -1211,7 +1211,7 @@ def handle_test_completion(self, test_index: int) -> None:
         # Remove the test from the worker's pending queue
         self._pending_test_by_index.pop(test_index)
 
-    def release_pending_tests(self) -> List[_TestProxy]:
+    def release_pending_tests(self) -> list[_TestProxy]:
         """Reset the worker's pending tests, returning those pending tests.
 
         :return: A (possibly empty) list of pending tests.
@@ -1316,7 +1316,7 @@ def enqueue_test(self, test: _TestProxy) -> None:
         # Update high watermark
         self._high_water = max(self._high_water, len(self._test_by_index))
 
-    def dequeue_tests(self, num_tests: int) -> List[_TestProxy]:
+    def dequeue_tests(self, num_tests: int) -> list[_TestProxy]:
         """
         Remove and return the given number of tests from the head of the
         collection.