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

[FEA] Improve regexp testing with regexp-aware fuzzer #4330

Closed
andygrove opened this issue Dec 8, 2021 · 0 comments · Fixed by #4504
Closed

[FEA] Improve regexp testing with regexp-aware fuzzer #4330

andygrove opened this issue Dec 8, 2021 · 0 comments · Fixed by #4504
Assignees
Labels
test Only impacts tests

Comments

@andygrove
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The use of fuzzing has been very effective in finding unsupported edge cases in the regular expression support but I would now like to add an additional fuzzing technique where we aim to generate (mostly) valid regular expressions by using a fuzzer to generate a regexp AST made of valid constructs. This will likely catch bugs that we have not yet seen with the existing fuzz tests.

Describe the solution you'd like
Generate RegexAST structures using valid regular expression components.

Describe alternatives you've considered
None

Additional context
None

@andygrove andygrove added feature request New feature or request ? - Needs Triage Need team to review and classify labels Dec 8, 2021
@andygrove andygrove self-assigned this Dec 8, 2021
@Salonijain27 Salonijain27 removed the ? - Needs Triage Need team to review and classify label Dec 14, 2021
@andygrove andygrove added this to the Jan 10 - Jan 28 milestone Jan 7, 2022
@andygrove andygrove added test Only impacts tests and removed feature request New feature or request labels Jan 18, 2022
@andygrove andygrove linked a pull request Jan 24, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Only impacts tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants