-
Notifications
You must be signed in to change notification settings - Fork 319
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
Replace nose with pytest #836
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #836 +/- ##
=======================================
+ Coverage 32% 53% +21%
=======================================
Files 88 88
Lines 14979 9848 -5131
Branches 1515 1841 +326
=======================================
+ Hits 4857 5299 +442
+ Misses 9773 4164 -5609
- Partials 349 385 +36
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
I'm ok to move from nose to pytest, motivations (#638) looks legit. |
This is fun. nose when running tests does 722 tests, when you run |
@@ -31,7 +31,6 @@ | |||
'README.rst' | |||
) | |||
).read(), | |||
test_suite='nose.collector', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was for setup.py test
, which is now deprecated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, no problem
I'm just guessing... Maybe 'SkipTest' is just skipped at runtime or something like that, and so, and therefore counted only in other cases (collect), although it seems unlikely that we'll skip only a single test... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
God job! Thanks @itamarst
Fixes #638
Unless nose3 starts being maintained again, this is a prerequisite for Python 3.12 support. nose3 is, in any case, essentially a dead project.