-
Notifications
You must be signed in to change notification settings - Fork 80
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(wallet-tab): include error message when token fetching fails #5308
Conversation
f8b884b
to
cf9d38a
Compare
@@ -713,109 +721,3 @@ describe.each([ | |||
expect(getElementText(tree.getByTestId('TotalTokenBalance'))).toEqual('₱-') | |||
}) | |||
}) | |||
|
|||
describe('renders the network icon on the home screen to differentiate between ETH on Ethereum, Arbitrum, and Optimism', () => { |
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.
these tests were superfluous, as it was testing the underlying behavior of TokenIcon component using HomeTokenBalance, just updated a test in FiatExchangeTokenBalance to ensure TokenIcon is displayed when there's a single token with balance as HomeTokenBalance is going to be removed as part of the drawer nav cleanup
📸 Snapshot TestNo snapshots generated
🛸 Powered by Emerge Tools |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5308 +/- ##
=======================================
Coverage 86.05% 86.05%
=======================================
Files 747 747
Lines 30623 30624 +1
Branches 5308 5308
=======================================
+ Hits 26353 26355 +2
+ Misses 4038 4037 -1
Partials 232 232
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
…lora-inc#5308) ### Description The home token balance component included a toast error message when token fetching fails. This adds the same to the wallet tab. Also added more tests for AssetsTokenBalance (used in the wallet tab) and cleaned up some redundant tests in preparation for removing the home token balance (part of cleaning up the drawer nav) ### Test plan Unit tests, manually failing the token fetch request ### Related issues - Part of ACT-1133 ### Backwards compatibility Yes ### Network scalability If a new NetworkId and/or Network are added in the future, the changes in this PR will: - [x] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)
Description
The home token balance component included a toast error message when token fetching fails. This adds the same to the wallet tab. Also added more tests for AssetsTokenBalance (used in the wallet tab) and cleaned up some redundant tests in preparation for removing the home token balance (part of cleaning up the drawer nav)
Test plan
Unit tests, manually failing the token fetch request
Related issues
Backwards compatibility
Yes
Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will: