safe-global / safe-client-gateway / 14132067830

89%

push

github

fix: remove excessive transaction verification checks (#2510)

On top of the transaction integrity/validity checks, we ensure there are no duplicate owners/signatures. As we inherently verify each signature, these checks are unnecessary. Their respective values would be validated. As we have also seen no logs for the above as well, this removes the respective duplicate checks (and the associated tests):

- Remove duplicate checks from `TransactionVerifier`
- Remove associated tests

3201 of 3837 branches covered (83.42%)

Branch coverage included in aggregate %.

10964 of 11818 relevant lines covered (92.77%)

534.86 hits per line