Skip to content

Conversation

@camd
Copy link
Collaborator

@camd camd commented Jan 24, 2026

PR 2: camd/pr2-test-coverage (6 commits, based on PR 1)

  • Adds comprehensive unit tests for RSPack migration
  • Improves test coverage in key areas

@camd camd force-pushed the camd/pr2-test-coverage branch 2 times, most recently from d69b2d7 to 1c8f7d0 Compare January 24, 2026 18:56
@codecov-commenter
Copy link

codecov-commenter commented Jan 24, 2026

Codecov Report

❌ Patch coverage is 46.41350% with 127 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.41%. Comparing base (fac1470) to head (ace20a6).

Files with missing lines Patch % Lines
ui/intermittent-failures/Layout.jsx 0.00% 10 Missing ⚠️
ui/shared/RevisionInformation.jsx 0.00% 9 Missing ⚠️
ui/job-view/details/summary/SummaryPanel.jsx 0.00% 7 Missing ⚠️
ui/intermittent-failures/BugDetailsView.jsx 0.00% 5 Missing ⚠️
ui/job-view/details/DetailsPanel.jsx 72.22% 5 Missing ⚠️
ui/perfherder/alerts/AlertTable.jsx 44.44% 5 Missing ⚠️
ui/shared/FilterControls.jsx 0.00% 5 Missing ⚠️
ui/perfherder/graphs/GraphsContainer.jsx 42.85% 4 Missing ⚠️
ui/infra-compare/InfraCompareTableControls.jsx 0.00% 3 Missing ⚠️
ui/intermittent-failures/MainView.jsx 0.00% 3 Missing ⚠️
... and 40 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9179      +/-   ##
==========================================
+ Coverage   82.05%   82.41%   +0.35%     
==========================================
  Files         602      602              
  Lines       33794    33889      +95     
  Branches     3017     3273     +256     
==========================================
+ Hits        27731    27929     +198     
+ Misses       5938     5655     -283     
- Partials      125      305     +180     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@camd camd self-assigned this Jan 24, 2026
camd and others added 2 commits January 24, 2026 11:20
- Update pnpm version
- Optimize tests and docker configuration
- Add markdown formatting scripts and fix lint errors
- Upgrade packages
- Fix deprecations with defaultProps
- Fix React console warnings for duplicate keys and null props

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants