Skip to content

Skip test_formatannotation_still_unfixed test on Python >= 3.10.1#349

Merged
mhils merged 2 commits intomitmproxy:mainfrom
Dliwk:remove-formatannotation-test
Feb 13, 2022
Merged

Skip test_formatannotation_still_unfixed test on Python >= 3.10.1#349
mhils merged 2 commits intomitmproxy:mainfrom
Dliwk:remove-formatannotation-test

Conversation

@Dliwk
Copy link
Contributor

@Dliwk Dliwk commented Feb 13, 2022

Description

inspect.formatannotation was fixed and it seems to have appeared in Python 3.10.1+. (Currently CI uses maximum 3.10.0).

I'm curious why snapshots have x: list annotations instead of x: list[int] if they use formatannotation from doc_types.py which have a fix 🤔.

@Dliwk
Copy link
Contributor Author

Dliwk commented Feb 13, 2022

Though it'd be better to just skip that test... and I will be glad if someone explains to me why CI renders exactly x: list.

@Dliwk Dliwk changed the title Remove test_formatannotation_still_unfixed test Skip test_formatannotation_still_unfixed test on Python >= 3.10.1 Feb 13, 2022
@mhils
Copy link
Member

mhils commented Feb 13, 2022

Thanks for the PR! 🍰
The snapshot tests expecting list and not list[int] is definitely unintended. I'll take a look. 😃

Co-authored-by: Roman Trapeznikov <roma190505@yandex.ru>
@mhils mhils force-pushed the remove-formatannotation-test branch from 82fec54 to fd0df3e Compare February 13, 2022 11:11
@mhils mhils merged commit 4d07277 into mitmproxy:main Feb 13, 2022
@Dliwk Dliwk deleted the remove-formatannotation-test branch February 13, 2022 11:35
@mhils mhils mentioned this pull request Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants