fix: parsing of non-string event arguments from simulation

[matthewwalsh0]

Explanation

Update the simulation data parsing logic to avoid failed simulations if an event includes a non-string argument.

For example, the ApprovalForAll event from the ERC-1155 ABI, which includes the boolean approved argument.

Specifically, the changes are:

• Only attempt to call toLowerCase on string event arguments.
• Skip the parsing of unsupported events.
• Rename the parseX methods to normalizeX for clarity.

References

[Pending]

Changelog

@metamask/transaction-controller

• FIXED: Simulation of transactions creating ApprovalForAll events.

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've highlighted breaking changes using the "BREAKING" category above as appropriate