codecov[bot]
commented
2 months ago Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 70.61%. Comparing base (
293490d) to head (e23196a).
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #1941 +/- ## ======================================= Coverage 70.61% 70.61% ======================================= Files 97 97 Lines 2600 2600 Branches 380 380 ======================================= Hits 1836 1836 Misses 668 668 Partials 96 96 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
derek-ho
RyanL1997
cwperks
Description
This PR adds running all the associated FTR tests on every PR getting merged into the security dashboards plugin. This will raise awareness of the implicit connection here, and also let contributors know when their change needs to make a corresponding test in the FTR, instead of waiting until release time to figure this out.
Category
Enhancement
Why these changes are required?
Fix: #1936
What is the old behavior before changes and new behavior after changes?
Before: not all FTR tests were being run After: now, all tests run during release time are run with every raised PR
Issues Resolved
Close: #1936
Testing
Existing tests pass
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.