fix(unity): update SentryEvent to have IsCaptured to allow dropping screenshots of filtered events
#5162
@sentry/warden / warden
completed
Apr 23, 2026 in 1m 12s
1 issue
Low
Missing test coverage for IsCaptured flag functionality - `src/Sentry/SentryClient.cs:417`
The new IsCaptured flag functionality has no test coverage. The change introduces business logic where @event.IsCaptured is set to true only when CaptureEnvelope succeeds, but no tests verify this behavior. Tests should cover: (1) IsCaptured is true when capture succeeds, (2) IsCaptured remains false when capture fails, (3) IsCaptured remains false when event is filtered/dropped earlier in the pipeline.
4 skills analyzed
| Skill | Findings | Duration | Cost |
|---|---|---|---|
| code-review | 1 | 43.5s | $0.44 |
| find-bugs | 0 | 1m 10s | $0.80 |
| gha-security-review | 0 | 1m 5s | $0.13 |
| security-review | 0 | 58.1s | $0.36 |
Duration: 3m 57s · Tokens: 1.0M in / 12.3k out · Cost: $1.74
Loading