Skip to content
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

Fix a bunch of tests broken by python 3.10 #10404

Merged
merged 1 commit into from
May 4, 2021

Conversation

freundTech
Copy link
Contributor

Python 3.10 changes a bunch of parser error messages. This commit introduces the version>= argument to [out] sections of tests, which allows testing against the new messages on newer python versions.

[out] sections are overwritten by other [out] sections following them.

I also thought about supporting other comparisons such as <, == and others, while not having later out sections overwrite earlier out sections, but decided on this approach for simplicity.

This was tested on Python 3.10.0a7+ (heads/master:0cad068, Apr 30 2021, 14:45:23) (shortly before 3.10.0b1). Note that Travis CI nighly is still stuck on 3.10.0a5+.

Python 3.10 changes a bunch of parser error messages. This commit
introduces the version>= argument to [out] sections of tests, which
allows testing against the new messages on newer python versions.
@freundTech freundTech changed the title Fix a bunch of test broken by python 3.10 Fix a bunch of tests broken by python 3.10 May 3, 2021
Copy link
Collaborator

@JukkaL JukkaL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This is a nice approach for dealing with differences between Python versions. Looks good.

@JukkaL JukkaL merged commit 42e2bb4 into python:master May 4, 2021
@freundTech freundTech mentioned this pull request Oct 14, 2021
21 tasks
JukkaL pushed a commit that referenced this pull request Dec 16, 2021
Followup to #11752. The syntax error changed between 3.10.0 and 3.10.1.
https://bugs.python.org/issue46004

I missed a few the first time around unfortunately.
https://bugs.python.org/issue46004

Instead of duplicating the original output, I chose to extend the test syntax 
introduced with #10404 and added support for == version checks.
JukkaL pushed a commit that referenced this pull request Dec 16, 2021
Followup to #11752. The syntax error changed between 3.10.0 and 3.10.1.
https://bugs.python.org/issue46004

I missed a few the first time around unfortunately.
https://bugs.python.org/issue46004

Instead of duplicating the original output, I chose to extend the test syntax 
introduced with #10404 and added support for == version checks.
tushar-deepsource pushed a commit to DeepSourceCorp/mypy that referenced this pull request Jan 20, 2022
Followup to python#11752. The syntax error changed between 3.10.0 and 3.10.1.
https://bugs.python.org/issue46004

I missed a few the first time around unfortunately.
https://bugs.python.org/issue46004

Instead of duplicating the original output, I chose to extend the test syntax 
introduced with python#10404 and added support for == version checks.
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