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

Add a functional tests for 4685 #4686

Merged

Conversation

Pierre-Sassoulas
Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas commented Jul 7, 2021

Description

Add a functional test for a python 3.10 crash that occurred when inferring variables assigned in match patterns.

Type of Changes

Type
βœ“ πŸ› Bug fix

Related Issue

Issue fixed with: pylint-dev/astroid#1093
Closes #4685

@Pierre-Sassoulas
Copy link
Member Author

@cdce8p here's the first crash with python 3.10 πŸ˜„ ! Someone is using it πŸŽ‰

@cdce8p
Copy link
Member

cdce8p commented Jul 7, 2021

Someone is using it πŸŽ‰

That is the only way we'll catch them all πŸ˜„

--
I'll take a look at it soon, but I would guess the fix needs to happen in astroid.

@Pierre-Sassoulas Pierre-Sassoulas added the Needs astroid update Needs an astroid update (probably a release too) before being mergable label Jul 8, 2021
@cdce8p cdce8p force-pushed the regression-tests-for-4685 branch from 5833fc5 to a806175 Compare July 10, 2021 22:50
@cdce8p cdce8p added this to the 2.9.4 milestone Jul 10, 2021
@cdce8p
Copy link
Member

cdce8p commented Jul 10, 2021

@Pierre-Sassoulas I've updated the PR to use the test cases mentioned in pylint-dev/astroid#1093. Does that work for you? I could revert the commits to restore the original test case, but I believe the current one is more precise and even covers more errors.

@Pierre-Sassoulas
Copy link
Member Author

Sure, no problem !

@Pierre-Sassoulas Pierre-Sassoulas modified the milestones: 2.9.4, 2.9.5 Jul 11, 2021
@cdce8p cdce8p modified the milestones: 2.9.5, 2.9.4 Jul 14, 2021
@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.058% when pulling 1bc0350 on Pierre-Sassoulas:regression-tests-for-4685 into 3a6f08e on PyCQA:main.

@Pierre-Sassoulas Pierre-Sassoulas merged commit 6950464 into pylint-dev:main Jul 19, 2021
@Pierre-Sassoulas Pierre-Sassoulas deleted the regression-tests-for-4685 branch July 20, 2021 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Crash πŸ’₯ A bug that makes pylint crash Needs astroid update Needs an astroid update (probably a release too) before being mergable python 3.10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MatchAs object has no attribute assigned_stmts
3 participants