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

SPIKE: Wallet Automated Testing #749

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Funkatronics
Copy link
Contributor

@Funkatronics Funkatronics commented Mar 13, 2024

Experimental testing script to test fakedapp against Phantom

setup:

  • setup a phone or emulator to run tests from Android Studio
  • manually install Phantom apk on device
  • ensure phantom is setup with a wallet
  • ensure phantom is set to test/dev mode with the solana testnet cluster selected

to run:
run the tests in the MainActivityTestsPhantom file in fakedapps test package

TODOs:

  • need to investigate why Phantom is returning invalid signatures with this script
  • investigate mainnet/testnet discrepancy mention above
  • rethink the approach here. Should we be using fakedapp? it might be better to simulate/mock the client side like we do in the fakewallet tests

@Funkatronics
Copy link
Contributor Author

Funkatronics commented Mar 13, 2024

NOTE: CI should fail if it tries to run these tests. they currently fail due to an invalid signature returned from Phantom (using latest version from dapp store) more importantly the CI does not install phantom on the test device lol

@Funkatronics
Copy link
Contributor Author

Funkatronics commented Mar 18, 2024

tests here should start working locally once #752 is merged.

@Funkatronics
Copy link
Contributor Author

tests are passing locally with Phantom setup on an emulator. CI needs to be adjusted to only run the fakewallet tests and ignore these phantoms tests for now.

@Funkatronics Funkatronics changed the title [WIP] SPIKE: Wallet Automated Testing SPIKE: Wallet Automated Testing Mar 18, 2024
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

Successfully merging this pull request may close these issues.

1 participant