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

Add tests with ton-contract-executor #64

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

AminRezaei0x443
Copy link

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

@AminRezaei0x443
Copy link
Author

AminRezaei0x443 commented Oct 25, 2022

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 AminRezaei0x443 changed the title Add test with ton-contract-executor Add tests with ton-contract-executor Oct 25, 2022
@arseny30
Copy link
Collaborator

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

@AminRezaei0x443
Copy link
Author

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

@11771100
Copy link

11771100 commented Feb 28, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants