WIP: Introduce Fuzz(...) #205
Conversation
Fuzz allows for easy grammar-driven fuzzing via Participle grammars.
| for i := 0; i < items; i++ { | ||
| s.WriteString(g.expr.Fuzz(l)) | ||
| if i < items-1 { | ||
| s.WriteString(" ") |
There was a problem hiding this comment.
This is a really interesting idea, but this implementation makes assumptions about whitespace that will not hold true for all lexers. For example, it is possible to have a lexer that relies on indentation, which this would break on.
I love the idea though, so perhaps there is a way to make it work in the general case.
There was a problem hiding this comment.
This is a really interesting idea, but this implementation makes assumptions about whitespace that will not hold true for all lexers. For example, it is possible to have a lexer that relies on indentation, which this would break on.
Hm, how would you model this? I was trying to keep it stateless, but I'm not sure if I could avoid that and fix this problem.
There was a problem hiding this comment.
Yes, that is definitely the question, and I'm not sure what the answer is :(
|
I'm going to close this, but feel free to send a new PR if you come up with a general purpose solution. |
Fuzz allows for easy grammar-driven fuzzing via Participle grammars.
WIP because I can't figure out why performance is dying in the TestFuzz_LookAhead test, and also the panic("todo")
Fixes #204.