telemint icon indicating copy to clipboard operation
telemint copied to clipboard

Add tests with ton-contract-executor

Open AminRezaei0x443 opened this issue 2 years ago • 5 comments

This PR implements a basis for testing the contracts using ton-contract-executor.

AminRezaei0x443 avatar Oct 25 '22 13:10 AminRezaei0x443

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:

  • Testing is not implemented completely and is just added as a concept (I can complete the test suites later)
  • 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.

AminRezaei0x443 avatar Oct 25 '22 14:10 AminRezaei0x443

Good work, but it is out of the contest's scope.

arseny30 avatar Oct 25 '22 17:10 arseny30

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 :)

AminRezaei0x443 avatar Oct 25 '22 17:10 AminRezaei0x443

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: @.***>

11771100 avatar Feb 28 '23 12:02 11771100