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

Import srp onboarding unit tests #18255

Merged
merged 25 commits into from
Apr 20, 2023

Conversation

tmashuang
Copy link
Contributor

Explanation

Import srp onboarding unit tests
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:47
@tmashuang tmashuang requested a review from mcmire March 21, 2023 02:47
@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 [22327d7]
Page Load Metrics (1403 ± 18 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint871069652
domContentLoaded1346148114003718
load1346148114033618
domInteractive1346148114003718
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #18255 (8f72e66) into develop (f92e463) will increase coverage by 0.01%.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##           develop   #18255      +/-   ##
===========================================
+ Coverage    65.15%   65.17%   +0.01%     
===========================================
  Files          936      936              
  Lines        35965    35963       -2     
  Branches      9231     9231              
===========================================
+ Hits         23432    23436       +4     
+ Misses       12533    12527       -6     

see 7 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 [094c8a3]
Page Load Metrics (1860 ± 58 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint101175132189
domContentLoaded15782040184411555
load15782066186012258
domInteractive15782040184411555
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2b67527]
Page Load Metrics (1842 ± 67 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint962831364019
domContentLoaded15752103180815574
load15752121184214067
domInteractive15742103180815574
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [ab35666]
Page Load Metrics (1599 ± 47 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint972951204120
domContentLoaded1472187115779144
load1472187115999947
domInteractive1472187115779144
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [9af3340]
Page Load Metrics (1629 ± 43 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint94160121168
domContentLoaded1475177416238842
load1475179716299143
domInteractive1475177416238842
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [1db1b08]
Page Load Metrics (1785 ± 68 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1032431373015
domContentLoaded15562078177313766
load15562078178514268
domInteractive15562078177313766
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [0c3969b]
Page Load Metrics (1709 ± 49 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint961941262211
domContentLoaded15052019169711254
load15772019170910249
domInteractive15052019169711254
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [f3095e6]
Page Load Metrics (1680 ± 47 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint97131113105
domContentLoaded1541185816668742
load1543190116809847
domInteractive1541185816668742
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2033bc8]
Page Load Metrics (1554 ± 35 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint93158110168
domContentLoaded1384169915526933
load1384172015547235
domInteractive1384169915526933
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [8a15fbf]
Page Load Metrics (1635 ± 38 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint94154121178
domContentLoaded1492175816176431
load1492178216357938
domInteractive1491175816176431
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2339cc3]
Page Load Metrics (1774 ± 45 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1002301232713
domContentLoaded1571196717439043
load1571196717749445
domInteractive1570196717439043
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [8f72e66]
Page Load Metrics (1592 ± 62 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint952711163818
domContentLoaded14342007157812560
load14342007159212962
domInteractive14342007157812560
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [b745d68]
Page Load Metrics (1602 ± 64 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint913441355627
domContentLoaded13831848159613163
load13831848160213464
domInteractive13831848159613163

@metamaskbot
Copy link
Collaborator

Builds ready [60c92f7]
Page Load Metrics (1544 ± 49 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint93140106115
domContentLoaded1408180315258842
load14081889154410249
domInteractive1408180315258842
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [a63d8b2]
Page Load Metrics (1651 ± 46 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint941811162110
domContentLoaded1483190016379244
load1483190016519746
domInteractive1483190016379244
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [92dbb08]
Page Load Metrics (1634 ± 42 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint93164118178
domContentLoaded1502190216199043
load1522190216348842
domInteractive1502190216199043
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [fcc7a04]
Page Load Metrics (1690 ± 81 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint972721263818
domContentLoaded14552071167314971
load14692169169017081
domInteractive14552071167314971
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [09b8866]
Page Load Metrics (1592 ± 53 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint94142112136
domContentLoaded1464177515749445
load14651837159211153
domInteractive1464177515749445
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2843c99]
Page Load Metrics (1899 ± 75 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1062991434019
domContentLoaded16442233189315474
load16452240189915675
domInteractive16442233189315474
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Took a look at the source for ImportSrp and compared it to these tests. Makes sense to me!

@metamaskbot
Copy link
Collaborator

Builds ready [8e37ebd]
Page Load Metrics (1560 ± 36 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint9312710594
domContentLoaded1435174215487034
load1435176115607536
domInteractive1435174215487034
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [e5d55f2]
Page Load Metrics (1802 ± 77 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint962291303015
domContentLoaded14912023177615172
load15342109180216077
domInteractive14912023177615172
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@tmashuang tmashuang merged commit 928bd1c into develop Apr 20, 2023
@tmashuang tmashuang deleted the 17191-onboarding-unit-tests-import-srp branch April 20, 2023 16:06
@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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants