microsoft / onefuzz

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

Move initial coverage reporting to later in startup to capture read-only inputs #3605

Closed nharper285 closed 9 months ago

nharper285 commented 9 months ago

Summary of the Pull Request

What is this about? We were reporting 'initial_coverage' before running the readonly inputs. It needs to be emitted afterward.

PR Checklist

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

codecov-commenter commented 9 months ago

Codecov Report

Merging #3605 (710e2cb) into main (909b130) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3605   +/-   ##
=======================================
  Coverage   28.91%   28.91%           
=======================================
  Files         136      136           
  Lines       14765    14765           
  Branches     1728     1728           
=======================================
  Hits         4269     4269           
  Misses      10175    10175           
  Partials      321      321