Skip to content

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
@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.

@oetr oetr force-pushed the FUZZ-372-coverage-reports branch from 1f2e41a to 7c14d36 Compare January 25, 2023 13:18
@kyakdan kyakdan force-pushed the FUZZ-372-coverage-reports branch from 7c14d36 to e23001f Compare January 26, 2023 03:39
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 force-pushed the FUZZ-372-coverage-reports branch from 61fcc31 to fd88733 Compare January 26, 2023 14:44
@oetr oetr force-pushed the FUZZ-372-coverage-reports branch from fd88733 to 0456540 Compare January 26, 2023 15:19
@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.

2 participants