test: disable Python devmode in pytest

Author: XuehaiPan

tests/CMakeLists.txt

@@ -567,7 +567,7 @@ set(PYBIND11_PYTEST_ARGS
 # A single command to compile and run the tests
 add_custom_target(
   pytest
-  COMMAND ${PYBIND11_TEST_PREFIX_COMMAND} ${PYTHON_EXECUTABLE} -X dev -X faulthandler -m pytest
+  COMMAND ${PYBIND11_TEST_PREFIX_COMMAND} ${PYTHON_EXECUTABLE} -m pytest
           ${PYBIND11_ABS_PYTEST_FILES} ${PYBIND11_PYTEST_ARGS}
   DEPENDS ${test_targets}
   WORKING_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}"

tests/test_methods_and_attributes.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import gc
 import sys
 
 import pytest
@@ -20,12 +19,6 @@
 )
 
 
-def gc_collect(repeat=5):
-    """Collect garbage multiple times to ensure that objects are actually deleted."""
-    for _ in range(repeat):
-        gc.collect()
-
-
 def test_self_only_pos_only():
     assert (
         m.ExampleMandA.__str__.__doc__
@@ -316,7 +309,7 @@ def test_property_rvalue_policy():
     sys.version_info == (3, 14, 0, "beta", 1)
     or sys.version_info == (3, 14, 0, "beta", 2),
     reason="3.14.0b1/2 bug: https://github.com/python/cpython/issues/133912",
-    strict=False,
+    strict=True,
 )
 def test_dynamic_attributes():
     instance = m.DynamicClass()
@@ -344,32 +337,24 @@ def test_dynamic_attributes():
     cstats = ConstructorStats.get(m.DynamicClass)
     assert cstats.alive() == 1
     del instance
-    gc_collect(repeat=10)
     assert cstats.alive() == 0
 
     # Derived classes should work as well
     class PythonDerivedDynamicClass(m.DynamicClass):
         pass
 
-    for cls in (m.CppDerivedDynamicClass, PythonDerivedDynamicClass):
+    for cls in m.CppDerivedDynamicClass, PythonDerivedDynamicClass:
         derived = cls()
         derived.foobar = 100
         assert derived.foobar == 100
 
         assert cstats.alive() == 1
         del derived
-        gc_collect(repeat=10)
         assert cstats.alive() == 0
 
 
 # https://foss.heptapod.net/pypy/pypy/-/issues/2447
 @pytest.mark.xfail("env.PYPY")
-@pytest.mark.xfail(
-    sys.version_info == (3, 14, 0, "beta", 1)
-    or sys.version_info == (3, 14, 0, "beta", 2),
-    reason="3.14.0b1/2 bug: https://github.com/python/cpython/issues/133912",
-    strict=False,
-)
 @pytest.mark.skipif("env.GRAALPY", reason="Cannot reliably trigger GC")
 def test_cyclic_gc():
     # One object references itself
@@ -379,7 +364,6 @@ def test_cyclic_gc():
     cstats = ConstructorStats.get(m.DynamicClass)
     assert cstats.alive() == 1
     del instance
-    gc_collect(repeat=10)
     assert cstats.alive() == 0
 
     # Two object reference each other
@@ -390,7 +374,6 @@ def test_cyclic_gc():
 
     assert cstats.alive() == 2
     del i1, i2
-    gc_collect(repeat=10)
     assert cstats.alive() == 0
 
 

tests/test_pickling.py

@@ -72,7 +72,7 @@ def test_roundtrip(cls_name):
                 sys.version_info == (3, 14, 0, "beta", 1)
                 or sys.version_info == (3, 14, 0, "beta", 2),
                 reason="3.14.0b1/2 bug: https://github.com/python/cpython/issues/133912",
-                strict=False,
+                strict=True,
             ),
         ),
         "PickleableWithDictNew",