Skip to content

Conversation

@seaona
Copy link
Member

@seaona seaona commented Nov 28, 2025

Description

Controllers take longer than usual to load if we use a wallet with a big state, causing the spec to fail as controllers are not loaded within 10 seconds (the css class which indicates that).
This increases the timeout for this specific test.

image

Open in GitHub Codespaces

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

  1. Check ci -- note the spec was run multiple times due to the e2e quality gate, and it passed 100% of times

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@metamaskbot metamaskbot added the team-qa QA team label Nov 28, 2025
@seaona seaona changed the title test: debug port test: fix flaky test Port Stream Chunking can load the wallet UI with a huge background state (~128MB) Nov 28, 2025
@seaona seaona self-assigned this Nov 28, 2025
@seaona seaona added flaky tests no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed labels Nov 28, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [f341474]
UI Startup Metrics (1211 ± 96 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup121199715359612871374
load101786311978910841181
domContentLoaded101185711898710761172
domInteractive261595202189
firstPaint5259112443959961150
backgroundConnect21119426210216230
firstReactRender312059103753
getState3515118194199
initialActions105112
loadScripts80665997285868965
setupStore1162841221
numNetworkReqs1257820573
BrowserifyPower User HomeuiStartup21061757271824823062670
load1030898150313310401356
domContentLoaded1016890149613310211348
domInteractive37172283433127
firstPaint60310314613979841264
backgroundConnect25420462191233531
firstReactRender64401261871102
getState18413027733207252
initialActions1010114
loadScripts80568112931338121138
setupStore221164102541
numNetworkReqs103662835498269
WebpackStandard HomeuiStartup8577191172899011049
load65758589871683827
domContentLoaded65257989071676819
domInteractive2716106232295
firstPaint23181895173226652
backgroundConnect1167381126
firstReactRender28206073340
getState2913197203647
initialActions107112
loadScripts64957788169674809
setupStore1274651421
numNetworkReqs1257720573
WebpackPower User HomeuiStartup16951287267427619712212
load6985951237110695964
domContentLoaded6885911221110681958
domInteractive38192053235123
firstPaint3041161233221292732
backgroundConnect66766915819580
firstReactRender624710796875
getState19213373977190221
initialActions102012
loadScripts6855891211108679949
setupStore221056122551
numNetworkReqs1756443084215419
FirefoxBrowserifyStandard HomeuiStartup13081075179115513771640
load1082923152311411431348
domContentLoaded1082923152311411431348
domInteractive64321633486136
firstPaint------
backgroundConnect45221732859104
firstReactRender23185062442
getState127121141022
initialActions102122
loadScripts1054906146310510991259
setupStore125221211021
numNetworkReqs1156616663
BrowserifyPower User HomeuiStartup25972025421035727363329
load1190979166315011891578
domContentLoaded1190979166315011891578
domInteractive12937571120118508
firstPaint------
backgroundConnect1093046365125235
firstReactRender61421321765105
getState23168795161233676
initialActions3125338
loadScripts1158956163314611591535
setupStore17512769201207685
numNetworkReqs101613146380244
WebpackStandard HomeuiStartup15421363198912516181768
load1271112714947813151426
domContentLoaded1271112614937813151426
domInteractive75291502793120
firstPaint------
backgroundConnect47191872948113
firstReactRender29238293041
getState1871352514114
initialActions103122
loadScripts1244111214257012931382
setupStore156122151441
numNetworkReqs1256717763
WebpackPower User HomeuiStartup29642265509568829134713
load14721156303142714262746
domContentLoaded14711155303142814252746
domInteractive11328590118102434
firstPaint------
backgroundConnect1842813382811471150
firstReactRender66392823764144
getState324661255252520793
initialActions207123
loadScripts13601132271922814061785
setupStore1647759186257598
numNetworkReqs100583176080238
📊 Page Load Benchmark Results

Current Commit: f341474 | Date: 11/28/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.02s (±40ms) 🟡 | historical mean value: 1.04s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 710ms (±36ms) 🟢 | historical mean value: 724ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 75ms (±11ms) 🟢 | historical mean value: 78ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.02s 40ms 996ms 1.30s 1.07s 1.30s
domContentLoaded 710ms 36ms 689ms 976ms 740ms 976ms
firstPaint 75ms 11ms 56ms 164ms 84ms 164ms
firstContentfulPaint 75ms 11ms 56ms 164ms 84ms 164ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 768 Bytes (0.02%)
  • ui: 2.16 KiB (0.03%)
  • common: 7.39 KiB (0.08%)

@github-actions github-actions bot added size-S and removed size-XS labels Nov 28, 2025
@seaona seaona marked this pull request as ready for review November 28, 2025 12:20
Copy link
Contributor

@chloeYue chloeYue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@metamaskbot
Copy link
Collaborator

Builds ready [d20aea0]
UI Startup Metrics (1226 ± 96 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup1226104114309612981391
load102586512148710811189
domContentLoaded102085812098610771177
domInteractive251596202186
firstPaint4609512123889851174
backgroundConnect21819824710224234
firstReactRender3320133153758
getState39141272045101
initialActions105112
loadScripts80765098383849968
setupStore1162641221
numNetworkReqs1257720573
BrowserifyPower User HomeuiStartup21151735331328423142640
load1032896174814810291417
domContentLoaded1018889173914810111410
domInteractive36181513034130
firstPaint5428817574029651306
backgroundConnect261209661106230543
firstReactRender5943100146395
getState18713433733204255
initialActions107112
loadScripts80868815201478041208
setupStore21107292432
numNetworkReqs1576741777205325
WebpackStandard HomeuiStartup8697311182979151068
load67659298388697878
domContentLoaded67158697587694873
domInteractive28171222423100
firstPaint26282982196240680
backgroundConnect1165471127
firstReactRender28215973242
getState271455103544
initialActions103112
loadScripts66858496686692864
setupStore1172541421
numNetworkReqs1257720573
WebpackPower User HomeuiStartup16341264221526019042123
load66757695890682908
domContentLoaded65757095191669902
domInteractive37181683434136
firstPaint29393960211290770
backgroundConnect73761515419559
firstReactRender6244113126585
getState1658722520174209
initialActions108112
loadScripts65456894289666894
setupStore24765153255
numNetworkReqs1266837466133320
FirefoxBrowserifyStandard HomeuiStartup13431091191114714381570
load109594713529011431272
domContentLoaded109494613519011421272
domInteractive65312233586125
firstPaint------
backgroundConnect48233544244100
firstReactRender25195382644
getState157233271233
initialActions103122
loadScripts106693213268211131227
setupStore145201271131
numNetworkReqs1156614653
BrowserifyPower User HomeuiStartup26922018507959327224068
load1216963299832811811750
domContentLoaded1216963299732811811750
domInteractive12233678128103486
firstPaint------
backgroundConnect134241103165123434
firstReactRender6135240266197
getState31359941227453838
initialActions2146523
loadScripts1172932280229711391620
setupStore1488759186151612
numNetworkReqs102593136380242
WebpackStandard HomeuiStartup15271259200914916351800
load12571092155110513461425
domContentLoaded12571092155010513461424
domInteractive67232343890127
firstPaint------
backgroundConnect52183154152113
firstReactRender25208172635
getState167153241381
initialActions102122
loadScripts1226106915219713091378
setupStore175243321160
numNetworkReqs1156816760
WebpackPower User HomeuiStartup29452113497566229674566
load14751169300141614692740
domContentLoaded14751169300041614682740
domInteractive11130528110100395
firstPaint------
backgroundConnect159311436232134461
firstReactRender70413824367145
getState28368930209385734
initialActions2033323
loadScripts14031144277133014142510
setupStore1625814210207703
numNetworkReqs100582505883243
📊 Page Load Benchmark Results

Current Commit: d20aea0 | Date: 11/28/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±43ms) 🟡 | historical mean value: 1.05s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 731ms (±42ms) 🟢 | historical mean value: 728ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 79ms (±12ms) 🟢 | historical mean value: 78ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 43ms 1.02s 1.40s 1.07s 1.40s
domContentLoaded 731ms 42ms 703ms 1.07s 751ms 1.07s
firstPaint 79ms 12ms 60ms 184ms 92ms 184ms
firstContentfulPaint 79ms 12ms 60ms 184ms 92ms 184ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 768 Bytes (0.02%)
  • ui: 2.25 KiB (0.03%)
  • common: 7.4 KiB (0.08%)

@seaona seaona added this pull request to the merge queue Nov 28, 2025
Merged via the queue into main with commit f855a5e Nov 28, 2025
197 of 198 checks passed
@seaona seaona deleted the flaky-controllers-heavy-data branch November 28, 2025 13:53
@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2025
@metamaskbot metamaskbot added the release-13.13.0 Issue or pull request that will be included in release 13.13.0 label Nov 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

flaky tests no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-13.13.0 Issue or pull request that will be included in release 13.13.0 size-S team-qa QA team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants