-
Notifications
You must be signed in to change notification settings - Fork 144
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
Implement Amt fuzzer #871
Implement Amt fuzzer #871
Conversation
@@ -22,6 +22,6 @@ criterion = "0.3.1" | |||
ipld_blockstore = { path = "../blockstore", features = ["tracking"] } | |||
|
|||
[[bench]] | |||
name = "amt_beckmark" | |||
name = "amt_benchmark" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
haha beckmark
ipld/amt/fuzz/Cargo.toml
Outdated
[package] | ||
name = "ipld_amt-fuzz" | ||
version = "0.0.0" | ||
authors = ["Automatically generated"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does this mean btw
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just what cargo fuzz generates when creating a target
Summary of changes
Changes introduced in this pull request:
cargo-fuzz
for the AmtWhat this does is fuzz test the usual operations on the Amt, and also performs the same operations on a map to ensure consistent functionality. This does two things: gives much larger confidence there will be no panics encountered with the Amt, and also gives more confidence of the functionality of the Amt in many different cases.
I ran the fuzzer with the go-interop feature on and off, and found no issues. I just wanted to fuzz the Amt and Hamt before cutting releases/ migrating to v2 actors
Reference issue to close (if applicable)
Closes
Other information and links