reduce rename-split test-data to minimal possible size#719
reduce rename-split test-data to minimal possible size#719ceblanton merged 6 commits intorename-splitfrom
rename-split test-data to minimal possible size#719Conversation
Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
rename-split test-data to minimal possible size
rename-split test-data to minimal possible sizerename-split test-data to minimal possible size
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## rename-split #719 +/- ##
=============================================
Coverage 82.94% 82.94%
=============================================
Files 69 69
Lines 4655 4655
=============================================
Hits 3861 3861
Misses 794 794
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Completed in commit b3300ce. ocean_annual test files reduced from 2835 lines to 97 lines (97% reduction):
All other test cases remain unchanged. All 15 tests pass. |
|
annoying but because i removed the one-off workflow trigger, it un-associated the workflow from this PR. the proof-in-pudding workflow is linked here |
|
|
Describe your changes
Restored previously deleted test fixtures while minimizing atmos_daily and ocean_annual test files:
Test Data Restored (23 CDL files):
Test Configuration Updated:
Minimized Test Files:
Other Test Files: Remain at original size (river_month: 232 lines, ocean_static: 104 lines)
Test Results: 15 total (13 passed, 2 xfail)
Overall Impact: Total test directory size reduced from 1.1M to 444K (60% reduction)
Issue ticket number and link (if applicable)
N/A
Checklist before requesting a review
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.