-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Meta issue: improving test runner #11662
Comments
My progress:
Now works. But, for some reason - this does not:
🤔 |
It needs to be
If you add |
I found lots of duplicate test names. It might hide important errors, be hard to reproduce or rerun. Refs #11662 Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
One other wish list item for me is to make it easy to exclude tests that depend on lxml. This would be useful for 3.11-dev / for future prerelease Pythons. It would also be useful for wheel builds, we currently do |
I found lots of duplicate test names. It might hide important errors, be hard to reproduce or rerun. Refs python#11662 Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
Detect if lxml is importable in the test suite, if it is not, then skip the report tests which depend on it. This is useful for enabling CI on new Python versions that may not have lxml wheels yet. Closes #11662.
Right now there are several things I would love to see improved in our unit tests:
.test
file. For example, when I need to runcheck-namedtuple.test
, I need to go totestcheck.py
, comment all other tests cases and then run it. It is very annoying. Maybe there's a hidden way? If this is the case, then we need to update the docs. Now you can run separate mypy test modules 🎉 #11668case
name detection. Right now we can have tests with non-unique names. For example:mypy/test-data/unit/check-newsemanal.test
Line 2308 in cee5d3e
mypy/test-data/unit/check-newsemanal.test
Line 2317 in cee5d3e
3.11-dev
to test suite and allow it to fail, later we can disable failures, but for now it is too unstable: Add3.11-dev
asTODO
item says #11677I will send a couple of PRs with these features if no one has any rejections 🙂
The text was updated successfully, but these errors were encountered: