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.

Sign up for GitHub

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

Jump to bottom

Add contract call test scripts #48

Merged
merged 23 commits into from
Dec 19, 2024
Merged

Add contract call test scripts #48

merged 23 commits into from
Dec 19, 2024

Conversation

sskeirik
Copy link

@sskeirik sskeirik commented Dec 18, 2024 •
edited
Loading

This PR does several things:

  1. splits out the fund account code from the deploy contract code
  2. adds a call contract script
  3. adds a random account creation script
  4. updates the README to use the new scripts
  5. updates scripts to only emit extra debug info on error (quiet on success)
  6. added a bash test script erc20_test.sh

sskeirik
sskeirik commented Dec 18, 2024
View reviewed changes
pykwasm/src/pykwasm/call.py Outdated Show resolved Hide resolved
@sskeirik sskeirik force-pushed the erc20_test branch 3 times, most recently from 84d9698 to 69d1db7 Compare December 18, 2024 19:33
@sskeirik sskeirik marked this pull request as ready for review December 19, 2024 15:43
@sskeirik sskeirik requested a review from traiansf December 19, 2024 15:43
virgil-serbanuta
virgil-serbanuta approved these changes Dec 19, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
traiansf and others added 4 commits December 19, 2024 11:18
@sskeirik sskeirik merged commit 1bb4927 into master Dec 19, 2024
2 checks passed
@sskeirik sskeirik deleted the erc20_test branch December 19, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@virgil-serbanuta virgil-serbanuta virgil-serbanuta approved these changes

@traiansf traiansf Awaiting requested review from traiansf

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sskeirik @virgil-serbanuta @traiansf