Skip to content

[3.11] gh-96189: Fix test_invalid_utf8 on a number of build bots (GH-96190) #96218

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

Merged
merged 1 commit into from
Aug 23, 2022

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Aug 23, 2022

The clearing of the temporary directory is not working on some platforms and
leaving behind files.

This has been updated to use the pattern in test_cmd_line.py [1] using the
special TESTFN rather than a test directory.

[1] https://github.com/python/cpython/blob/main/Lib/test/test_cmd_line.pyGH-L559
(cherry picked from commit 054328f)

Co-authored-by: Michael Droettboom [email protected]

…onGH-96190)

The clearing of the temporary directory is not working on some platforms and
leaving behind files.

This has been updated to use the pattern in test_cmd_line.py [1] using the
special TESTFN rather than a test directory.

[1] https://github.com/python/cpython/blob/main/Lib/test/test_cmd_line.pyGH-L559
(cherry picked from commit 054328f)

Co-authored-by: Michael Droettboom <[email protected]>
@miss-islington
Copy link
Contributor Author

@mdboom: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

@mdboom: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington
Copy link
Contributor Author

Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

Status check is done, and it's a success ✅ .

@pablogsal pablogsal merged commit c01fc9d into python:3.11 Aug 23, 2022
@miss-islington miss-islington deleted the backport-054328f-3.11 branch August 23, 2022 22:28
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.

4 participants