fix: test critical_section for no-gil only

Author: XuehaiPan

tests/test_scoped_critical_section.cpp

@@ -127,6 +127,7 @@ TEST_SUBMODULE(scoped_critical_section, m) {
                                 .def("get", &BoolWrapper::get)
                                 .def("set", &BoolWrapper::set);
     auto BoolWrapperHandle = py::handle(BoolWrapperClass);
+    (void) BoolWrapperHandle.ptr(); // suppress unused variable warning
 
 #ifdef PYBIND11_HAS_BARRIER
     m.attr("has_barrier") = true;

tests/test_scoped_critical_section.py

@@ -2,21 +2,25 @@
 
 import pytest
 
+import env
 from pybind11_tests import scoped_critical_section as m
 
 
+@pytest.mark.skipif(not env.PY_GIL_DISABLED, reason="requires GIL disabled")
 @pytest.mark.skipif(not m.has_barrier, reason="no <barrier>")
 def test_scoped_critical_section() -> None:
     for _ in range(64):
         assert m.test_scoped_critical_section() is True
 
 
+@pytest.mark.skipif(not env.PY_GIL_DISABLED, reason="requires GIL disabled")
 @pytest.mark.skipif(not m.has_barrier, reason="no <barrier>")
 def test_scoped_critical_section2() -> None:
     for _ in range(64):
         assert m.test_scoped_critical_section2() == (True, True)
 
 
+@pytest.mark.skipif(not env.PY_GIL_DISABLED, reason="requires GIL disabled")
 @pytest.mark.skipif(not m.has_barrier, reason="no <barrier>")
 def test_scoped_critical_section2_same_object_no_deadlock() -> None:
     for _ in range(64):