safe-global / safe-client-gateway / 13182186438

89%

Build Type

push

github

Committed by web-flow

Commit Message

fix: validate/coerce multisig `safeTxGas`,`baseGas` and `nonce` values to numbers (#2331)

This adjusts validation of `safeTxGas`,`baseGas` and `nonce`, allowing both string/number and coerces the value to a string to not affect types client-side:

- Abstract previous schema change to [related change on the Transaction Service](https://github.com/safe-global/safe-transaction-service/pull/2367) to `CoercedNumberSchema`
- Use schema for `safeTxGas`, `baseGas` and `nonce` of multisig transactions
- Add appropriate test coverage

Run Details

2939 of 3556 branches covered (82.65%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 3 files covered. (100.0%)

1 existing line in 1 file now uncovered.

9851 of 10528 relevant lines covered (93.57%)

511.19 hits per line