Fix integration test.py multitest case

microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform

MIT License

2.82k stars 198 forks source link

Fix integration test.py multitest case #3595

Closed AdamL-Microsoft closed 1 year ago

AdamL-Microsoft commented 1 year ago

Summary of the Pull Request

What is this about?

PR Checklist

[ ] Applies to work item: #xxx
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

codecov-commenter commented 1 year ago

Codecov Report

Merging #3595 (16bc0dc) into main (44c2c39) will increase coverage by 8.45%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3595      +/-   ##
==========================================
+ Coverage   39.96%   48.42%   +8.45%     
==========================================
  Files         303      139     -164     
  Lines       37635    17861   -19774     
  Branches     1721        0    -1721     
==========================================
- Hits        15042     8649    -6393     
+ Misses      22274     9212   -13062     
+ Partials      319        0     -319

see 195 files with indirect coverage changes

AdamL-Microsoft commented 1 year ago

This turned out to be unnecessary, issue was in ADO not the integration-test.py script.