test(mobile): add e2e coverage for wallet 4.0 asset discoverability#18760
Draft
mcayuelas-ledger wants to merge 1 commit into
Draft
test(mobile): add e2e coverage for wallet 4.0 asset discoverability#18760mcayuelas-ledger wants to merge 1 commit into
mcayuelas-ledger wants to merge 1 commit into
Conversation
Wiz Scan Summary
To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension. |
Contributor
Web Tools Build Status
|
mcayuelas-ledger
force-pushed
the
support/live-32761-e2e-asset-discoverability-mobile
branch
from
eae4755 to
1c98aa4
Compare
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✅ Checklist
npx changesetwas attached.assetDiscoverabilityis now enabled by default in the sharedWALLET_40_FEATURE_FLAGSe2e constant — verify other wallet40 specs are unaffected📝 Description
Wallet 4.0 introduces the "asset discoverability" experience (stocks discovery/holdings sections on the portfolio and category-aware global search), but it had no end-to-end coverage.
This PR adds Detox E2E specs under
e2e/mobile/specs/wallet40/assetDiscoverability.spec.ts, with supporting page objects:TopBarSearchPagefor the global search screen (categories, search input, results).MarketPagehelpers to assert the selected market category and to go back.PortfolioPagehelpers for the stocks discovery/holdings sections and the full stocks list page.portfolioWithManyStocksuserdata fixture for the holdings scenario.The shared
WALLET_40_FEATURE_FLAGSconstant now enablesassetDiscoverabilityby default; the suite also sets it explicitly so it stays self-describing about the flag it relies on. Synchronization is disabled for the suite because the Wallet 4.0 market & global-search screens run continuous animations (Detox never reaches idle on iOS).Scenarios (TMS):
B2CQA-5955(empty stocks discovery),B2CQA-5956(stocks holdings),B2CQA-5957(search categories),B2CQA-5958(search ranking).❓ Context
🧐 Checklist for the PR Reviewers