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

Create Password onboarding unit test, test-ids, and snapshot #18257

Merged
merged 28 commits into from
Apr 20, 2023

Conversation

tmashuang
Copy link
Contributor

Explanation

Create Password onboarding unit test, test-ids, 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:48
@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 [bfa98c1]
Page Load Metrics (1548 ± 54 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint922881204019
domContentLoaded13621858154011254
load13621858154811254
domInteractive13621858154011254
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #18257 (ce3640d) into develop (f92e463) will increase coverage by 0.06%.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##           develop   #18257      +/-   ##
===========================================
+ Coverage    65.15%   65.21%   +0.06%     
===========================================
  Files          936      936              
  Lines        35965    35963       -2     
  Branches      9231     9231              
===========================================
+ Hits         23432    23451      +19     
+ Misses       12533    12512      -21     
Impacted Files Coverage Δ
...onboarding-flow/create-password/create-password.js 95.51% <ø> (+22.47%) ⬆️

... and 6 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 [c3c37df]
Page Load Metrics (1867 ± 85 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1013131454622
domContentLoaded15142141184618388
load15272152186717785
domInteractive15142140184618388
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [e689e08]
Page Load Metrics (1765 ± 88 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1073371394723
domContentLoaded14872057173515776
load14912182176518388
domInteractive14872057173415776
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [4afc859]
Page Load Metrics (1716 ± 71 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint98178124209
domContentLoaded15222119170614369
load15222120171614771
domInteractive15222119170614369
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [764559f]
Page Load Metrics (1618 ± 38 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint95164117168
domContentLoaded1460168215966832
load1472176216187938
domInteractive1460168115966832
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [090933d]
Page Load Metrics (1537 ± 29 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint893061144522
domContentLoaded1401163815186129
load1435163815376029
domInteractive1401163815186129
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [3c0e16c]
Page Load Metrics (1555 ± 31 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint90138107115
domContentLoaded1395161915335326
load1395167115556531
domInteractive1395161915335326
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [d3172d4]
Page Load Metrics (1711 ± 84 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint862791314019
domContentLoaded14702093169317082
load14702093171117584
domInteractive14702093169317082
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [cd9693b]
Page Load Metrics (1544 ± 49 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint90145107147
domContentLoaded1344175315309948
load13611753154410149
domInteractive1344175315309948
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [ce3640d]
Page Load Metrics (1559 ± 33 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint89136108136
domContentLoaded1399166015486632
load1400166015596833
domInteractive1399166015486632
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [6be39d7]
Page Load Metrics (1585 ± 35 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint903871256230
domContentLoaded1419171115677134
load1419171115857335
domInteractive1419171115677134

@metamaskbot
Copy link
Collaborator

Builds ready [a6a049b]
Page Load Metrics (1724 ± 59 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint981921222411
domContentLoaded14291979170712258
load14411979172412359
domInteractive14291979170712258
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [85c5648]
Page Load Metrics (1628 ± 47 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint952051242512
domContentLoaded1440173916148441
load1459182716289747
domInteractive1440173916148441
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [a292366]
Page Load Metrics (1594 ± 66 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint921941232613
domContentLoaded14151904157813163
load14151929159413866
domInteractive14151904157813163
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [1cb6a4e]
Page Load Metrics (1620 ± 44 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint92148116168
domContentLoaded1475174216068541
load1476182016209144
domInteractive1475174216068541
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [a680228]
Page Load Metrics (1746 ± 113 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint105169132199
domContentLoaded148621801735239115
load149121801746235113
domInteractive148621801734239115
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [1528f97]
Page Load Metrics (1478 ± 28 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint87128104115
domContentLoaded1366161714706029
load1393162614785828
domInteractive1366161714706029
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [f7c84a0]
Page Load Metrics (1578 ± 34 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint891741132010
domContentLoaded1447170215586732
load1447178315787134
domInteractive1447170215586732
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

@brad-decker brad-decker merged commit 8d29fc9 into develop Apr 20, 2023
@brad-decker brad-decker deleted the 17191-onboarding-unit-tests-create-password branch April 20, 2023 18:33
@github-actions github-actions bot locked and limited conversation to collaborators Apr 20, 2023
@metamaskbot
Copy link
Collaborator

Builds ready [545cd08]
Page Load Metrics (1589 ± 39 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint90155112147
domContentLoaded1423176715739144
load1441176815898139
domInteractive1422176715739144
Bundle size diffs
  • background: 0 bytes
  • ui: 177 bytes
  • common: 0 bytes

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.

6 participants