Fix: definition and usage of fixture skip_nfi

Author: fangerer

graalpython/lib-graalpython/modules/hpy/test/conftest.py

@@ -59,7 +59,7 @@ def compiler(request, tmpdir, hpy_devel, hpy_abi, ExtensionTemplate):
                              ExtensionTemplate=ExtensionTemplate)
 
 @pytest.fixture()
-def skip_nfi(self, hpy_abi):
+def skip_nfi(hpy_abi):
     # skip all tests in this class for NFI mode
     if hpy_abi == 'nfi':
         pytest.skip()

graalpython/lib-graalpython/modules/hpy/test/test_hpytype_legacy.py

@@ -23,11 +23,12 @@
 
 """ HPyType tests on legacy types. """
 
+import pytest
+
 from .support import HPyTest
 from .test_hpytype import PointTemplate, TestType as _TestType
 
 
-@pytest.mark.usefixtures('skip_nfi')
 class LegacyPointTemplate(PointTemplate):
     """
     Override PointTemplate to instead define a legacy point type that

graalpython/lib-graalpython/modules/hpy/test/test_slots_legacy.py

@@ -22,6 +22,7 @@
 # SOFTWARE.
 
 """ HPyType slot tests on legacy types. """
+import pytest
 
 from .support import HPyTest
 from .test_hpytype_legacy import LegacyPointTemplate