Closed tlambert03 closed 3 months ago
SystemError
is often an internal error in the bindings... I would report this upstream, seems like a bug to me.
k
would it be trivial for you to suggest an MRE without using pytest-qt's _AbstractSignalBlocker
? it's ok if not
would it be trivial for you to suggest an MRE without using pytest-qt's _AbstractSignalBlocker? it's ok if not
I don't have the time to work on it right now, but I suspect if you just connect a signal and try to disconnect it twice you will get the SystemError
.
If you do report it upstream, please link it here for reference. :+1:
PYSIDE-2705 suggests that the exception-like warning in 6.7.0 has been converted to an actual warning, compare https://codereview.qt-project.org/c/pyside/pyside-setup/+/558142/3/sources/pyside6/libpyside/pysidesignal.cpp. So there should be no more (unhandled) RuntimeError
.
I was able to suppress these errors like this (pyproject.toml
):
[tool.pytest.ini_options]
filterwarnings = [
# https://github.com/pytest-dev/pytest-qt/issues/558
"ignore:Failed to disconnect .* from signal:RuntimeWarning",
]
That gets rid of pytest
reporting both the new RuntimeWarning
as well as the underlying, and seemingly caught, SystemError
.
ahh, thank you for noting that. indeed I turn my warnings into exceptions, and I failed to note that this was just a warning and not an actual exception. adding that ignore works fine for my case as well. I'll close this issue, but @nicoddemus feel free to reopen if you want to track anything else related to this change
Don't see an existing issue for this yet, so opening one. This seems related to https://github.com/pytest-dev/pytest-qt/issues/552, but has now turned into an exception with pyside 6.7.1:
running this test raises:
on Pyside 6.7.0, this worked but caused the RuntimeError warning as noted in #552.