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

error conformance + fixes to problems found in fuzz testing #433

Merged
merged 18 commits into from
Jun 11, 2024
Merged

Conversation

biojppm
Copy link
Owner

@biojppm biojppm commented May 24, 2024

No description provided.

Copy link

codecov bot commented May 24, 2024

Codecov Report

Attention: Patch coverage is 85.90308% with 32 lines in your changes missing coverage. Please review.

Project coverage is 97.24%. Comparing base (95866e8) to head (ae1a1b1).
Report is 54 commits behind head on master.

Files with missing lines Patch % Lines
src/c4/yml/parse_engine.def.hpp 89.28% 12 Missing ⚠️
src/c4/yml/emit.def.hpp 50.00% 7 Missing ⚠️
src/c4/yml/event_handler_tree.hpp 88.88% 6 Missing ⚠️
tools/parse_emit.cpp 60.00% 4 Missing ⚠️
src/c4/yml/emit.hpp 75.00% 1 Missing ⚠️
src/c4/yml/parse_engine.hpp 0.00% 1 Missing ⚠️
tools/yaml_events.cpp 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #433      +/-   ##
==========================================
- Coverage   97.41%   97.24%   -0.18%     
==========================================
  Files          33       33              
  Lines       11118    11239     +121     
==========================================
+ Hits        10831    10929      +98     
- Misses        287      310      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@biojppm biojppm force-pushed the fuzztest branch 2 times, most recently from 68a1e5a to a28b2c3 Compare May 25, 2024 12:32
@biojppm biojppm force-pushed the fuzztest branch 4 times, most recently from 5a69df0 to b730fca Compare May 28, 2024 19:04
@biojppm biojppm force-pushed the fuzztest branch 5 times, most recently from bdd4498 to f3888da Compare June 2, 2024 12:35
@biojppm biojppm changed the title Fuzztest error conformance + fixes to problems found in fuzz testing Jun 2, 2024
@biojppm biojppm force-pushed the fuzztest branch 3 times, most recently from 5b08b6d to 0a8f85a Compare June 10, 2024 09:51
@biojppm biojppm merged commit 0507de6 into master Jun 11, 2024
242 of 244 checks passed
@biojppm biojppm deleted the fuzztest branch June 11, 2024 15:49
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.

1 participant