Skip to content

Improve performance of test discovery in large projects#5937

Merged
DanTup merged 5 commits intomasterfrom
test-discovery-perf-improvements
Feb 26, 2026
Merged

Improve performance of test discovery in large projects#5937
DanTup merged 5 commits intomasterfrom
test-discovery-perf-improvements

Conversation

@DanTup
Copy link
Member

@DanTup DanTup commented Feb 26, 2026

For the pkg folder in SDK, this gets down from around 9.4s to around 1.3s on my machine.

I can't currently test the whole SDK due to #5936

… of the project

This will be much faster for Pub Workspaces
…lude the word "flutter"

This will be slightly slower if Flutter _is_ referenced, but _way_ faster if it's not.
This can remove hundreds of duplicate checks on the pubspec.yaml during test discovery
@DanTup DanTup added this to the v3.130.0 milestone Feb 26, 2026
@DanTup DanTup added in testing Relates to test execution of Dart/Flutter tests for end users is performance labels Feb 26, 2026
@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

❌ Patch coverage is 89.83051% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.60%. Comparing base (811c14f) to head (912e54a).
⚠️ Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
src/shared/test/version.ts 75.00% 2 Missing and 1 partial ⚠️
src/shared/utils/fs.ts 88.00% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5937   +/-   ##
=======================================
  Coverage   67.60%   67.60%           
=======================================
  Files         169      169           
  Lines       12992    13014   +22     
  Branches     2572     2577    +5     
=======================================
+ Hits         8783     8798   +15     
- Misses       3750     3755    +5     
- Partials      459      461    +2     

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

@DanTup DanTup merged commit 1bd4cc3 into master Feb 26, 2026
38 of 39 checks passed
@DanTup DanTup deleted the test-discovery-perf-improvements branch February 26, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in testing Relates to test execution of Dart/Flutter tests for end users is performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant