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

Jazzer.js now supports coverage report generation. Simply add "--coverage". #272

Merged
merged 4 commits into from
Jan 26, 2023

Conversation

oetr
Copy link
Contributor

@oetr oetr commented Jan 12, 2023

No description provided.

@oetr oetr force-pushed the FUZZ-372-coverage-reports branch 12 times, most recently from 16809cd to 5de038e Compare January 20, 2023 09:52
@oetr oetr marked this pull request as ready for review January 20, 2023 10:08
@oetr oetr requested a review from a team January 20, 2023 10:09
.config/test-jazzerjsrc.json Show resolved Hide resolved
.github/workflows/run-all-tests.yaml Outdated Show resolved Hide resolved
tests/code_coverage/package.json Outdated Show resolved Hide resolved
packages/core/cli.ts Show resolved Hide resolved
packages/core/cli.ts Show resolved Hide resolved
docs/jest-integration.md Outdated Show resolved Hide resolved
docs/jest-integration.md Show resolved Hide resolved
packages/core/core.ts Outdated Show resolved Hide resolved
packages/instrumentor/instrument.ts Outdated Show resolved Hide resolved
packages/instrumentor/instrument.ts Outdated Show resolved Hide resolved
@oetr oetr force-pushed the FUZZ-372-coverage-reports branch 5 times, most recently from d243b53 to 1f2e41a Compare January 24, 2023 08:03
@oetr oetr requested a review from kyakdan January 24, 2023 08:30
@kyakdan
Copy link
Member

kyakdan commented Jan 25, 2023

@oetr Could you rebase on main? There have been several changes as part of supporting libFuzzer's fork mode.

docs/fuzz-targets.md Outdated Show resolved Hide resolved
packages/core/cli.ts Show resolved Hide resolved
packages/instrumentor/instrument.ts Outdated Show resolved Hide resolved
examples/custom-hooks/package.json Outdated Show resolved Hide resolved
Copy link
Member

@kyakdan kyakdan left a comment

Choose a reason for hiding this comment

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

Great improvement!

@oetr oetr force-pushed the FUZZ-372-coverage-reports branch 2 times, most recently from e569de8 to 61fcc31 Compare January 26, 2023 14:39
- Add integration tests for source code coverage report generation
- Refine the interplay between the flags "dry run" (-d) and "custom
hooks" (-h). Now the custom hooking works even in dry run mode.
- Simplify the FuzzedDataProvider test to decrease the chances of the
test failing on MacOS (this fuzz test is expected to find an exception
within some time)
@oetr oetr merged commit ce9ab5e into main Jan 26, 2023
@oetr oetr deleted the FUZZ-372-coverage-reports branch January 26, 2023 15:34
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

2 participants