Commit 43e0d5ee authored by James Lopez's avatar James Lopez

Merge branch '277326_insert_uuid_values_into_security_findings_table' into 'master'

Insert `uuid` values on Security::Finding creation

See merge request gitlab-org/gitlab!50031
parents de487838 4ac6cf6d
......@@ -44,6 +44,7 @@ module Security
{
severity: report_finding.severity,
confidence: report_finding.confidence,
uuid: report_finding.uuid,
project_fingerprint: report_finding.project_fingerprint,
scanner: persisted_scanner_for(report_finding.scanner),
position: position
......
......@@ -40,6 +40,7 @@ RSpec.describe Security::StoreFindingsMetadataService do
expect { store_findings }.to change { security_scan.findings.count }.by(2)
.and change { security_scan.findings.first&.severity }.to(security_finding_1.severity.to_s)
.and change { security_scan.findings.first&.confidence }.to(security_finding_1.confidence.to_s)
.and change { security_scan.findings.first&.uuid }.to(security_finding_1.uuid)
.and change { security_scan.findings.first&.project_fingerprint }.to(security_finding_1.project_fingerprint)
.and change { security_scan.findings.first&.position }.to(0)
.and change { security_scan.findings.last&.position }.to(1)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment