Skip to content
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

Metametrics onboarding unit test, snapshot #18253

Merged
merged 24 commits into from
Apr 20, 2023

Conversation

tmashuang
Copy link
Contributor

Explanation

Metametrics onboarding unit test and snapshot
Broken apart from #17360.
Partially completes #17191

Screenshots/Screencaps

Before

After

Manual Testing Steps

Pre-merge author checklist

  • I've clearly explained:
    • What problem this PR is solving
    • How this problem was solved
    • How reviewers can test my changes
  • Sufficient automated test coverage has been added

Pre-merge reviewer checklist

  • Manual testing (e.g. pull and build branch, run in browser, test code being changed)
  • PR is linked to the appropriate GitHub issue
  • IF this PR fixes a bug in the release milestone, add this PR to the release milestone

If further QA is required (e.g. new feature, complex testing steps, large refactor), add the Extension QA Board label.

In this case, a QA Engineer approval will be be required.

@tmashuang tmashuang requested a review from a team as a code owner March 21, 2023 02:46
@tmashuang tmashuang requested a review from PeterYinusa March 21, 2023 02:46
@github-actions
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
Copy link
Collaborator

Builds ready [a288b16]
Page Load Metrics (1579 ± 34 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint92143112147
domContentLoaded1466172315777235
load1466172315797234
domInteractive1466172315777235
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #18253 (2127eef) into develop (f92e463) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 2127eef differs from pull request most recent head 704895e. Consider uploading reports for the commit 704895e to get more accurate results

@@             Coverage Diff             @@
##           develop   #18253      +/-   ##
===========================================
+ Coverage    65.15%   65.18%   +0.02%     
===========================================
  Files          936      936              
  Lines        35965    35963       -2     
  Branches      9231     9231              
===========================================
+ Hits         23432    23439       +7     
+ Misses       12533    12524       -9     

see 8 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@metamaskbot
Copy link
Collaborator

Builds ready [fd9c8b4]
Page Load Metrics (1634 ± 56 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint98145119136
domContentLoaded1458180116139345
load14581843163411656
domInteractive1458180116139345
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [e410a26]
Page Load Metrics (1693 ± 50 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint92157124189
domContentLoaded14411851167410048
load14411851169310350
domInteractive14411850167410048
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [611d177]
Page Load Metrics (1484 ± 33 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint87151103157
domContentLoaded1378159914746632
load1378161714846833
domInteractive1378159914746632
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [dbf2c92]
Page Load Metrics (1510 ± 64 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint832921104320
domContentLoaded13281993150013565
load13281994151013364
domInteractive13281993150013565
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [b0392e5]
Page Load Metrics (1437 ± 34 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint8114996136
domContentLoaded1335168114267134
load1335168114377234
domInteractive1335168114267134
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [a50ba9d]
Page Load Metrics (1726 ± 46 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1032091262311
domContentLoaded1561189017149646
load1561189017269546
domInteractive1561189017149646
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [3484d52]
Page Load Metrics (1516 ± 38 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint86141104146
domContentLoaded1392170014997536
load1409170015167838
domInteractive1392170014997536
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [ce6c5b6]
Page Load Metrics (1442 ± 28 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint861189884
domContentLoaded1349156214354622
load1349160914425928
domInteractive1349156214354622
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2127eef]
Page Load Metrics (1597 ± 66 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint91134108115
domContentLoaded13491878158213967
load13491896159713866
domInteractive13491878158213967
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [d5e08df]
Page Load Metrics (1539 ± 41 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint93150109136
domContentLoaded1427175615358541
load1427176915398441
domInteractive1427175615358541

@metamaskbot
Copy link
Collaborator

Builds ready [2e910aa]
Page Load Metrics (1640 ± 58 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint91137108115
domContentLoaded14401789161710148
load14561869164012058
domInteractive14401789161710148
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [79eb20c]
Page Load Metrics (1707 ± 59 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint962611313316
domContentLoaded14571899168111756
load14571922170712359
domInteractive14571899168111756
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [8c7a6d1]
Page Load Metrics (1730 ± 59 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1031931242311
domContentLoaded15441979169311354
load15581980173012459
domInteractive15441979169311354
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [ab1d042]
Page Load Metrics (1719 ± 72 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1002051312412
domContentLoaded14982019171015072
load14982019171915172
domInteractive14982019171015072
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [00978e7]
Page Load Metrics (1633 ± 60 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint943151224622
domContentLoaded14311929162413263
load14701929163312660
domInteractive14311929162413263
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [9c1a9ad]
Page Load Metrics (1602 ± 45 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint972421243215
domContentLoaded1429177415999445
load1429177516029445
domInteractive1429177415999445
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [c8d7439]
Page Load Metrics (1656 ± 62 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint96150114147
domContentLoaded14622036163713565
load14622036165612962
domInteractive14622036163713565
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [704895e]
Page Load Metrics (1780 ± 69 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint88150122189
domContentLoaded14862161176615976
load15082161178014469
domInteractive14862161176515976
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@brad-decker brad-decker merged commit f9f6286 into develop Apr 20, 2023
@brad-decker brad-decker deleted the 17191-onboarding-unit-tests-metametrics branch April 20, 2023 18:12
@github-actions github-actions bot locked and limited conversation to collaborators Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants