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

Confirm recovery phrase onboarding unit test #18250

Merged
merged 29 commits into from
Apr 21, 2023

Conversation

tmashuang
Copy link
Contributor

Explanation

Confirm recovery phrase onboarding unit test
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:44
@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 [a9902a5]
Page Load Metrics (1461 ± 35 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint87145107168
domContentLoaded1324154114465727
load1337164214617335
domInteractive1324154014465727
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #18250 (c58987a) into develop (f92e463) will increase coverage by 0.05%.
The diff coverage is n/a.

❗ Current head c58987a differs from pull request most recent head df26c16. Consider uploading reports for the commit df26c16 to get more accurate results

@@             Coverage Diff             @@
##           develop   #18250      +/-   ##
===========================================
+ Coverage    65.15%   65.20%   +0.05%     
===========================================
  Files          936      936              
  Lines        35965    35963       -2     
  Branches      9231     9231              
===========================================
+ Hits         23432    23447      +15     
+ Misses       12533    12516      -17     

see 9 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 [ec988e0]
Page Load Metrics (1488 ± 44 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint922751143919
domContentLoaded1358167214767737
load1358175414889244
domInteractive1358167214767737
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [745c3a7]
Page Load Metrics (1660 ± 51 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint94173125199
domContentLoaded14751943164510852
load14931943166010651
domInteractive14751943164510852
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [f05a1be]
Page Load Metrics (1607 ± 47 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint100154119168
domContentLoaded1440179416009948
load1440179416079747
domInteractive1440179416009948
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [6e4ac97]
Page Load Metrics (1575 ± 105 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint841841062411
domContentLoaded136123471560218105
load136123471575218105
domInteractive136123471560218105
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [73811dd]
Page Load Metrics (1732 ± 106 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint912381253316
domContentLoaded143723021720223107
load144923021732222106
domInteractive143623021720223107
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [1ed2776]
Page Load Metrics (1612 ± 61 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint923721326632
domContentLoaded14491859159411555
load14541953161212761
domInteractive14491859159411555
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [1751da0]
Page Load Metrics (1555 ± 55 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint952041152412
domContentLoaded14061810154110550
load14061810155511455
domInteractive14061810154010550
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [f19336f]
Page Load Metrics (1604 ± 89 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint87142108136
domContentLoaded13972102158616378
load13972217160418489
domInteractive13972102158616378
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [b1bb8fe]
Page Load Metrics (1549 ± 52 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint90171109199
domContentLoaded1426182715399947
load14261839154910952
domInteractive1426182715399947
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [1991ada]
Page Load Metrics (1593 ± 42 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint922231263015
domContentLoaded1474181515828239
load1474181515938742
domInteractive1474181515828239
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [c58987a]
Page Load Metrics (1727 ± 53 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1052021272110
domContentLoaded14281969170812259
load15271969172711053
domInteractive14281969170812259
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [c58987a]
Page Load Metrics (1727 ± 53 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1052021272110
domContentLoaded14281969170812259
load15271969172711053
domInteractive14281969170812259
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [da9c8be]
Page Load Metrics (2200 ± 91 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint993671725225
domContentLoaded16412467216818388
load17722636220018991
domInteractive16412466216818388
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [f06c746]
Page Load Metrics (1604 ± 45 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint942901254120
domContentLoaded1455180415869445
load1455180416049445
domInteractive1455180415869445
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [414426c]
Page Load Metrics (1603 ± 45 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint9513311095
domContentLoaded1415182815919244
load1415182816039545
domInteractive1415182815919244
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [a6aa131]
Page Load Metrics (1504 ± 22 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint852481083416
domContentLoaded1422156314933617
load1422161015044522
domInteractive1422156314933617
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [ed783ba]
Page Load Metrics (1739 ± 64 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint953091354622
domContentLoaded15131951172312761
load15132031173913464
domInteractive15131951172312761
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [01286cd]
Page Load Metrics (1505 ± 43 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint941901122110
domContentLoaded1339167014968641
load1339167015059043
domInteractive1339167014968641
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [27cb32d]
Page Load Metrics (1638 ± 56 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint872531263617
domContentLoaded14571874163011957
load14631874163811756
domInteractive14571874163011957
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [df26c16]
Page Load Metrics (1623 ± 71 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint92153119178
domContentLoaded14092029160514469
load14092050162314971
domInteractive14092029160514469
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@tmashuang tmashuang merged commit 7d080dd into develop Apr 21, 2023
@tmashuang tmashuang deleted the 17191-onboarding-unit-tests-confirm-recovery-phrase branch April 21, 2023 01:25
@github-actions github-actions bot locked and limited conversation to collaborators Apr 21, 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