6529-Collections / nftdelegation

Delegation contract
MIT License
16 stars 7 forks source link

An approach to automated tests? #24

Closed brookr closed 1 year ago

brookr commented 1 year ago

Would be great to see some automated tests covering the core functionality of the contract.

Is there a plan for tests? There is a PR that proposes some tests, but it would be great to know if this is the preferred tooling. Once we settle on how tests can/should be added, we can go to town on getting different edge cases tested.

rookmate commented 1 year ago

I do like foundry toolset for tests. The test section PR #3 would definitely be a good addition to this repo

rookmate commented 1 year ago

I see that PR #31 is suggesting Hardhat as the go to for automated testing. Personally, I think foundry is a much better option to go with