Skip to content
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

bpo-37223: test_io: silence destructor errors #13954

Merged
merged 1 commit into from
Jun 11, 2019
Merged

bpo-37223: test_io: silence destructor errors #13954

merged 1 commit into from
Jun 11, 2019

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Jun 11, 2019

Implement also MockNonBlockWriterIO.seek() method.

https://bugs.python.org/issue37223

Implement also MockNonBlockWriterIO.seek() method.
@bedevere-bot bedevere-bot added tests Tests in the Lib/test dir awaiting core review labels Jun 11, 2019
@vstinner vstinner merged commit b589cef into python:master Jun 11, 2019
@vstinner vstinner deleted the test_io_silence branch June 11, 2019 01:11
vstinner added a commit that referenced this pull request Jun 12, 2019
* bpo-18748: Fix _pyio.IOBase destructor (closed case) (GH-13952)

_pyio.IOBase destructor now does nothing if getting the closed
attribute fails to better mimick _io.IOBase finalizer.

(cherry picked from commit 4f6f7c5)

* bpo-37223: test_io: silence destructor errors (GH-13954)

Implement also MockNonBlockWriterIO.seek() method.

(cherry picked from commit b589cef)

* bpo-37223, test_io: silence last 'Exception ignored in:' (GH-14029)

Use catch_unraisable_exception() to ignore 'Exception ignored in:'
error when the internal BufferedWriter of the BufferedRWPair is
destroyed. The C implementation doesn't give access to the
internal BufferedWriter, so just ignore the warning instead.

(cherry picked from commit 913fa1c)
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
Implement also MockNonBlockWriterIO.seek() method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants