Skip to content

test: Migrate setting-general e2e test to Typescript and POM #32247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

benjisclowder
Copy link
Contributor

@benjisclowder benjisclowder commented Apr 24, 2025

Description

Migrating the settings-general e2e test to Typescript and Page Object Model.

Open in GitHub Codespaces

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MMQA-505?atlOrigin=eyJpIjoiMjEwZDhiMjAyNWM0NGUzMWJhNjQxMjhlODBhODhjNDAiLCJwIjoiaiJ9

Manual testing steps

  1. Go to this page...

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.

@benjisclowder benjisclowder requested a review from chloeYue April 24, 2025 15:11
@benjisclowder benjisclowder marked this pull request as draft April 24, 2025 15:12
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-qa QA team label Apr 24, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [5a477e3]
UI Startup Metrics (1234 ± 64 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1234110814496412671348
load106795512606010961173
domContentLoaded106195012536010901162
domInteractive18146981829
firstPaint758167121840610761150
backgroundConnect74182813
firstReactRender22156262232
getState1353871728
initialActions001001
loadScripts819719101457845918
setupStore84334813
WebpackHomeuiStartup20961713248618622282350
load16151335203415017531826
domContentLoaded16071328188714617471819
domInteractive16115391445
firstPaint1716833559193302
backgroundConnect2612164192661
firstReactRender21055373122329364
getState154335331432
initialActions318135
loadScripts16021325188214617451815
setupStore1674772031
FirefoxBrowserifyHomeuiStartup13601196167611114391596
load12061013155410112671389
domContentLoaded12051013155310112661388
domInteractive1023725234112176
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2212156191950
firstReactRender23195972330
getState12420331810
initialActions001001
loadScripts1185999153910212541363
setupStore10418324630
WebpackHomeuiStartup15481377192712516031850
load13251178169711313901596
domContentLoaded13241177169611313901596
domInteractive87582682797130
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21144662336
firstReactRender36295243744
getState943251017
initialActions002111
loadScripts13061158168211413731578
setupStore11527427815
Benchmark value 13 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 30 exceeds gate value 27 for firefox browserify home p95 setupStore
Sum of mean exceeds: 4ms | Sum of p95 exceeds: 3ms
Sum of all benchmark exceeds: 7ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -998 Bytes (-0.02%)
  • ui: 732 Bytes (0.01%)
  • common: 1.07 KiB (0.01%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-qa QA team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants