Repository navigation
test: move unit tests into unitCases - #22132
Conversation
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (128)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughChangesUnit-test files now reside under Unit test relocation
Priority: ⬇️ Low Change: Other Merge Risk: ⚪ Minimal · up to The unit-test relocation preserves the reviewed discovery and path-resolution behavior, with no active merge-blocking risk identified. 🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (3 passed)
Full details: Title checkExplanation The title uses the valid Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This PR is packaged and the instant preview is available (f9817c9). Install it locally:
npm i -D webpack@https://pkg.pr.new/webpack@f9817c9
yarn add -D webpack@https://pkg.pr.new/webpack@f9817c9
pnpm add -D webpack@https://pkg.pr.new/webpack@f9817c9 |
Types CoverageCoverage after merging coderabbit/test-unit-cases into main will be
Coverage Report |
Generated code sizeComparing
No asset built from unchanged case source changed size — every change below comes with a case whose own source moved too. 1 asset(s) changed size, and so did their case's source
No runtime gained or lost a runtime module. Built |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #22132 +/- ##
=======================================
Coverage 94.88% 94.89%
=======================================
Files 734 734
Lines 103485 103485
Branches 31846 31846
=======================================
+ Hits 98194 98197 +3
+ Misses 5291 5288 -3
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:
|
Merging this PR will degrade performance by 25.49%
|
Summary
Move root-level unit tests and their snapshots into
test/unitCases/so each test family has a dedicated directory.What kind of change does this PR introduce?
test
Did you add tests for your changes?
No new coverage; all 120 existing unit suites were relocated and validated from their new paths.
Does this PR introduce a breaking change?
No.
If relevant, what needs to be documented once your changes are merged or what have you already documented?
Updated
AGENTS.mdandTESTING_DOCS.md.Use of AI
CodeRabbit moved the files, updated path-sensitive configuration and references, and ran validation under Alexander Akait's direction.
Final Prompt
Open PR
Final Plan
Opened PR #22132.
test/unitCases/**— 120 tests + 4 snapshots relocated,+644/-634+29/-29+673/-663Validation:
yarn lintpassed. Unit suite: 119/120 suites passed; the only failure requires unavailable local Chrome.Branches on origin:
coderabbit/test-unit-cases.Initiative Context
Summary by CodeRabbit
Documentation
Tests
test/unitCasesdirectory.