-
Notifications
You must be signed in to change notification settings - Fork 492
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
fix flaky axfer txn e2e test #2973
Conversation
test/scripts/e2e_subs/app-assets.sh
Outdated
|
||
ASSETID=$(asset-create 1000000 --name "e2e" --unitname "e" | asset-id) | ||
[ "$(balance "$SMALL")" = 997000 ] # 1000 fee | ||
[ "$(balance "$SMALL")" = 996000 ] # 1000 fee | ||
|
||
${gcmd} clerk send -a 1000000 -f "$ACCOUNT" -t "$APPACCT" |
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.
this one is still 1 Algo
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.
fixed. thanks.
Codecov Report
@@ Coverage Diff @@
## master #2973 +/- ##
==========================================
- Coverage 47.61% 47.60% -0.01%
==========================================
Files 358 358
Lines 57906 57906
==========================================
- Hits 27570 27566 -4
- Misses 27208 27213 +5
+ Partials 3128 3127 -1
Continue to review full report at Codecov.
|
Summary
The account balance check sometimes fails. This is because accounts with more than 1 algo are eligible to receive rewards and it varies when rewards would be added to an account.
This PR fixes this issue by funding accounts less than 1 algo.