Add Diff Fuzz Target by DaveLak · Pull Request #1914 · gitpython-developers/GitPython (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation1 Commits2 Checks26 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

DaveLak

Adds a new fuzz_diff.py fuzz target that covers Diff class initialization using fuzzed data.

Local test run stats:

#46050	DONE   cov: 308 ft: 338 corp: 8/241b lim: 499 exec/s: 51 rss: 86Mb
Done 46050 runs in 901 second(s)
stat::number_of_executed_units: 46050
stat::average_exec_per_sec:     51
stat::new_units_added:          2
stat::slowest_unit_time_sec:    0
stat::peak_rss_mb:              86

@DaveLak

Adds a new fuzz_diff.py fuzz target that covers Diff class initialization using fuzzed data.

DaveLak added a commit to gitpython-developers/qa-assets that referenced this pull request

May 8, 2024

@DaveLak

@DaveLak

Property access and private methods on the Diff class are complex and involve encoding and decoding operations that warrant being tested.

This test borrows its design from the test_diff.py unit test file.

Byron

Choose a reason for hiding this comment

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

Thanks a lot, looks good to me!

@DaveLak DaveLak deleted the add-diff-fuzz-target branch

May 9, 2024 14:23

2 participants

@DaveLak @Byron