Skip to content

Commit b671c5a

Browse files
Merge pull request #2914 from nicoddemus/addfinalizer-refactor
Make SubRequest.addfinalizer an explicit method
2 parents 742f9cb + f320686 commit b671c5a

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

_pytest/fixtures.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -575,7 +575,6 @@ def __init__(self, request, scope, param, param_index, fixturedef):
575575
self.param_index = param_index
576576
self.scope = scope
577577
self._fixturedef = fixturedef
578-
self.addfinalizer = fixturedef.addfinalizer
579578
self._pyfuncitem = request._pyfuncitem
580579
self._fixture_values = request._fixture_values
581580
self._fixture_defs = request._fixture_defs
@@ -586,6 +585,9 @@ def __init__(self, request, scope, param, param_index, fixturedef):
586585
def __repr__(self):
587586
return "<SubRequest %r for %r>" % (self.fixturename, self._pyfuncitem)
588587

588+
def addfinalizer(self, finalizer):
589+
self._fixturedef.addfinalizer(finalizer)
590+
589591

590592
class ScopeMismatchError(Exception):
591593
""" A fixture function tries to use a different fixture function which

0 commit comments

Comments
 (0)