Add wallet import + project scan and investment cancellation E2E tests#753
Open
dangershony wants to merge 2 commits intomainfrom
Open
Add wallet import + project scan and investment cancellation E2E tests#753dangershony wants to merge 2 commits intomainfrom
dangershony wants to merge 2 commits intomainfrom
Conversation
Implement two new E2E integration tests based on PR #749 test coverage analysis: - WalletImportAndProjectScanTest: validates seed import UI flow, wallet identity determinism (same seed = same WalletId), balance preservation, and ScanFounderProjects discovery after device loss recovery. - InvestmentCancellationTest: validates above-threshold pending investment cancellation, fund release back to investor, and successful re-investment after cancel (CancelInvestmentRequest had zero E2E coverage). Both tests follow the multi-profile WithProfileWindow pattern and include companion markdown documentation.
…cancel AddToPortfolio does an optimistic local insert without WalletId or TransactionId. CancelInvestmentAsync requires both fields. The test now polls LoadInvestmentsFromSdkAsync to wait for the indexer to populate those fields before attempting cancellation.
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.
Summary
CancelInvestmentAsync, fund release back to investor, and successful re-investment after cancel — this path previously had zero E2E coverage.WithProfileWindowpattern and include companion markdown documentation.Test Results
All 9 integration tests pass (including the 2 new ones):
New Files
src/design/App.Test.Integration/WalletImportAndProjectScanTest.cs(~658 lines)src/design/App.Test.Integration/WalletImportAndProjectScanTest.mdsrc/design/App.Test.Integration/InvestmentCancellationTest.cs(~754 lines)src/design/App.Test.Integration/InvestmentCancellationTest.md