fix: avoid gas estimation and add missing test from #15 [APE-1019]

ApeWorX / ape-optimism

Optimism network provider plugin for the Ape Framework

https://www.apeworx.io/

Apache License 2.0

9 stars 9 forks source link

fix: avoid gas estimation and add missing test from #15 [APE-1019] #16

Closed antazoey closed 1 year ago

antazoey commented 1 year ago

What I did

adds missing test from #15 fixes: #12 Fixes: APE-1019

How I did it

How to verify it

Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

linear[bot] commented 1 year ago

APE-485 "`ape.reverts()` fails" (ApeWorX/ape-optimism #12)

### Environment information Latest ape macOS ### What went wrong? When using `ape.reverts()` on a reverted call, you get this error: ``` class 'OverflowError'> occurred: cannot fit 'int' into an index-sized integer ``` ### How can it be fixed? * Figure out the decoding differences and network differences and re-assess. --- [ApeWorX/ape-optimism #12](https://github.com/ApeWorX/ape-optimism/issues/12) by [unparalleled-js](https://github.com/unparalleled-js) on GitHub *via LinearSync*

linear[bot] commented 1 year ago

APE-1019 "test: add missing test from #15" (ApeWorX/ape-optimism #16)

### What I did adds missing test from #15 ### How I did it ### How to verify it ### Checklist * [ ] Passes all linting checks (pre-commit and CI jobs) * [ ] New test cases have been added and are passing * [ ] Documentation has been updated * [ ] PR title follows [Conventional Commit](https://www.conventionalcommits.org/en/v1.0.0/) standard (will be automatically included in the changelog) --- [ApeWorX/ape-optimism #16](https://github.com/ApeWorX/ape-optimism/pull/16) by [antazoey](https://github.com/antazoey) on GitHub *via LinearSync*