fix: return error when failing to decode output in parse_call_result

[aalan3]

Motivation

I ran into this issue when trying to decode the output for a call where the ABI had the incorrect type. I was calling many different ERC20 contracts for symbol() but found a contract that has a different return type, bytes32 instead of the expected string.

Without this change it's hard to understand which call is failing because the entire Multicall returns an error instead of individually for each call.

Solution

parse_call_result now returns Err(bytes) for each call output that fails to decode. It previously just returned an Err immediately from the failing decode function due to ?.

Not sure if it's still ideal to just return bytes since call and call_array will assume it's a revert.

The resulting different behaviour is that instead getting an error for call_raw() you would get an error when parsing the results similar to a regular revert error for example.

PR Checklist

• Added Tests
• Added Documentation
• Breaking changes