besu icon indicating copy to clipboard operation
besu copied to clipboard

#7608: Fix evmtool JSON Error Field to Return Hex String Instead of Long Null

Open itfat opened this issue 1 year ago • 0 comments

PR description

Modified the logic in evmtool to check if a revert reason is present and convert it to a hex string. Modified revertReason to hex string in EvmToolCommand and EvmToyCommand as well.

Fixed Issue(s)

fixes #7608

Thanks for sending a pull request! Have you done the following?

  • [ ] Checked out our contribution guidelines?
  • [ ] Considered documentation and added the doc-change-required label to this PR if updates are required.
  • [ ] Considered the changelog and included an update if required.
  • [ ] For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • [ ] unit tests: ./gradlew build
  • [ ] acceptance tests: ./gradlew acceptanceTest
  • [ ] integration tests: ./gradlew integrationTest
  • [ ] reference tests: ./gradlew ethereum:referenceTests:referenceTests

itfat avatar Oct 15 '24 19:10 itfat