e2e: Added Tests for importing existing wallet configs and validating wallet config's schema#475
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@Legend101Zz Kindly review this draft PR and please suggest any changes to be made if required . |
|
Hi @isid555 , thanks for the PR! I’ve recently opened an issue related to a problem in our E2E test suite: I’m currently working on resolving that. Until the fix is implemented and merged, I’ll hold off on reviewing this PR since the changes from that fix will likely require some adjustments to your code as well. Thanks a lot for your patience, and I appreciate the contribution! |
|
@isid555 Thanks for your patience. Could you please pull in the latest changes and review the updated E2E setup we’ve implemented, then adjust your changes accordingly? |
|
Sure @Legend101Zz , I'll modify and update it accordingly . |
What kind of change does this PR introduce?
Issue Number:
Fixes #442
Importing an existing wallet configuration fileIf relevant, did you update the documentation?
No
Summary
This PR adds automated coverage for importing existing wallet configurations.
Two layers of validation are introduced.
Before touching the UI, the tests assert that wallet configuration files contain all fields required by the application (name, network, address type, client metadata, quorum, and signer information).
This prevents malformed fixtures from producing misleading UI failures and documents the minimum structure the importer relies on.
The fixtures are then uploaded through the wallet import flow and the test verifies that:
Does this PR introduce a breaking change?
No
Checklist
npm run changeset)Other information
All wallet fixtures used by these tests are sourced from:
apps/coordinator/fixtures/caravanHave you read the contributing guide?
Yes
For information on creating and using changesets, please refer to our documentation on changesets.