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

Update expected exception line numbers for Python 3.10.0rc1 #194

Merged
merged 1 commit into from Aug 6, 2021

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Aug 5, 2021

It seems that upstream has fixed line numbers in some of the expections
in Python 3.10.0rc1, so update the tests accordingly. This means that
test_non_async_in_async() gets the correct line again,
and test_default_except_error_postition() no longer suffers from
the apparent off-by-one problem.

This doesn't fix tests entirely with Python 3.10 but it's a step
forward.

@mgorny
Copy link
Contributor Author

mgorny commented Aug 5, 2021

NB: I've skipped AUTHORS.txt for this tiny change but if you believe I should add myself anyway, please lemme know.

Copy link
Collaborator

@isidentical isidentical left a comment

Choose a reason for hiding this comment

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

Thanks for the patch, due to PEP 626 there have been various fixes overall about line numbers of particular expression/statements and probably these got fixed a long the way.

@davidhalter
Copy link
Owner

There's a small linting issue. Can you fix that?

Otherwise thanks for the patch!

It seems that upstream has fixed line numbers in some of the expections
in Python 3.10.0rc1, so update the tests accordingly.  This means that
test_non_async_in_async() gets the correct line again,
and test_default_except_error_postition() no longer suffers from
the apparent off-by-one problem.

This doesn't fix tests entirely with Python 3.10 but it's a step
forward.
@mgorny
Copy link
Contributor Author

mgorny commented Aug 6, 2021

I'm sorry about that typo. Should be fixed now.

@davidhalter davidhalter merged commit e152301 into davidhalter:master Aug 6, 2021
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.

None yet

3 participants