feat: simulate with mode

[emidev98]

Description

This pull request puts the Simulate and Broadcast transaction closer to parity. When a transaction fails on-chain it will anyways return the gas used. Same as developed in this PR to return the gas used but in order to avoid breaking changes I have decided to add a new field named mode which by default it behaves as know until now, but when the mode is set to GAS_USED it will return also the amount of gas used with the status code 200 even if the simulation fails.

Author Checklist

I have...

• included the correct type prefix in the PR title
• added ! to the type prefix if API or client breaking change
• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• followed the guidelines for building modules
• included the necessary unit and integration tests
• added a changelog entry to CHANGELOG.md
• included comments for documenting Go code
• updated the relevant documentation or specification
• reviewed "Files changed" and left comments if necessary
• confirmed all CI checks have passed

Reviewers Checklist

I have...

• confirmed the correct type prefix in the PR title
• confirmed ! in the type prefix if API or client breaking change
• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

model GAS_USED

model WITH_RESULT

model undefined

[alexanderbez]

Hey @emidev98 thanks for the contribution!

To be honest, it wasn't clear to me from the title or the description what this PR is actually for...I'd suggest tweaking that a bit.

However, after looking at the changes I see what you're trying to achieve and why. However, I don't think it's necessary. At least there is a somewhat easier way of going about it. What I'd suggest is actually adding context to the err itself. Because I believe it's still useful to return an error -- so let's just add context to it. WDYT?

[emidev98]

@alexanderbez I close this PR since I have create this new one with the only changes you suggested #13435

[alexanderbez]

Thanks for the contribution @emidev98! Amazing work 👏