feat(bedrock): Add input fuzzer for MerkleTrie tests#3847
feat(bedrock): Add input fuzzer for MerkleTrie tests#3847clabby wants to merge 2 commits intosc/ctb-mini-trie-refactorfrom
MerkleTrie tests#3847Conversation
|
Fix linter issues
b55c3e9 to
967fdff
Compare
tynes
left a comment
There was a problem hiding this comment.
Before we can merge this I think we will need to talk during office hours about introducing rust into the codebase. I do think for fuzzing its appropriate, we currently use JS for fuzzing and its really slow. The fuzzing exists in JS currently because we had implementations of primitives in JS but since then, I've implemented a lot of our primitives in go so i was thinking of porting the fuzzing to go.
If we have consensus on adding rust, to merge this we will need to add the rust toolchain to the ci-builder docker image that runs our tests as well as something to the package.json so that CI will run these tests. There is a build:differential command that would be appropriate to include building the fuzzer
packages/contracts-bedrock/scripts/merkle-trie-fuzzer/README.md
Outdated
Show resolved
Hide resolved
|
Closing this in response to our discussion above. |

Overview
Description
Adds an input fuzzer program for bedrock's
MerkleTrietests for use in #3771.Checklist
patriciabinary is built in the CIpackage.jsonTests
Adds the
testFuzzTrie_validProofstest.todo- error case fuzzing
Additional context
n/a
Metadata
n/a