fix: testcase for send call message #88
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
v1.6.1
- setAdmin by Owner - setAdmin by unauthorized caller
- setProtocolFee by Owner - setProtocolFee by added admin - setProtocolFee by unauthrorized user
- setProtocolFeeHandler by Owner/Admin - setProtocolFee by unauthrorized user
- sendCallMessage using defaultConnection - sendCallMessage using multiConnection
Closed
3 tasks
…e-sendCallMessage
Base automatically changed from
fix/add-default-connection-setter
to
feature/add-evm-multi-protocol-xcall
September 19, 2023 05:30
…case-sendCallMessage
AntonAndell
reviewed
Sep 19, 2023
AntonAndell
requested changes
Sep 21, 2023
…case-sendCallMessage
…case-sendCallMessage
AntonAndell
approved these changes
Sep 26, 2023
AntonAndell
merged commit d69e2da
into
feature/add-evm-multi-protocol-xcall
2 checks passed
AntonAndell
added a commit
that referenced
this pull request
Oct 23, 2023
* feat: setup foundry structure (#79) * remove msg encoding in encodeCSMessageResponse * chore: forge init * forge install: forge-std v1.6.1 * initial foundry setup * forge install: openzeppelin-contracts-upgradeable v4.9.3 * forge install: openzeppelin-contracts v4.9.3 * add contract setup script for Callservice for Xcall * add initial test setup for Xcall * feat: testcase set admin (#80) * add setAdmin testCases - setAdmin by Owner - setAdmin by unauthorized caller --------- Co-authored-by: AntonAndell <[email protected]> * feat: testcase set protocol fee (#81) * add setProtocolFee and getProtocolFee testCases - setProtocolFee by Owner - setProtocolFee by added admin - setProtocolFee by unauthrorized user --------- Co-authored-by: AntonAndell <[email protected]> * feat: testcase set protocol fee handler (#82) * add setProtocolFeeHandler and getProtocolFeeHandler testCases - setProtocolFeeHandler by Owner/Admin - setProtocolFee by unauthrorized user --------- Co-authored-by: AntonAndell <[email protected]> * fix: add default connection setter (#87) * add defaultConnection check on sendCallMessage * add defaultConnection setter for xCall * add revert messages on require() * fix: rlp encode library encoding fix for protocols (#89) * update encodeCSMessageRequest with protocol encoded code * feat: handle requests multiple testcases (#90) * add defaultConnection check on sendCallMessage * add defaultConnection setter for xCall * add revert messages on require() * - add CallMessage eventLog for assertion - Add Connection instances - Add required const variables for netTo for NID and dstAccount for addr * add testCase for HandleResponse - DefaultProtocol with access - DefaultProtocol with unauthorized access * TestCases - handleResponse with singleProtocol - handleResponse with singleProtocol by unauthorized * TestCases - handleResponse with MultiProtocol --------- Co-authored-by: AntonAndell <[email protected]> * add successfulResponses record for responses * update successfulResponses variable after response is success * feat: execute call testcases (#91) * TestCases - executeCall using singleProtocol * Add `CallExecuted` eventlogs for assertions * TestCases - executeCall using defaultProtocol * TestCases - executeCall using multiProtocols * fix: clearing out unused code (#98) * fix: clearing out unused code * fix: clearing out contract part in library --------- Co-authored-by: AntonAndell <[email protected]> * docs: solidity xcall deployment (#99) * update solidity document to deploy tthe contract as well as getting ready with forge commands * update solidity document to deploy tthe contract as well as getting ready with forge commands * feat: rollback testcases (#92) * TestCases - Rollback using singleProtocol * TestCases - Rollback using DefaultProtocol - Rollback using DefaultProtocol NotAuthorized * TestCases - Rollback using MultiProtocol - Rollback with SuccessMessage * fix rollbackSuccess testcase to remove rollbackMessage eventlog * remove rollbackMessage eventlogs from executeCall() * feat: execute rollback testcases (#93) * - Fix variables - Fix Imports - TestCases - ExecuteRollback using SingleProtocol - ExecuteRollback with DefaultProtocol - ExecuteRollback using MultiProtocol * fix executeRollack testcases * fix: testcase for send call message (#88) * fix sendCallMessageSingleProtocol testCase * add `CallMessgeSent` eventLog for test * add testCase for sendMessage using multiProtocol * add testCase for sendMessage using defaultProtocol * Fix expectCall on sendMessage TestCases --------- Co-authored-by: AntonAndell <[email protected]> * fix: deployment script updated (#100) * fix: deployment script updated * fix: foundry.toml * fix: nid taken from env * update deployment script to add the setup params for xcall-multi * update deploy script on shellScript file to deploy the contracts easily from CLI * add .env example file to follow to deploy the xcall --------- Co-authored-by: sdpbhandari <[email protected]> * fix: add verify success test cases (#101) * fix imports error after merge * Fix verifySuccess() Add assert on verifySuccess for rollback tests Add assert on verifySuccess for executeRollback tests * tests: Fix testExecuteRollbackMultiProtocol test * feat: ci add solidity code coverages (#140) * ci: add solidity coverage workflow * ci: test codcov workflow --------- Co-authored-by: Deepak Bomjan <[email protected]> * refactor: evm folder structure * fix: solidity xcall cod coverage * feat: added evm multiprotocol dapp * fix: removed unnecessary imports * fix: remove cache file * fix: add cache to gitignore * fix: Add full expects of sendMessage to connections * Ignore libraries in solidity code coverage --------- Co-authored-by: Sudeep Bhandari <[email protected]> Co-authored-by: gcranju <[email protected]> Co-authored-by: Deepak Bomjan <[email protected]> Co-authored-by: red__larva <[email protected]> Co-authored-by: qwerty0789 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Please check the commit messages
see the guidelines for commit messages.
Changelog Entry
Checklist: