-
Notifications
You must be signed in to change notification settings - Fork 44.3k
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
Extend & improve file operations tests #3404
Extend & improve file operations tests #3404
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
91d1563
to
1575279
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #3404 +/- ##
==========================================
+ Coverage 57.19% 58.26% +1.07%
==========================================
Files 68 68
Lines 3079 3079
Branches 516 516
==========================================
+ Hits 1761 1794 +33
+ Misses 1180 1148 -32
+ Partials 138 137 -1
☔ View full report in Codecov by Sentry. |
9d6eefc
to
9d917a7
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
9d917a7
to
cb70c4b
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
eab45e7
to
381393c
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
381393c
to
44622b1
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
44622b1
to
c94cf64
Compare
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
c94cf64
to
d1ac531
Compare
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size |
Co-authored-by: Reinier van der Leer <[email protected]>
Background
Try to improve code coverage.
Switch from Unittest to Pytest
Changes
Migrate tests for file operations
PR Quality Checklist