Apply review comment

This commit is contained in:
Sadra Barikbin 2023-09-05 19:48:40 +03:30
parent 120be26962
commit 5e02d9c064
2 changed files with 7 additions and 2 deletions

View File

@ -1079,6 +1079,8 @@ class IdentityFixture(FixtureDef[FixtureValue]):
fixturemanager: "FixtureManager", fixturemanager: "FixtureManager",
argname: str, argname: str,
scope: Union[Scope, _ScopeName, Callable[[str, Config], _ScopeName], None], scope: Union[Scope, _ScopeName, Callable[[str, Config], _ScopeName], None],
*,
_ispytest: bool = False,
): ):
super().__init__( super().__init__(
fixturemanager, fixturemanager,
@ -1087,7 +1089,7 @@ class IdentityFixture(FixtureDef[FixtureValue]):
lambda request: request.param, lambda request: request.param,
scope, scope,
None, None,
_ispytest=True, _ispytest=_ispytest,
) )

View File

@ -1376,7 +1376,10 @@ class Metafunc:
fixturedef = name2pseudofixturedef[argname] fixturedef = name2pseudofixturedef[argname]
else: else:
fixturedef = IdentityFixture( fixturedef = IdentityFixture(
self.definition.session._fixturemanager, argname, scope_ self.definition.session._fixturemanager,
argname,
scope_,
_ispytest=True,
) )
if name2pseudofixturedef is not None: if name2pseudofixturedef is not None:
name2pseudofixturedef[argname] = fixturedef name2pseudofixturedef[argname] = fixturedef