test: cover the migration failure branches codecov flagged - #125
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #125 +/- ##
============================================
+ Coverage 71.09% 78.64% +7.55%
Complexity 52 52
============================================
Files 11 11
Lines 384 384
============================================
+ Hits 273 302 +29
+ Misses 111 82 -29
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Neither dbDelta failing nor an unrecognized table shape had a test forcing them, so the version-not-recorded guard in sync_storage_create_table() and sync_storage_upgrade_table() ran untested. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
8b6370d to
66f421c
Compare
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
1 participant
#110 shipped with schema.php's dbDelta-failure and unrecognized-table-shape branches untested (codecov/patch flagged 23 uncovered lines). Adds three tests that force each branch through a real MySQL failure rather than a mock.