5afe / safe-core-protocol / 5936864797 / 1

100%

main: 100%

push

github

Support function handlers (#70)

* [#46] Implement Guard interface in SafeProtocolManager

* [#47] Create FunctionHandlerManager.sol and inherit in SafeProtocolManager

* [#47] Create BaseManager contract, rename modifier, rename error, check registry while adding function handler

* [#47] Update natspec doc

* [#46] Setup Safe

* [#46] Fix EOF

* [#46] User setupTest function

* [#46] Fix lint issue

* [#46] Add tests

* [#46] Add test with delegateCall for hooks flow

* [#46] User temporary variable for storing hooks address

* [#47] Implement logic for non-static calls to function handler manager, test to set function handler

* [#47] Add tests for Function Handler

* [#47] Pass sender address in handle function

* [#47] Fix test

* [#47] Use ZeroAddress from ethers

* [#46] Reset tempHooksAddress

* [#47] Test static call to function handler

* [#47] Fix lint issue

* [#47] Fix typo

* [#46] Refactor tests for SafeProtocolManager as Guard

* [#46] Fix failing test

* [#46] Update comment

* [#47] Update tests for Function Handler

* [#47] Update tests for function handler

* [#47] Remove test function handler from .solcover.js

* [#47] Return data from handle function

* [#47] Verify call data passed to handle(...)

* [#47] Update doc string

* [#47] Check if function handler is whitelisted

* [#47] Make fallback function non-payable, optimize codesize

* [#47] Fix lint issue

* [#47] Add ERC165 check while adding function handler

* [#47] Allow only self calls to manager

* [#47] Add tests for function handler

* [#47] Add tests

* [#47] Rename function to `checkCallerisSender()`

* [#47] Rename to OnlyAccountCallable, add docstrings

* [#47] Update string in describe

* [#47] Update natspec doc for RegistryManager constructor

* [#47] Update natspec doc for FunctionHandlerManager

120 of 120 branches covered (100.0%)

Branch coverage included in aggregate %.

170 of 170 relevant lines covered (100.0%)

13.09 hits per line