algorand / pyteal

Algorand Smart Contracts in Python
https://pyteal.readthedocs.io
MIT License
285 stars 131 forks source link

ABI: Add Router routing graviton tests #394

Closed michaeldiamant closed 1 year ago

michaeldiamant commented 2 years ago

Problem

As of writing, pyteal does not implement graviton tests confirming ABI Router behavior. The story requests closing the gap by:

Related: https://github.com/algorand/graviton/pull/21 adds ABI routing tests to graviton. It's possible to imagine extending these facilities in pyteal.

Solution

Dependencies

Urgency

compiler_test.py offers baseline confidence because generated TEAL will change if a regression is introduced. So, the issue seems less dire than it may initially appear.

tzaffi commented 2 years ago

A couple of tasks that were mentioned in the graviton PR and are relevant to this issue:

tzaffi commented 1 year ago

634 which is a child pr of #600 introduces the RouterSimulation class and tests/integration/abi_router_test.py for testing the router.

I'm closing the issue now as #600 is expected to merged in to master in the next couple of days.