test: PyPy fixes by tiran · Pull Request #543 · python-ldap/python-ldap · GitHub
Skip to content

test: PyPy fixes#543

Closed
tiran wants to merge 1 commit intopython-ldap:mainfrom
tiran:pypy-fix
Closed

test: PyPy fixes#543
tiran wants to merge 1 commit intopython-ldap:mainfrom
tiran:pypy-fix

Conversation

@tiran
Copy link
Copy Markdown
Member

@tiran tiran commented Oct 30, 2023

On PyPy 3.9 sock.close() sometimes fails with Bad file descriptor, because the fd is already closed. Silence the exception. The problem is not a bug in python-ldap.

Really use PyPy 3.9 in tox.ini.

On PyPy 3.9 `sock.close()` sometimes fails with `Bad file descriptor`,
because the fd is already closed. Silence the exception. The problem is
not a bug in python-ldap.

Really use PyPy 3.9 in `tox.ini`.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@mistotebe
Copy link
Copy Markdown
Contributor

@tiran
Copy link
Copy Markdown
Member Author

tiran commented Oct 30, 2023

Be my guest and good luck figuring it out... The problem only occurs with PyPy 3.9 on GitHub Actions and only every few test runs. It's rather random. I have not been able to reproduce the issue locally either.

In the mean time, I can work around the issue by restarting the failed test manually.

@mistotebe
Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants