microsoft / onefuzz

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

Add optional condition when updating/re-opening Work Items #3227

Closed tevoinea closed 1 year ago

tevoinea commented 1 year ago

Summary of the Pull Request

What is this about?

Work Items marked as "Won't Fix" or similar shouldn't be re-opened. Our AdoTemplate didn't have an affordance for that capability.

This PR adds an "Unless" field to on_duplicate. When all of the fields and values in "Unless" match the work item, we will skip updating it.

PR Checklist

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 #3227 (e46ce90) into main (512bb07) will increase coverage by 0.10%. The diff coverage is 79.31%.

@@            Coverage Diff             @@
##             main    #3227      +/-   ##
==========================================
+ Coverage   29.55%   29.65%   +0.10%     
==========================================
  Files         332      332              
  Lines       39199    39211      +12     
==========================================
+ Hits        11584    11629      +45     
+ Misses      27615    27582      -33     
Impacted Files Coverage Δ
src/ApiService/ApiService/OneFuzzTypes/Model.cs 72.72% <71.42%> (+0.03%) :arrow_up:
...Service/ApiService/onefuzzlib/notifications/Ado.cs 11.53% <86.66%> (+11.53%) :arrow_up:

... and 1 file with indirect coverage changes