microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform
MIT License
2.82k stars 199 forks source link

Impl AsyncWrite for cobertura output #3431

Open tevoinea opened 1 year ago

tevoinea commented 1 year ago

This PR allows us to write the cobertura coverage file asynchronously without need to spawn it in a separate thread.

codecov-commenter commented 1 year ago

Codecov Report

Merging #3431 (78f0800) into main (896329d) will increase coverage by 0.25%. The diff coverage is 80.40%.

@@            Coverage Diff             @@
##             main    #3431      +/-   ##
==========================================
+ Coverage   39.32%   39.58%   +0.25%     
==========================================
  Files         299      299              
  Lines       36270    36512     +242     
==========================================
+ Hits        14264    14454     +190     
- Misses      22006    22058      +52     
Files Changed Coverage Δ
...c/agent/onefuzz-task/src/tasks/coverage/generic.rs 86.42% <80.00%> (-0.21%) :arrow_down:
src/agent/cobertura/src/lib.rs 80.30% <80.41%> (-2.39%) :arrow_down: