safe-global / safe-client-gateway / 12870243599 / 1

89%

main: 89%

push

github

Deprecate `executedSurplusFee`, adding `executedFee` and `executedFeeToken` (#2247)

Deprecates `executedSurplusFee`, adding `executedFee` and `executedFeeToken` to `Order`s in its place:

- Add `executedFee`/`executedFeeToken` to `OrderSchema`.
- Add fields to relevant swap/TWAP entities, removing wrong nullable properties.
- Map new fields in respective swap/TWAP mappers.
- Update builder to use same values for `executedSurplusFee`<>`executedFee` and `sellToken`<>`executedFeeToken`.
- Update tests accordingly.

Eventually, the `executedFeeToken` will also need updating as it will will be expressed in SURPLUS tokens (BUY tokens for SELL orders and SELL tokens for BUY orders). For now, it remains the `sellToken`.

2985 of 3625 branches covered (82.34%)

Branch coverage included in aggregate %.

19451 of 20737 relevant lines covered (93.8%)

232.41 hits per line