-
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fails with PyQt6 #3
Comments
>>> __import__("PyQt6.uic.widget-plugins.qscintilla")
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/lib/python3.12/site-packages/PyQt6/uic/widget-plugins/qscintilla.py", line 25, in <module>
pluginType = MODULE
^^^^^^
NameError: name 'MODULE' is not defined So it's giving the correct result. That said, since this is not a valid package name, I wonder if it should just skip it. |
Yeah, didn't mean to say it was incorrect, more that it probably shouldn't be testing it if it's going to need some special plugin loader. If there's a way to do this automatically it'd be nice, otherwise I guess I'll try EPYTEST_DESELECT. |
Well, I have mixed feelings about this. On one hand, it clearly isn't importable out of the box and some packages deliberately name stuff like that to prevent them from being accidentally imported. But then, in some cases it probably still makes sense to test them. |
Tried to add import-check to PyQt6, and while most tests passed fine the few in widgets-plugins/ all failed. Not familiar with what PyQt6 is doing here albeit gives me the impression of just being a odd edge case that I could (manually) deselect if need be.
Figured would report anyway in case have a better idea, either to make it work (Edit: which doesn't sound doable to me if it needs some special loader) or automatically skip them.
(click to see full test output)
Fails with:
The text was updated successfully, but these errors were encountered: