Skip to content
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

Tests: Integration #119

Open
jrwbabylonlab opened this issue Dec 12, 2024 · 4 comments
Open

Tests: Integration #119

jrwbabylonlab opened this issue Dec 12, 2024 · 4 comments
Assignees

Comments

@jrwbabylonlab
Copy link
Collaborator

jrwbabylonlab commented Dec 12, 2024

  • Set up the tests in the wallet repo
  • The test shall be an integration test in which it will open up the wallet and perform basic operations such as connect the wallet
@gbarkhatov
Copy link
Contributor

@jrwbabylonlab please add details

@gbarkhatov gbarkhatov changed the title Wallet component test skeleton setup Tests: Integration - wallet component test skeleton setup Dec 17, 2024
@gbarkhatov
Copy link
Contributor

I made a scope of this ticket to be integration related - tests that verify the correct interaction between multiple components or modules of the library itself
In this particular case we need to have

  • storybook story where we can connect both wallets
  • see their addresses and pk hexes
  • able to sign the psbt and message from BTC side
  • able to simulate messages via signingStargateClient.simulate from BBN side

@gbarkhatov gbarkhatov changed the title Tests: Integration - wallet component test skeleton setup Tests: Integration Dec 17, 2024
@gbarkhatov
Copy link
Contributor

@jrwbabylonlab should we postpone integration and unit testing #5 so we can focus on the most important - real e2e tests?

@jrwbabylonlab
Copy link
Collaborator Author

@jrwbabylonlab should we postpone integration and unit testing #5 so we can focus on the most important - real e2e tests?

The ticket created was basically the e2e as you understood. I just called it in a different terms. Feel free to close it if it's cleaner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants