-
Notifications
You must be signed in to change notification settings - Fork 3
Static Code Analysis
ntreese edited this page Mar 9, 2020
·
3 revisions

We have been able to get our unit test coverage to be greater than 50%. Furthermore, we were able to reduce our code duplication down to 0 by refactoring our backend code. Unfortunately, we ended up with 23 code smell warnings but luckily these are easy to fix. Most of them are related to having unused packages, or commented out code which is no longer in use. The security vulnerabilities that we have will have to get address. The @RequestMapping needs to be a method otherwise the mapping can be used for both POST and GET requests.