Skip to content

Conversation

@seaona
Copy link
Member

@seaona seaona commented Oct 31, 2025

Description

A PR which changes the text from Create Account to Add Account was merged yesterday:
#37288

This introduces flakiness in our e2e, as we had a condition to mitigate flakiness, which relied on Creating Account message which now has changed.

This PR updates the copy, so we mitigate the flakiness again.

See in the logs how we waited for Creating account .. and since this message is never there anymore, we jump into the next step causing the spec to fail. We should wait until the account creating has completed, by using the new copy.

image image

Open in GitHub Codespaces

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

  1. Check ci

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.

Note

Updates e2e selector to check for Adding account... instead of Creating account... before opening the multichain account menu.

  • Tests (e2e)
    • In test/e2e/page-objects/pages/account-list-page.ts:
      • Replace creatingAccountMessage with addingAccountMessage ('Creating account...' -> 'Adding account...').
      • Update openMultichainAccountMenu to assert absence of addingAccountMessage before proceeding.

Written by Cursor Bugbot for commit 2a8f851. This will update automatically on new commits. Configure here.

@metamaskbot metamaskbot added the team-qa QA team label Oct 31, 2025
@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

🧪 @MetaMask/qa (1 files, +6 -6)
  • 📁 test/
    • 📁 e2e/
      • 📁 page-objects/
        • 📁 pages/
          • 📄 account-list-page.ts +6 -6

@seaona seaona self-assigned this Oct 31, 2025
@seaona seaona added flaky tests no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed labels Oct 31, 2025
@seaona seaona marked this pull request as ready for review October 31, 2025 09:05
@seaona seaona requested a review from a team as a code owner October 31, 2025 09:05
text: 'Adding account...',
tag: 'p',
};

Copy link
Member Author

Choose a reason for hiding this comment

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

I've also re-ordered the selector, to follow alphabetic order with the new name

@metamaskbot
Copy link
Collaborator

📊 Page Load Benchmark Results

Current Commit: 2a8f851 | Date: 10/31/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.02s (±43ms) 🟡 | historical mean value: 1.04s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 713ms (±39ms) 🟢 | historical mean value: 725ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 75ms (±13ms) 🟢 | historical mean value: 76ms ⬇️ (historical data)
📈 Detailed Results
Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.02s 43ms 1000ms 1.33s 1.05s 1.33s
domContentLoaded 713ms 39ms 689ms 991ms 737ms 991ms
firstPaint 75ms 13ms 60ms 184ms 88ms 184ms
firstContentfulPaint 75ms 13ms 60ms 184ms 88ms 184ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms

Results generated automatically by MetaMask CI

@metamaskbot
Copy link
Collaborator

Builds ready [2a8f851]
UI Startup Metrics (1278 ± 88 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup1278112715278813311455
load109096113098311431264
domContentLoaded108394913018311351258
domInteractive19145281946
firstPaint67790129145011111227
backgroundConnect23421833813237254
firstReactRender30187993547
getState23895132540
initialActions71709725
loadScripts8577101060829101018
setupStore1061831116
numNetworkReqs96327631
BrowserifyPower User HomeuiStartup22841846453372828434533
load1191954195934315531959
domContentLoaded1178947191233315431912
domInteractive321596235196
firstPaint743170195754710241957
backgroundConnect30522311882292701188
firstReactRender26233232732
getState19513733948207339
initialActions73307830
loadScripts920719149427612661494
setupStore1191521315
numNetworkReqs342189695187582695
WebpackStandard HomeuiStartup8547301161828691068
load63858895875643886
domContentLoaded63158295174636875
domInteractive15114071334
firstPaint20859959211185607
backgroundConnect22125672444
firstReactRender27174673235
getState1262641520
initialActions3116247
loadScripts62758094171634863
setupStore1052531215
numNetworkReqs96368831
WebpackPower User HomeuiStartup16771266255438621622554
load75761212891968461289
domContentLoaded71760011691578301169
domInteractive191250102150
firstPaint43312411743397091174
backgroundConnect12619438152200438
firstReactRender25233632636
getState15210422725156227
initialActions160159399159
loadScripts71259811591548181159
setupStore17664152464
numNetworkReqs328162709165538709
FirefoxBrowserifyStandard HomeuiStartup14621292184711315281710
load1236109414888012911386
domContentLoaded1236109414878012911386
domInteractive1143338053121245
firstPaint------
backgroundConnect4326131185083
firstReactRender26215062642
getState10421722724
initialActions5275947
loadScripts1210107314677812591355
setupStore147254241320
numNetworkReqs96327727
BrowserifyPower User HomeuiStartup25532188327031727673270
load13271136152411914531524
domContentLoaded13271136152411914531524
domInteractive1656430771236307
firstPaint------
backgroundConnect15734457142221457
firstReactRender39295074850
getState1295819133153191
initialActions813081530
loadScripts12941118149111414211491
setupStore3861454150145
numNetworkReqs24215234068321340
WebpackStandard HomeuiStartup16351438199111817081907
load14031211177110214601633
domContentLoaded14021211177110314601633
domInteractive1093437254112248
firstPaint------
backgroundConnect4824108195985
firstReactRender322284143273
getState84496814
initialActions7118920418
loadScripts13751195173810214341612
setupStore1276471320
numNetworkReqs96347828
WebpackPower User HomeuiStartup27642400389444929663894
load16121333220222617772202
domContentLoaded16121332220122617762201
domInteractive1487831470198314
firstPaint------
backgroundConnect12438403101200403
firstReactRender41315684856
getState1458420936181209
initialActions825112751
loadScripts15731300213822017342138
setupStore5082517345251
numNetworkReqs24113744491302444
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 68 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 10 Bytes (0%)

@seaona seaona enabled auto-merge October 31, 2025 09:41
@seaona seaona added this pull request to the merge queue Oct 31, 2025
Merged via the queue into main with commit 0181bdf Oct 31, 2025
520 of 530 checks passed
@seaona seaona deleted the flaky-add-account-identity branch October 31, 2025 11:02
@metamaskbot metamaskbot added the release-13.9.0 Issue or pull request that will be included in release 13.9.0 label Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants