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

Dryrun printer utility #162

Merged
merged 3 commits into from
May 4, 2022
Merged

Dryrun printer utility #162

merged 3 commits into from
May 4, 2022

Conversation

ivnsch
Copy link
Contributor

@ivnsch ivnsch commented Apr 14, 2022

Ported: algorand/py-algorand-sdk#283

Example call:

let req = create_dryrun(algod, &signed_txs).await?;
let res = algod.dryrun_teal(&req).await?;
for tx in res.txns {
    println!("{}", app_trace(&tx)?);
}

Output:
Screenshot 2022-04-14 at 22 15 11

Added new settings to specify individual columns width and display byte values as addresses.

@ivnsch ivnsch force-pushed the dryrun_printer branch 4 times, most recently from 1290cc5 to 17d18ba Compare April 15, 2022 08:59
Base automatically changed from fix-dryrun-endpoint to main April 15, 2022 09:59
@ivnsch ivnsch force-pushed the dryrun_printer branch 2 times, most recently from 8695086 to 152deb1 Compare April 15, 2022 10:30
@ivnsch
Copy link
Contributor Author

ivnsch commented Apr 24, 2022

Noting that this is partly blocked by algorand/go-algorand#3892 - algod returns a panic error message for some request formats. I updated the serialization locally to not send zero fields etc., but still getting the error and don't have more time to fiddle with this. I was able to reproduce it with the Java SDK for some of the requests.

@ivnsch
Copy link
Contributor Author

ivnsch commented May 4, 2022

I'll merge this so it doesn't possibly get out of sync, with the warning that making dry run requests mostly doesn't work. The issue is unlikely to relate directly to this PR, this is not critical functionality, and it's unclear when/if the node issue will be addressed. This most likely will fix itself when implementing the dry run cucumber tests and revising the overall serialization.

@ivnsch ivnsch merged commit 30a251e into main May 4, 2022
@ivnsch ivnsch deleted the dryrun_printer branch May 4, 2022 17:38
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