Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Capture target output, allow Loader reuse #2716

Merged
merged 5 commits into from
Dec 21, 2022

Conversation

ranweiler
Copy link
Member

  • Add an OS-generic CoverageRecorder builder for recording coverage from common parameters
  • Capture the output of the target child process when recording coverage
  • Enable Loader reuse for when coverage is recorded many times for the same target

@ranweiler ranweiler requested review from Porges and chkeita December 21, 2022 02:20
@codecov-commenter
Copy link

Codecov Report

Merging #2716 (694e01f) into main (f7bd508) will decrease coverage by 3.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2716      +/-   ##
==========================================
- Coverage   28.57%   25.39%   -3.19%     
==========================================
  Files         319      123     -196     
  Lines       38026    12889   -25137     
==========================================
- Hits        10866     3273    -7593     
+ Misses      27160     9616   -17544     
Impacted Files Coverage Δ
src/agent/coverage/src/lib.rs
src/agent/coverage/src/record/linux.rs
src/agent/coverage/src/record/linux/debugger.rs
src/agent/coverage/src/record/windows.rs
...efuzz-task/src/tasks/generic/input_poller/tests.rs
src/agent/onefuzz/src/input_tester.rs
src/agent/onefuzz/src/triage.rs
src/agent/onefuzz-agent/src/reboot/double.rs
...c/agent/onefuzz-task/src/local/generic_analysis.rs
src/agent/stacktrace-parser/src/lib.rs
... and 186 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ranweiler ranweiler merged commit 3a5101b into microsoft:main Dec 21, 2022
@ranweiler ranweiler deleted the recording-output branch December 21, 2022 17:14
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jan 5, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jan 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants