search

curvefi / stableswap-ng

Automatic Market Maker (AMM) for 2 or more pegged assets, written in Vyper.
Other
34 stars 15 forks source link

fix: view contracts corrections #45

Closed AlbertoCentonze closed 7 months ago

AlbertoCentonze commented 8 months ago

Implemented the correction suggested in #43 and #44.

For #43 in addition to the suggested correction is_static_call=True had to be used to maintain the view function visibility.

Some simple tests have been added for both cases. I've also started marking tests that were previously broken as expected to fail to lay the foundation for the improvements on the testing infrastructure.