Skip to content

add a round trip test to the fuzzer #1060

Merged
phlptp merged 23 commits intoCLIUtils:mainfrom
phlptp:fuzz_roundtrip
Sep 23, 2024
Merged

add a round trip test to the fuzzer #1060
phlptp merged 23 commits intoCLIUtils:mainfrom
phlptp:fuzz_roundtrip

Conversation

@phlptp
Copy link
Copy Markdown
Collaborator

@phlptp phlptp commented Jul 27, 2024

This is the next phase of the fuzzer. It runs a round trip and makes sure that the config files generated by the app will load into the same results, to test full round trip on the config files.

Issues fixed

  • fix a bug in the string escape code caught by initial round trip tests
  • resolve inconsistencies in handling of {} for empty vector indication between config and cli parsing

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (e4ee3af) to head (83694dc).
Report is 35 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##              main     #1060    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           17        17            
  Lines         4546      4599    +53     
  Branches         0       987   +987     
==========================================
+ Hits          4546      4599    +53     

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

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