-
Notifications
You must be signed in to change notification settings - Fork 772
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
"RuntimeError: Event loop is closed" when running tests with pytest-asyncio 0.23.5.post1 #1631
Comments
I think that this PR fixes this issue: #1634 |
https://build.opensuse.org/request/show/1170112 by user dgarcia + anag+factory - Add upstream patch fix-wamp-tests.patch to make it work with modern python-pytest-asyncio package gh#crossbario/autobahn-python#1631
yes, this is released on https://github.com/crossbario/autobahn-python/tree/v23.6.2 |
it is not. v23.6.2 dates from June 2023. |
will there be a corresponding pypi release? |
For what it's worth, I applied the fix as a patch on top of 23.6.2 and it fixes the issue reported. At least we know it's working. 😉 |
yes, I will push a release soonish. I tried actually at the time when I tagged the release - exactly like we always did. But even though I have 2FA on my PyPI, I ran into issues .. they changed sth. Anyways, I will look to get it done as soon as I find time. |
Here's my latest knowledge on that, stated as a Makefile: https://github.com/meejah/fowl/blob/main/Makefile#L32 (You have to create a "token" now, ideally scoped to just the one thing) |
I tried with
but it fails (auth error) ... will recheck tomorrow |
this was trying "manual release publishing". yet another issue is making "automatic release publishing" ( https://github.com/crossbario/autobahn-python/blob/master/.github/workflows/deploy.yml work again. |
no release yet? |
sorry, the PyPI issue got me unrailed trying to fix the release:( I will try again .. hope to get back to a reasonable / working flow .. |
We intend to update
pytest-asyncio
in Fedora to version 0.23.5.post1. Testing dependent packages showed failing tests inautobahn
inwamp
andwebsocket
:Test errors
Switching
autobahn
to usetwisted
for the time being wasn't as straight forward as I had hoped, either.Cc: @Jenselme (Fedora package maintainer)
The text was updated successfully, but these errors were encountered: