feat: Scope sync attachments #5038
Merged
@sentry/warden / warden: find-bugs
completed
Mar 20, 2026 in 3m 21s
1 issue
find-bugs: Found 1 issue (1 low)
Low
SentryAttachment constructed with fileName and contentType swapped in tests - `test/Sentry.Tests/ScopeTests.cs:752`
Multiple test cases create SentryAttachment(default, default, default, "test.txt") where the intent appears to be setting the fileName to "test.txt". However, the constructor signature is (AttachmentType type, IAttachmentContent content, string fileName, string? contentType), so "test.txt" is being assigned to contentType while fileName is null. This creates malformed attachment objects that may not accurately test the scope sync behavior if the observer implementation relies on the attachment filename.
Duration: 3m 19s · Tokens: 2.0M in / 24.3k out · Cost: $3.90 (+extraction: $0.00, +fix_gate: $0.00)
Loading