-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
feat: refactor e2e wallet rendered helper function #25352
Conversation
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. |
52e004b
to
cb3e0e7
Compare
e7dd602
to
4fc939b
Compare
text: `${balance} ETH`, | ||
}); | ||
const balanceSelector = '[data-testid="eth-overview__primary-currency"]'; | ||
if (ganacheServer) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are situations there the account is not having the balance allocated. eg: we create 2nd account. So to re-use the selector logic, I combined this with waitForAccountRendered
downstairs
d6b6aa5
to
5d22c55
Compare
a244190
to
434ac90
Compare
434ac90
to
008851b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #25352 +/- ##
========================================
Coverage 65.38% 65.38%
========================================
Files 1382 1382
Lines 54767 54767
Branches 14368 14368
========================================
Hits 35809 35809
Misses 18958 18958 ☔ View full report in Codecov by Sentry. |
Builds ready [008851b]
Page Load Metrics (57 ± 17 ms)
Bundle size diffs
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Refactor some e2e tests to use new helper function
logInWithBalanceValidation
orlocateAccountBalanceDOM
and remove previous functionwaitForAccountRendered
. The former also two involves balance check from ganache server, we can migrate all to those and remove the DOM only check helper function.Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist