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

feat: include echidna fuzz swap tests for native ether and ERC1155 asset #226

Open
4 tasks
0xneves opened this issue May 19, 2024 · 0 comments
Open
4 tasks
Labels
enhancement New feature or request

Comments

@0xneves
Copy link
Member

0xneves commented May 19, 2024

Feature Request

Describe the Feature Request

The PR #218 introduces the tests with ERC1155 and PR #213 introduces unit tests on TestSwapFactory.test.ts and TestSwaplace.test.ts. Such unit tests don't exist in echidna for fuzzing.

Describe Preferred Solution

The files in question are TestSwaplace.sol and TestSwapFactory.sol in echidna folder.

The definition of done:

  • Add tests that utilize native ether swaps sent by the swap owner in the TestSwaplace.sol file.
  • Add tests that utilize native ether swaps sent by the swap aceptee in the TestSwaplace.sol file.
  • Add tests that build ERC1155 assets in the TestSwapFactory.sol file
  • Add tests for swaps that utilize ERC1155 assets in the TestSwaplace.sol file.
@0xneves 0xneves added the enhancement New feature or request label May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🔖 TODO
Development

No branches or pull requests

1 participant