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

Onboarding app header onboarding unit test and snapshot #18254

Merged
merged 12 commits into from
Apr 5, 2023

Conversation

tmashuang
Copy link
Contributor

Explanation

Onboarding app header 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 hmalik88 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.

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #18254 (9d83b07) into develop (7390756) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #18254      +/-   ##
===========================================
+ Coverage    65.08%   65.10%   +0.02%     
===========================================
  Files          935      935              
  Lines        35924    35924              
  Branches      9228     9228              
===========================================
+ Hits         23378    23385       +7     
+ Misses       12546    12539       -7     

see 1 file 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 [78bb08e]
Page Load Metrics (1584 ± 45 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint962591163416
domContentLoaded1428179515698842
load1428179515849445
domInteractive1428179515698842
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [12e683d]
Page Load Metrics (1600 ± 100 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint972491294019
domContentLoaded137421691593209100
load137421691600209100
domInteractive137421691593209100
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [d0d4c6b]
Page Load Metrics (1663 ± 59 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint92158119168
domContentLoaded14911990164712962
load14951991166312259
domInteractive14911990164712962
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [5033326]
Page Load Metrics (1595 ± 41 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint98158120157
domContentLoaded1453183215758139
load1453183315958641
domInteractive1453183215758139
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [532b22c]
Page Load Metrics (1631 ± 86 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint93172123199
domContentLoaded14552032161616579
load14552146163118086
domInteractive14552032161616579
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [393f223]
Page Load Metrics (1644 ± 77 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint981971222512
domContentLoaded14892001162412862
load14902211164416077
domInteractive14892001162412862
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [3aca18a]
Page Load Metrics (1522 ± 34 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint97135109105
domContentLoaded1415162715146531
load1416166615227134
domInteractive1415162715146531
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [b5399dd]
Page Load Metrics (1825 ± 115 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint962001272311
domContentLoaded145822261803230110
load152423051825239115
domInteractive145822261803230110
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [9cee1ed]
Page Load Metrics (1531 ± 38 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint912711163718
domContentLoaded1401174915118641
load1409174915317838
domInteractive1401174915118641
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [9d83b07]
Page Load Metrics (2294 ± 85 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1103411884723
domContentLoaded19152558226517383
load19392572229417785
domInteractive19152558226517383
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@tmashuang tmashuang merged commit 3a38140 into develop Apr 5, 2023
@tmashuang tmashuang deleted the 17191-onboarding-unit-tests-onboarding-app-header branch April 5, 2023 02:59
@github-actions github-actions bot locked and limited conversation to collaborators Apr 5, 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