Skip to content
This repository has been archived by the owner on May 9, 2024. It is now read-only.

Add simulate command to CLI #76

Merged
merged 26 commits into from
Aug 26, 2021
Merged

Add simulate command to CLI #76

merged 26 commits into from
Aug 26, 2021

Conversation

waymobetta
Copy link
Contributor

@waymobetta waymobetta commented Aug 24, 2021

Description

The simulate subcommand allows the user to pass some data related to a previously failed transaction (txhash, block, from address) which will take this data, construct a message, and then call the contract with this message data. The command will return the contract results, but nothing is published or broadcasted on chain; this is not state altering.

TODO:

  • initial
  • refactor (may want to be a standalone command instead of a flag)

Related Issue Or Context

Closes: #75

How Has This Been Tested? Testing details.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation

Checklist:

  • I have commented my code, particularly in hard-to-understand areas.
  • I have ensured that all acceptance criteria (or expected behavior) from issue are met
  • I have updated the documentation locally and in chainbridge-docs.
  • I have added tests to cover my changes.
  • I have ensured that all the checks are passing and green, I've signed the CLA bot

@waymobetta waymobetta changed the title Add simulate flag to CLI Add simulate command to CLI Aug 24, 2021
@waymobetta waymobetta marked this pull request as ready for review August 24, 2021 16:26
@waymobetta waymobetta requested a review from P1sar August 26, 2021 13:21
@P1sar P1sar merged commit 7239393 into main Aug 26, 2021
@P1sar P1sar deleted the waymobetta/cli-simulate-flag branch August 26, 2021 13:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add simulate CLI utility command for debugging
2 participants