Skip to content

Test Suite for TapiocaZ Contracts #239

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
Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

Test Suite for TapiocaZ Contracts #239

wants to merge 49 commits into from

Conversation

0xRaz-b
Copy link

@0xRaz-b 0xRaz-b commented Aug 2, 2024

This PR introduces an extensive test suite for the TapiocaZ contracts, significantly enhancing the code quality and reliability. The new tests cover components of the cross-chain token system, including BaseTOFT, mTOFT, ModuleManager, and TOFTGenericReceiverModule.

Key additions:

  1. BaseTOFT Tests:

    • Administrative functions (pause/unpause)
    • Wrapping and unwrapping operations
    • Error handling for edge cases

  2. mTOFT Tests:

    • Cross-chain token transfers
    • Fee handling mechanisms
    • Integration with StargateRouter and LayerZero endpoints
    • Access control and administrative functions

  3. ModuleManager Tests:

    • Module setting, extraction, and execution
    • Access control for module operations

  4. TOFTGenericReceiverModule Tests:

    • Token receiving and unwrapping
    • Balance updates and transfer verifications

These tests ensure proper functionality across various scenarios, including success cases and expected failure conditions. Helper functions were added to streamline the testing process and improve test readability.

0xRaz-b added 30 commits August 2, 2024 11:47
@0xRaz-b
imports necessary contracts and libraries
9e2fc68
@0xRaz-b
setup contracts with two differents chains. Chain1 and Chain2
7b2e510
@0xRaz-b
Add unit tests for send function
a23af06
@0xRaz-b
Add unit tests for sendPacket function
a43e296
@0xRaz-b
Add unit tests for sendPacket function failure scenarios
76a8da3
@0xRaz-b
Add unit tests for getTypedDataHash function
6128a5a
@0xRaz-b
Add unit tests for wrap function failure scenarios
e4d09e1
@0xRaz-b
Add unit tests for wrap function
e754a40
@0xRaz-b
Add unit tests for unwrap function failure scenarios
3d81339
@0xRaz-b
Fix missing closing brace in unit test
4fdf501
@0xRaz-b
Fix missing closing brace in unit test
e9f92c4
@0xRaz-b
Add unit test for successful unwrap in mTOFTChain1
045c0c9
@0xRaz-b
Add uint tests for sgReceive when it's not called by stargateRouter
806fc3c
@0xRaz-b
Add unit tests for sgReceive functionwhen called by stargateRouter
8ac0d4d
@0xRaz-b
Add unit tests for rescuing eth by none owner
b29dbcf
@0xRaz-b
unit test for rescue eth by owner
6fdc153
@0xRaz-b
unit test for the owner modifier
e1c64a0
@0xRaz-b
Add unit tests for extracting underlying function failure scenarios
1f4668e
@0xRaz-b
ensure extract underlying work properly
2ad983b
@0xRaz-b
testing wrap and unwrap fees registration
819af20
@0xRaz-b
test receive function
f47bcd0
@0xRaz-b
add helper functions
8ff7807
@0xRaz-b
test withdrawing fees
f162461
@0xRaz-b
test revert of setting owner state for setting a new mint cap smaller…
5bdd117 
… thatn total supply
@0xRaz-b
Add unit test for sendPacketFrom success scenario in mTOFTChain1
2b85a7e
@0xRaz-b
import libraries and contracts
cb89dee
@0xRaz-b
setup a baseTOFT contract
c5bf61f
@0xRaz-b
test admin capabilities to pause and assert that a random user cannot…
bee8dcb 
… pause or unpause
@0xRaz-b
unit test for the funciton wrap working perfectly
e553855
@0xRaz-b
testing failure scenario for the funciton wrap
c5d9e04
0xRaz-b added 19 commits August 2, 2024 12:34
@0xRaz-b
assert that wrapNative is working as inteded
1038569
@0xRaz-b
unit test for unwrap function from TOFT to ERC20
b1577ea
@0xRaz-b
adding helper function to setApprovals, create TOFTInitstruct and set…
3b2efae 
…VaultOwnership
@0xRaz-b
add BaseTOFTMock to test internal function for BaseTOFT and also modu…
2913cf0 
…leManager
@0xRaz-b
Remove unused error TOFT_VaultWrongOwner
817c945
@0xRaz-b
create a new error that suits the case when the caller should be bala…
eba194e 
…ncer
@0xRaz-b
import contracts and using BaseTOFTTest setup
daacf89
@0xRaz-b
test for setting a module
6760648
@0xRaz-b
test for extracting the module address
fdb369c
@0xRaz-b
Testing extraction of non-existent module
2018862
@0xRaz-b
Test executing a module
713f34e
@0xRaz-b
Helper function to white liste modules
bef7275
@0xRaz-b
Setup for GenericReceiverModule
85e2dc7
@0xRaz-b
Testing receiver with param function in the module
7a775b5
@0xRaz-b
Testing a failure scenario of receiveWithParamsReceiver when unwrap i…
6e73cc9 
…s false and there is a msg.value
@0xRaz-b
Add MockMtoftFeeGetter to mTOFTTest to enable calling SetOwnerStateDa…
6cb68f4 
…ta and align testing closely with production.
@0xRaz-b
Mock used to test TOFTGenericReceiverModule
a148a0e
@0xRaz-b
Adding a mint function to have an easier alternative to get TOFT inst…
e8bd844 
…ead of wrapping
@0xRaz-b
While test coverage is not working, here is a document resuming all t…
b83e399 
…ests done
@graphite-app graphite-app bot requested review from 0xRektora and maarcweiss August 2, 2024 19:47
@graphite-app Graphite App
Copy link

graphite-app bot commented Aug 2, 2024

Graphite Automations

"Add reviewers" took an action on this PR • (08/02/24)

2 reviewers were added to this PR based on 0xRektora's automation.

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

@0xRektora 0xRektora Awaiting requested review from 0xRektora

@maarcweiss maarcweiss Awaiting requested review from maarcweiss

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@0xRaz-b