search

pendulum-chain / pendulum-solidity-wrapper

A collection of Solidity wrapper contracts emulating common ERC interfaces to grant access to runtime features on Pendulum.
GNU General Public License v3.0
0 stars 0 forks source link

Add TestableERC20Wrapper #34

Closed gianfra-t closed 1 year ago

gianfra-t commented 1 year ago

Added TestableERC20Wrapper contract in order to replace MockERC20 in tests.

Mint/Burn functions are added that will only be called in tests, and that call corresponding chain extensions for minting/burning specified token.

Refer to Issue #26 from wasm-deploy

ebma commented 1 year ago

Is this ready for review? If so, please add some techies as reviewers.

gianfra-t commented 1 year ago

This is not so much of a very big change but the addition of the ERC20 wrapper with just two methods for testing.

gianfra-t commented 1 year ago

Indeed, I have added them to the standalone chain but I couldn't push changes I realize since I don't have access. @ebma if you can could you please give me write access to that repo? I will link those changes here.

ebma commented 1 year ago

Can you try pushing again @gianfra-t? I changed the settings for collaborators.

gianfra-t commented 1 year ago

Thanks! The changes to the chain can be found here.

As part of this changes I have tested and they work as expected.

Perhaps in the future when implementing on foucoco or amplitude, the chain extension definition should be moved to a different file for clarity.