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

Deeper Jest integration #585

Merged
merged 18 commits into from
Sep 19, 2023
Merged

Deeper Jest integration #585

merged 18 commits into from
Sep 19, 2023

Conversation

oetr
Copy link
Contributor

@oetr oetr commented Sep 7, 2023

Jazzer.js now supports all Jest features. A nice side-effect of this is that fuzzing of ECMAScript modules is now possible.

@bertschneider bertschneider force-pushed the FUZZ-780_missing_jest_features branch 3 times, most recently from 59a7046 to 32f33ad Compare September 8, 2023 09:24
@bertschneider bertschneider changed the title Fuzz 780 missing jest features Deeper Jest integration Sep 8, 2023
@oetr oetr force-pushed the FUZZ-780_missing_jest_features branch from 6baa277 to c38f656 Compare September 8, 2023 11:28
@bertschneider bertschneider force-pushed the FUZZ-780_missing_jest_features branch 5 times, most recently from b5a3577 to 9f31942 Compare September 14, 2023 11:01
@zgtm zgtm self-requested a review September 14, 2023 14:16
@bertschneider bertschneider force-pushed the FUZZ-780_missing_jest_features branch 8 times, most recently from 0e22acb to d27360e Compare September 15, 2023 07:08
@oetr oetr force-pushed the FUZZ-780_missing_jest_features branch 2 times, most recently from badf1be to 5aff461 Compare September 15, 2023 10:34
@bertschneider bertschneider force-pushed the FUZZ-780_missing_jest_features branch 3 times, most recently from 2f26d2f to 7423083 Compare September 18, 2023 11:10
Copy link
Contributor Author

@oetr oetr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this great PR! I especially like the large number of tests, great job @bertschneider!

docs/jest-integration.md Outdated Show resolved Hide resolved
docs/migration-guide.md Outdated Show resolved Hide resolved
examples/jest_integration/.jazzerjsrc.json Outdated Show resolved Hide resolved
examples/jest_integration/package.json Outdated Show resolved Hide resolved
packages/core/finding.test.ts Outdated Show resolved Hide resolved
packages/jest-runner/corpus.ts Show resolved Hide resolved
packages/jest-runner/errorUtils.ts Show resolved Hide resolved
packages/jest-runner/errorUtils.test.ts Show resolved Hide resolved
tests/helpers.js Show resolved Hide resolved
tests/helpers.js Outdated Show resolved Hide resolved
@bertschneider bertschneider force-pushed the FUZZ-780_missing_jest_features branch 2 times, most recently from 9b00c4b to 8b52e05 Compare September 19, 2023 11:57
bertschneider and others added 2 commits September 19, 2023 14:11
libFuzzer spawns new sub-processes, if the "minimize_crash" parameter is
specified.
Copy link
Member

@zgtm zgtm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a complete review, but a few comments of things I noticed. Nothing blocking this PR.

docs/migration-guide.md Show resolved Hide resolved
examples/jest_integration/target.js Show resolved Hide resolved
examples/protobufjs/protobufjs.fuzz.js Show resolved Hide resolved
examples/protobufjs/readme.md Show resolved Hide resolved
@oetr oetr enabled auto-merge (rebase) September 19, 2023 15:06
@oetr oetr merged commit b642d35 into main Sep 19, 2023
9 checks passed
@oetr oetr deleted the FUZZ-780_missing_jest_features branch September 19, 2023 15:06
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.

None yet

3 participants