Open AminRezaei0x443 opened 2 years ago
Testing properly is essential for such a productional smart contracts and all methods should be battle-tested. We can achieve this easily using TVM locally to ensure appropriate behavior of functions under all circumstances. Few notes:
ton-contract-executor
needed some patches to compile files using the new ton-compiler
so I forked and made a PR. Until the merge one can use https://github.com/AminRezaei0x443/ton-contract-executor to run tests.Good work, but it is out of the contest's scope.
Thanks, I agree. However, increasing test coverage is essential to find bugs that are not simply visible to the eye. Also, the contracts didn't have major security issues. That was the reason I turned to write tests but had less time to complete them :)
Good
في الثلاثاء، ٢٨ فبراير، ٢٠٢٣ ٢:٣٢ ص hehe2112 @.***> كتب:
@.**** approved this pull request.
— Reply to this email directly, view it on GitHub https://github.com/TelegramMessenger/telemint/pull/64#pullrequestreview-1316634755, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3YP6XZNXUOXBHSSP2VCWDTWZVBSHANCNFSM6AAAAAAROAEH6E . You are receiving this because you are subscribed to this thread.Message ID: @.***>
This PR implements a basis for testing the contracts using
ton-contract-executor
.