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.

Sign up for GitHub

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

Jump to bottom

Refactor Tupledore Test Cases - Remove Assembly and Add Edge Cases #40

Open

pali101 wants to merge 1 commit into RareSkills:main from pali101:improve/test-tupledore

+39 −41

Contributor

pali101 commented Nov 3, 2024

This PR refactors the test cases for the Tupledore contract by simplifying and enhancing them. Key changes include:

Removed Assembly: Replaced low-level assembly code with direct function calls, making the tests more readable.
Added Edge Case Tests:
- Tested setTuple with address(0).
- Verified behavior with the maximum possible uint256 value.
- Checked initial state to ensure userInfo is uninitialized.
- Added multiple setTuple calls to confirm only the latest values are retained.


          Refactor Tupledore test cases: removed assembly, added edge cases, an…

ed2f68b

…d enhanced assertions for readability

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet