Apply review comment

and fix a couple of Mypy issues

Author: sadra-barikbin

src/_pytest/assertion/rewrite.py

@@ -17,6 +17,7 @@
 from pathlib import Path
 from pathlib import PurePath
 from typing import Callable
+from typing import DefaultDict
 from typing import Dict
 from typing import IO
 from typing import Iterable
@@ -668,9 +669,9 @@ def __init__(
         else:
             self.enable_assertion_pass_hook = False
         self.source = source
-        self.scope: tuple[ast.AST, ...] = ()
-        self.variables_overwrite: defaultdict[
-            tuple[ast.AST, ...], Dict[str, str]
+        self.scope: Tuple[ast.AST, ...] = ()
+        self.variables_overwrite: DefaultDict[
+            Tuple[ast.AST, ...], Dict[str, str]
         ] = defaultdict(dict)
 
     def run(self, mod: ast.Module) -> None:

src/_pytest/python.py

@@ -60,7 +60,6 @@
 from _pytest.deprecated import INSTANCE_COLLECTOR
 from _pytest.deprecated import NOSE_SUPPORT_METHOD
 from _pytest.fixtures import _get_direct_parametrize_args
-from _pytest.fixtures import FixtureDef
 from _pytest.fixtures import FuncFixtureInfo
 from _pytest.fixtures import get_scope_node
 from _pytest.fixtures import IdentityFixtureDef
@@ -1189,7 +1188,7 @@ def id(self) -> str:
 
 
 # Used for storing pseudo fixturedefs for direct parametrization.
-name2pseudofixturedef_key = StashKey[Dict[str, FixtureDef[Any]]]()
+name2pseudofixturedef_key = StashKey[Dict[str, IdentityFixtureDef[Any]]]()
 
 
 @final
@@ -1379,7 +1378,7 @@ def parametrize(
         if node is None:
             name2pseudofixturedef = None
         else:
-            default: Dict[str, FixtureDef[Any]] = {}
+            default: Dict[str, IdentityFixtureDef[Any]] = {}
             name2pseudofixturedef = node.stash.setdefault(
                 name2pseudofixturedef_key, default
             )