Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: startBroadcast and stopBroadcast cheatcodes #201

Merged
merged 7 commits into from
Jan 8, 2024

Conversation

Karrq
Copy link
Contributor

@Karrq Karrq commented Dec 15, 2023

What 💻

  • Add support for startBroadcast and stopBroadcast cheatcodes
  • Related tests

Evidence 📷

image

Notes:

The broadcast tests are a bit of a false positive, the broadcast cheatcodes collect call data to generate transactions that can be broadcasted on chain - thus it has no observable effect in solidity.

@Karrq Karrq changed the title [WIP] startBroadcast and stopBroadcast [WIP] startBroadcast and stopBroadcast cheatcodes Dec 15, 2023
@dutterbutter
Copy link
Collaborator

@MexicanAce this is necessary for truly testing zkforge script cmd fyi.

@Karrq Karrq force-pushed the karrq/feat/cheatcode-ss-broadcast branch from 2321284 to 9f128f0 Compare January 2, 2024 13:14
@Karrq Karrq self-assigned this Jan 4, 2024
@Karrq Karrq marked this pull request as ready for review January 4, 2024 21:48
@Karrq Karrq changed the title [WIP] startBroadcast and stopBroadcast cheatcodes startBroadcast and stopBroadcast cheatcodes Jan 4, 2024
@Karrq Karrq force-pushed the karrq/feat/cheatcode-ss-broadcast branch from 1775058 to ab1ff83 Compare January 4, 2024 22:09
@Karrq Karrq changed the title startBroadcast and stopBroadcast cheatcodes feat(era-cheatcodes): startBroadcast and stopBroadcast cheatcodes Jan 4, 2024
@Jrigada Jrigada changed the title feat(era-cheatcodes): startBroadcast and stopBroadcast cheatcodes feat: startBroadcast and stopBroadcast cheatcodes Jan 5, 2024
@Karrq Karrq force-pushed the karrq/feat/cheatcode-ss-broadcast branch from 751b989 to 50873dc Compare January 5, 2024 21:25
@Karrq
Copy link
Contributor Author

Karrq commented Jan 5, 2024

Rebased on top of main

Karrq added 7 commits January 9, 2024 01:21
refactor(era:cc:broadcast): start/stop broadcast helpers
tests(era::cc:broadcast): simple test file
tests(era:cc:broadcast): add test for value, gas limit and nonce
refactor(era:cc): extract get/set nonce into method
@Karrq Karrq force-pushed the karrq/feat/cheatcode-ss-broadcast branch from 50873dc to 3d43013 Compare January 8, 2024 17:24
@Karrq Karrq merged commit 27d3e6f into main Jan 8, 2024
10 checks passed
@Karrq Karrq deleted the karrq/feat/cheatcode-ss-broadcast branch January 8, 2024 17:44
@Karrq Karrq mentioned this pull request Jan 8, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants