-
Notifications
You must be signed in to change notification settings - Fork 25
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
test(e2e): add send and receive test #3760
Conversation
…let into test(e2e)/add-send-test
…let into test(e2e)/add-send-test
…let into test(e2e)/add-send-test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need dockerode
? don't we have enough tools to run a docker image or something similar? and in case not, shouldn't that go among the devDependecies?
<ButtonNext disabled={invalid} testId={SEND_FORM_NEXT_BUTTON} /> | ||
</ButtonGroup> | ||
</ContentWithToolbar> | ||
</form> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this part?
…let into test(e2e)/add-send-test
# Motivation The new end-to-end tests for sending and receiving require a command runner. This command runner allows us to execute scripts that top up a fake balance during testing, enabling us to simulate and verify the associated flows. This pull request introduces that functionality. # Changes Created new command runner. # Tests Tested on PR: #3760 --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Antonio Ventilii <antonio.ventilii@dfinity.org>
Motivation
To further improve testing, we want to check if the receive and send tokens functionality is provided as intended.
Changes
configured a complete test that receives and sends tokens from a wallet.
Tests
tested thoroughly on local replica