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 test for nightly Python (3.9) #532

Merged
merged 2 commits into from
Apr 26, 2020
Merged

Fix test for nightly Python (3.9) #532

merged 2 commits into from
Apr 26, 2020

Conversation

PetterS
Copy link
Contributor

@PetterS PetterS commented Apr 19, 2020

Trying to get the nightly test in #530 to pass.

@PetterS PetterS force-pushed the py39 branch 4 times, most recently from 2397757 to b03aaaf Compare April 19, 2020 08:47
@PetterS
Copy link
Contributor Author

PetterS commented Apr 19, 2020

OK, test passes now but feels way too brittle.

Should be enough to assert that the error starts with sourcePath + ":8:"

@asottile
Copy link
Member

could you rebase this? it should be fixed upstream now

here's a simpler patch you can use instead

@PetterS
Copy link
Contributor Author

PetterS commented Apr 23, 2020

Done!

But the Travis check does not seem to have run for some reason.

@asottile
Copy link
Member

looks like the new PEG parser has broken these even further :(

@PetterS
Copy link
Contributor Author

PetterS commented Apr 25, 2020

Drop these tests for nightly? Seem a little too fragile?

@asottile
Copy link
Member

the tests are valuable, and have pointed out a bug in cpython

I think we'll probably allow_failures: ... for now though to allow patches to flow while we wait for cpython to fix things

@PetterS
Copy link
Contributor Author

PetterS commented Apr 26, 2020

OK, where is that setting?

(Should probably move some version of these tests into CPython if they are useful in catching CPython bugs)

@asottile
Copy link
Member

Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

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