You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2022/01/14 18:04:54 UTC

[GitHub] [trafficcontrol] ocket8888 opened a new pull request #6513: TPv2 Testing improvements

ocket8888 opened a new pull request #6513:
URL: https://github.com/apache/trafficcontrol/pull/6513


   This PR improves unit testing coverage just a bit from 
   - 34.93% Statements 612/1752
   - 21.12% Branches 109/516
   - 27.18% Functions 112/412
   - 33.51% Lines 562/1677
   
   to
   - 42.04% Statements 724/1722
   - 32.93% Branches 166/504
   - 35.13% Functions 143/407
   - 40.77% Lines 672/1648
   
   This also fixes an issue where coverage could not be generated due to using a legacy plugin for which support has been dropped, updates Angular to version 13.1.3, and reduces `npm audit`-found vulnerabilities from `40 vulnerabilities (2 low, 35 moderate, 3 high)` to `37 vulnerabilities (2 low, 35 moderate)`.
   <hr/>
   
   ## Which Traffic Control components are affected by this PR?
   None
   
   ## What is the best way to verify this PR?
   Make sure all the new tests pass, check out the code coverage to verify it if you want (it's still nothing to write home about, so it's not very important that the numbers I report here be super accurate, just that they represent a small improvement on the original).
   
   ## PR submission checklist
   - [x] This PR has tests
   - [ ] This PR has documentation
   - [ ] This PR has a CHANGELOG.md entry
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY**


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] shamrickus merged pull request #6513: TPv2 Testing improvements

Posted by GitBox <gi...@apache.org>.
shamrickus merged pull request #6513:
URL: https://github.com/apache/trafficcontrol/pull/6513


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org