microsoft / onefuzz

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

Return the failure from NotifyAdo as result instead of an exception #3555

Closed chkeita closed 10 months ago

chkeita commented 11 months ago

Summary of the Pull Request

closes #3554

codecov-commenter commented 11 months ago

Codecov Report

Merging #3555 (4a25e76) into main (d792b3b) will decrease coverage by 0.02%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3555      +/-   ##
==========================================
- Coverage   39.01%   39.00%   -0.02%     
==========================================
  Files         302      302              
  Lines       36954    36959       +5     
==========================================
- Hits        14417    14415       -2     
- Misses      22537    22544       +7     
Files Coverage Δ
src/ApiService/ApiService/OneFuzzTypes/Enums.cs 32.39% <ø> (ø)
...iService/ApiService/Functions/NotificationsTest.cs 0.00% <0.00%> (ø)
...rvice/TestHooks/NotificationOperationsTestHooks.cs 0.00% <0.00%> (ø)
...ce/ApiService/onefuzzlib/NotificationOperations.cs 23.07% <0.00%> (ø)
...piService/ApiService/Functions/QueueFileChanges.cs 10.41% <0.00%> (ø)
...Service/ApiService/onefuzzlib/notifications/Ado.cs 34.11% <0.00%> (-0.41%) :arrow_down:

... and 1 file with indirect coverage changes