test(appsec): add coverage for BlockingSecurityEvent helpers#4504
test(appsec): add coverage for BlockingSecurityEvent helpers#4504ManuelPalenzuelaDD wants to merge 1 commit intomainfrom
Conversation
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage (details) 🔗 Commit SHA: cd1e6a0 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback! |
BenchmarksBenchmark execution time: 2026-03-06 10:47:38 Comparing candidate commit cd1e6a0 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 155 metrics, 9 unstable metrics.
|
Motivation
IsSecurityErrorand theBlockingSecurityEvent.Error()message.Description
appsec/events/block_test.gowith a test forBlockingSecurityEvent.Error()validating the returned string.TestIsSecurityErrorwith subtests coveringnil, a directBlockingSecurityEvent, a wrappedBlockingSecurityEventusingerrors.Join, and a non-securityerrorcase.Testing
go test ./appsec/eventswhich completed successfully.Codex Task