Skip to content

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

Merged
pablogsal merged 1 commit into
python:3.11from
miss-islington:backport-054328f-3.11
Aug 23, 2022
Merged

pablogsal merged 1 commit into
python:3.11from
miss-islington:backport-054328f-3.11

Conversation

@miss-islington

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

Copy link
Copy Markdown
Contributor

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 mdboom@gmail.com

…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 <mdboom@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
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