Hi Paul,

I have checked in bonsai code base, and there is no reference to
should_close_asyncio_loop. It looks like a tornado thing.

However, looking at the full error trace, I can see that:

> ldapwhoami: unrecognized option -�
> Issue LDAP Who am I? operation to request user's authzid
>
> usage: ldapwhoami [options]
> Common options:
>   -d level   set LDAP debugging level to `level'
>   -D binddn  bind DN
> [...]

Which may cause the tests to be aborted early and cause
should_close_asyncio_loop to be accessed[1] before it is defined[2] in
tornado.

[1]: 
https://salsa.debian.org/python-team/packages/python-tornado/-/blob/master/tornado/testing.py#L282
[2]: 
https://salsa.debian.org/python-team/packages/python-tornado/-/blob/master/tornado/testing.py#L204-230

There were a few unreleased patches from bonsai[3], I'll make a new
release.

[3]: 
https://salsa.debian.org/python-team/packages/python-bonsai/-/commit/22d2533ab8094a299d1816a46917070e2b251265

In the meantime, I think that the should_close_asyncio_loop attribute
should be defined in tornado.testing.AsyncTestCase.__init__() instead of
in tornado.testing.AsyncTestCase.setUp().

Reply via email to