Skip to content

feat(ctb): MPT Fuzzing contract changes#4209

Closed
clabby wants to merge 1 commit intoclabby/ctb/go-fuzzfrom
clabby/ctb/go-fuzz-contract-changes
Closed

feat(ctb): MPT Fuzzing contract changes#4209
clabby wants to merge 1 commit intoclabby/ctb/go-fuzzfrom
clabby/ctb/go-fuzz-contract-changes

Conversation

@clabby
Copy link
Contributor

@clabby clabby commented Dec 5, 2022

Overview

Status: Awaiting decision on whether or not to merge changes in #3771


Adds a fuzz test input generator for testing Bedrock's MerkleTrie.sol.

Go version of #3847

Tests

Adds fuzz tests for MerkleTrie:

  • testFuzz_get_validProofs_success
  • testFuzz_get_invalidRoot_reverts
  • testFuzz_get_extraProofElements_reverts
  • testFuzz_get_invalidLargeInternalHash_reverts
  • testFuzz_get_invalidInternalNodeHash_reverts
  • testFuzz_get_corruptedProof_reverts
  • testFuzz_get_invalidDataRemainder_reverts
  • testFuzz_get_prefixedValidKey_reverts
  • testFuzz_get_emptyKey_reverts - Omitted
  • testFuzz_get_partialProof_reverts

Metadata
ENG-3048

@changeset-bot
Copy link

changeset-bot bot commented Dec 5, 2022

⚠️ No Changeset found

Latest commit: 4c089d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@clabby
Copy link
Contributor Author

clabby commented Dec 5, 2022

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@clabby clabby marked this pull request as draft December 5, 2022 18:55
@clabby clabby force-pushed the clabby/ctb/go-fuzz-contract-changes branch from 91eac3c to 41768cc Compare December 5, 2022 18:59
@clabby clabby force-pushed the clabby/ctb/go-fuzz branch from 8982b84 to 8d9c1bf Compare December 5, 2022 19:05
@clabby clabby force-pushed the clabby/ctb/go-fuzz-contract-changes branch from 41768cc to 118caa3 Compare December 5, 2022 19:05
@clabby clabby force-pushed the clabby/ctb/go-fuzz branch from 8d9c1bf to 5c4dab6 Compare December 5, 2022 19:36
@clabby clabby force-pushed the clabby/ctb/go-fuzz-contract-changes branch 2 times, most recently from fe3bf68 to f058159 Compare December 5, 2022 19:38
@clabby clabby force-pushed the clabby/ctb/go-fuzz-contract-changes branch from f058159 to 4c089d6 Compare December 5, 2022 19:45
@clabby clabby closed this Dec 6, 2022
@clabby clabby deleted the clabby/ctb/go-fuzz-contract-changes branch December 8, 2022 06:31
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.

1 participant