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

e2e tests for axfer, acfg, and afrz inner txns #2947

Merged
merged 10 commits into from
Sep 27, 2021

Conversation

shiqizng
Copy link
Contributor

Summary

This PR adds e2e tests for axfer, acfg, and afrz inner txns.

# rekey $USER to "$APPACCT"
${gcmd} clerk send --from "$USER" --to "$USER" -a 0 --rekey-to "$APPACCT"
# $USER should still have clawback auth. should have been authorized by "$APPACCT"
${gcmd} asset send -f "$SMALL" -t "$USER" -a "1000" --assetid "$ASSETID" --clawback "$USER" && exit 1
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$USER has been rekeyed to $APPACCT. Is there a way to sign this txn with $APPACCT or a different way to validate that $USER still has clawback auth?

@shiqizng shiqizng linked an issue Sep 24, 2021 that may be closed by this pull request
3 tasks
@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2021

Codecov Report

Merging #2947 (ed06379) into master (dbab85f) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2947      +/-   ##
==========================================
+ Coverage   47.29%   47.30%   +0.01%     
==========================================
  Files         356      356              
  Lines       57240    57240              
==========================================
+ Hits        27071    27080       +9     
+ Misses      27103    27098       -5     
+ Partials     3066     3062       -4     
Impacted Files Coverage Δ
util/metrics/gauge.go 68.00% <0.00%> (-2.67%) ⬇️
util/metrics/counter.go 78.57% <0.00%> (-2.39%) ⬇️
network/requestTracker.go 70.25% <0.00%> (-0.87%) ⬇️
catchup/service.go 69.35% <0.00%> (-0.78%) ⬇️
network/wsNetwork.go 60.90% <0.00%> (-0.19%) ⬇️
ledger/acctupdates.go 62.63% <0.00%> (+0.08%) ⬆️
cmd/tealdbg/debugger.go 73.86% <0.00%> (+1.00%) ⬆️
catchup/peerSelector.go 100.00% <0.00%> (+1.04%) ⬆️
crypto/merkletrie/node.go 93.48% <0.00%> (+1.86%) ⬆️
crypto/merkletrie/trie.go 68.61% <0.00%> (+2.18%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dbab85f...ed06379. Read the comment docs.

@algorandskiy algorandskiy changed the title Test/axfer e2e tests for axfer, acfg, and afrz inner txns Sep 24, 2021
@jannotti jannotti merged commit 7991a84 into algorand:master Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

axfer e2e tests
4 participants