Skip to content

Allow CWE(uppercase) for security training

Mehmet Emin INAC requested to merge minac_419554_support_CWE_for_security_training_feature into master

What does this MR do and why?

Fix the security training URL finder logic to support CWE(uppercase) identifiers. They were already supporting the cwe(lowercase) identifiers.

I also wanted to improve the already existing specs in terms of their performance and readability as I had to touch them anyway. These specs share too much in common so I think we need to write shared examples to DRY them up but for the sake of results and iteration, I kept my refactoring short.

Related to Support CWE (uppercase) (#419554 - closed).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Mehmet Emin INAC

Merge request reports