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

Conversation

@chkeita
Copy link
Contributor

@chkeita chkeita commented Oct 6, 2023

Summary of the Pull Request

closes #3554

@chkeita chkeita requested a review from tevoinea October 6, 2023 02:01
@codecov-commenter
Copy link

codecov-commenter commented Oct 6, 2023

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%) ⬇️

... and 1 file with indirect coverage changes

@chkeita chkeita merged commit 7afe06a into microsoft:main Oct 17, 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.

Notification test endpoint should return the validation failure if the template is invalid

3 participants