safe-global / safe-client-gateway / 13651052041

89%

push

github

fix: allow confirmation DTOs signatures under `signedSafeTxHash` property (#2423)

When adding [refinement of confirmations](https://github.com/safe-global/safe-client-gateway/pull/2382), we wrongly assumed only web used the confirmation endpoint. As such, we aligned the property name to match that of the Transaction Service.

Since then, we've confirmed that the property was correct. This allows both the new and previous name:

- Allow `signature` and `signedSafeTxHash` property, transforming the latter to `signature`
- Add appropriate test coverage

3116 of 3775 branches covered (82.54%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

10658 of 11460 relevant lines covered (93.0%)

504.95 hits per line