Skip to content

gh-132805: annotationlib: Fix handling of non-constant values in FORWARDREF #132812

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.

Sign up for GitHub

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
JelleZijlstra merged 15 commits into from
May 4, 2025
Merged

gh-132805: annotationlib: Fix handling of non-constant values in FORWARDREF #132812

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
fba5efa
Add a test for union forward references
DavidCEllis Apr 22, 2025
d810953
Make stringifiers create unions if create_unions is True
DavidCEllis Apr 22, 2025
1908a4a
modify broken test, move test to forwardref format group
DavidCEllis Apr 22, 2025
97abdc7
Apparently trim trailing whitespace was turned off
DavidCEllis Apr 22, 2025
dd36a62
Alternative approach
JelleZijlstra Apr 22, 2025
20f3dac
Merge remote-tracking branch 'upstream/main' into fix-union-forwardref
JelleZijlstra Apr 22, 2025
8266a5e
fixes
JelleZijlstra Apr 22, 2025
1c629dc
blurb
JelleZijlstra Apr 22, 2025
3d7fec8
Handle displays
JelleZijlstra Apr 22, 2025
ec4dd0e
remove create_unions
JelleZijlstra Apr 22, 2025
6e31c69
prefer empty dict
JelleZijlstra Apr 22, 2025
525a4e4
Do not use special names in STRING format
JelleZijlstra Apr 23, 2025
0a75e5f
Merge branch 'main' into extra-names
JelleZijlstra May 4, 2025
7dec46b
Merge branch 'main' into extra-names
JelleZijlstra May 4, 2025
22a9691
fix bad merge
JelleZijlstra May 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into extra-names
  • Loading branch information
@JelleZijlstra
JelleZijlstra authored May 4, 2025
commit 0a75e5f830ac11527369c67729ba2f1c5daaad82
6 changes: 3 additions & 3 deletions Lib/test/test_annotationlib.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -399,7 +399,7 @@
pass

with self.assertRaisesRegex(TypeError, format_msg):
annotationlib.get_annotations(f, format=Format.STRING)
get_annotations(f, format=Format.STRING)

def test_shenanigans(self):
# In cases like this we can't reconstruct the source; test that we do something
Expand All @@ -408,7 +408,7 @@
pass

self.assertEqual(
annotationlib.get_annotations(f, format=Format.STRING),
get_annotations(f, format=Format.STRING),
{"x": "x | <class 'int'>", "y": "<class 'int'>"},
)

Expand All @@ -418,7 +418,7 @@
# Forward refs provide a different introspection API. __name__ and
# __qualname__ make little sense for forward refs as they can store
# complex typing expressions.
fr = annotationlib.ForwardRef("set[Any]")
fr = ForwardRef("set[Any]")
self.assertFalse(hasattr(fr, "__name__"))
self.assertFalse(hasattr(fr, "__qualname__"))
self.assertEqual(fr.__module__, "annotationlib")
Expand Down Expand Up @@ -1289,7 +1289,7 @@
pass


class TestForwardRefClass(unittest.TestCase):

Check failure on line 1292 in Lib/test/test_annotationlib.py

View workflow job for this annotation

GitHub Actions / lint

Ruff (F811) 

Lib/test/test_annotationlib.py:1292:7: F811 Redefinition of unused `TestForwardRefClass` from line 416
def test_forwardref_instance_type_error(self):
fr = ForwardRef("int")
with self.assertRaises(TypeError):
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.