Update very outdated SAST report fixtures
Proposal
The report fixtures we use within the rails application are very, very old. They do not test real-world conditions and have a high likelihood of hiding bugs.
This also appears to be true for ~"Category:Dependency Scanning" and should be updated as well.
Implementation Plan
-
Remove https://gitlab.com/gitlab-org/gitlab/-/blob/master/spec/fixtures/security_reports/master/gl-sast-report.json (wrong namespace)this actually makes sense as we're moving more of SAST to core Add https://gitlab.com/gitlab-org/gitlab/-/blob/master/ee/spec/fixtures/security_reports/master/gl-sast-report.json- Update https://gitlab.com/gitlab-org/gitlab/-/blob/master/ee/spec/fixtures/security_reports/feature_branch/gl-sast-report.json
- Update all failing tests
Edited by Lucas Charles