-
Notifications
You must be signed in to change notification settings - Fork 68
Migrate tests to nose2 instead of nose
#747
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
Conversation
- Ignore more stuff in test files.
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #747 +/- ##
=======================================
Coverage 95.24% 95.24%
=======================================
Files 29 29
Lines 3578 3578
=======================================
Hits 3408 3408
Misses 170 170 ☔ View full report in Codecov by Sentry. |
|
There are still references to |
Good catch on |
|
All unit tests passed with 3.11. |
nosewithnose2in requirements file.unittest.cfgfor nose2 configurationskll.utils.testingmodule sincenose2doesn't pick up thetestsdirectory as a module.skll.utils.testing..coveragercwith proper exclusions.This PR closes #746.