safe-global / safe-client-gateway / 7553297378

89%

push

github

Decode `multiSend` alerts first and update tests to match module transactions (#1027)

This inverts the Alert-decoding logic to first try decoding `multiSend` and then Safe transactions.

Changes:

- `AlertsRepository['_decodeTransactionAdded']` tries to decode the `multiSend` call and then Safe methods either within the batch or if the initial decode fails.
- Each Safe method decode attempt is now handled by `AlertsRepository['_decodeRecoveryTransaction']` which checks that the function name matches that expected, otherwise throwing.
- `alerts.controller.spec` mock transactions have been updated to _not_ wrap `multiSend` recoveries in `execTransaction` calls.

1471 of 1741 branches covered (0.0%)

Branch coverage included in aggregate %.

5593 of 5795 relevant lines covered (96.51%)

170.97 hits per line