test_runner: add classname hierarchy for JUnit reporter by mag123c · Pull Request #60220 · nodejs/node
Conversation
Fixes: #59417
Implements classname hierarchy for the JUnit XML reporter.
Previously, all tests had classname="test".
Now the classname reflects the suite hierarchy (e.g., classname="Math.Addition" for nested suites).
Codecov Report
✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.80%. Comparing base (c1402c6) to head (175d312).
⚠️ Report is 1 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@ ## main #60220 +/- ## ========================================== + Coverage 89.77% 89.80% +0.02% ========================================== Files 672 672 Lines 203930 203944 +14 Branches 39207 39213 +6 ========================================== + Hits 183082 183151 +69 + Misses 13175 13116 -59 - Partials 7673 7677 +4
| Files with missing lines | Coverage Δ | |
|---|---|---|
| lib/internal/test_runner/test.js | 97.33% <100.00%> (+0.02%) |
⬆️ |
| lib/internal/test_runner/tests_stream.js | 89.77% <100.00%> (+0.11%) |
⬆️ |
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Hello @MoLow
Could you take a look? Thank you.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters