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.

Sign up for GitHub

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

Jump to bottom

Call Metadatum::print() to increase fuzzing coverage. #1806

Merged
merged 1 commit into from
Sep 9, 2021
Merged

Call Metadatum::print() to increase fuzzing coverage. #1806

merged 1 commit into from
Sep 9, 2021

Conversation

kevinbackhouse
Copy link
Collaborator

Increase fuzzing coverage by calling Metadatum::print().

@codecov
Copy link

codecov bot commented Jul 28, 2021 •
edited
Loading

Codecov Report

Merging #1806 (c710689) into main (f8bfad1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1806   +/-   ##
=======================================
  Coverage   60.77%   60.77%           
=======================================
  Files          96       96           
  Lines       18887    18887           
  Branches     9498     9498           
=======================================
  Hits        11478    11478           
  Misses       5116     5116           
  Partials     2293     2293

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f8bfad1...c710689. Read the comment docs.

@hassec hassec added enhancement feature / functionality enhancements testing Anything related to the tests and their infrastructure labels Jul 30, 2021
@kevinbackhouse
Copy link
Collaborator Author

I added iptcData and xmpData.

@kevinbackhouse kevinbackhouse added this to the v1.00 milestone Sep 2, 2021
@kevinbackhouse kevinbackhouse mentioned this pull request Sep 9, 2021
Merged
@piponazo piponazo self-requested a review September 9, 2021 18:57
piponazo
piponazo approved these changes Sep 9, 2021
View reviewed changes
Copy link
Collaborator

@piponazo piponazo left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@kevinbackhouse kevinbackhouse merged commit 1b53899 into Exiv2:main Sep 9, 2021
@kevinbackhouse kevinbackhouse deleted the FuzzMore branch September 9, 2021 21:40
@kevinbackhouse kevinbackhouse mentioned this pull request Sep 18, 2021
Closed
@nehaljwani nehaljwani mentioned this pull request Apr 30, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@piponazo piponazo piponazo approved these changes

@hassec hassec Awaiting requested review from hassec

Assignees
No one assigned
Labels
enhancement feature / functionality enhancements testing Anything related to the tests and their infrastructure
Projects
None yet
Milestone
v0.28.0
Development

Successfully merging this pull request may close these issues.
3 participants
@kevinbackhouse @piponazo @hassec