From a3fbf2438975d72887ef5b614d529bab893937ca Mon Sep 17 00:00:00 2001 From: Ran Benita Date: Sat, 15 Jul 2023 21:35:05 +0300 Subject: [PATCH] unittest: hide access to _request Let the parent class (Function) do this. Tiny bit better abstraction. There seems to be no reason for the `hasattr` bit. --- src/_pytest/unittest.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/_pytest/unittest.py b/src/_pytest/unittest.py index 27598cbde..34845cec1 100644 --- a/src/_pytest/unittest.py +++ b/src/_pytest/unittest.py @@ -200,10 +200,10 @@ class TestCaseFunction(Function): assert self.parent is not None self._testcase = self.parent.obj(self.name) # type: ignore[attr-defined] self._obj = getattr(self._testcase, self.name) - if hasattr(self, "_request"): - self._request._fillfixtures() + super().setup() def teardown(self) -> None: + super().teardown() if self._explicit_tearDown is not None: self._explicit_tearDown() self._explicit_tearDown = None