Skip to content

Conversation

@dr-carlos
Copy link
Contributor

@dr-carlos dr-carlos commented Nov 8, 2025

All statements within _Stringifier are now reported as covered in tests by coverage.py.

@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Nov 8, 2025
@JelleZijlstra JelleZijlstra added the needs backport to 3.14 bugs and security fixes label Nov 10, 2025
@JelleZijlstra JelleZijlstra merged commit 55ea132 into python:main Nov 10, 2025
52 checks passed
@miss-islington-app
Copy link

Thanks @dr-carlos for the PR, and @JelleZijlstra for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 10, 2025
…ythonGH-141220)

* Test `_Stringifier.__convert_to_ast()` for containers

* Test partial evaluation of `ForwardRef`s in `_Stringifier`
(cherry picked from commit 55ea132)

Co-authored-by: dr-carlos <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Nov 10, 2025

GH-141358 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Nov 10, 2025
JelleZijlstra pushed a commit that referenced this pull request Nov 10, 2025
…H-141220) (#141358)

gh-141174: Improve `annotationlib._Stringifier` test coverage (GH-141220)

* Test `_Stringifier.__convert_to_ast()` for containers

* Test partial evaluation of `ForwardRef`s in `_Stringifier`
(cherry picked from commit 55ea132)

Co-authored-by: dr-carlos <[email protected]>
@dr-carlos dr-carlos deleted the stringifier-tests branch November 10, 2025 22:02
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