test(external-reporter): verify consent revoked blocks reporter launch on inproc/breakpad/native

[d2anamaria]

• Add inproc/breakpad tests for consent-revoked with cache-keep blocking external reporter upload
• Add no-cache discard tests for inproc/breakpad and native when consent is revoked
• Add consent-give flush tests for cached crash envelopes with crash-reporter enabled
• Extend native external-reporter consent coverage beyond the test shipped in fix(consent): honor checks before launching crash reporter #1906

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 719cd30. Configure here.}

[cursor]

Race in native no-cache consent test

Medium Severity

test_native_external_crash_reporter_consent_revoked_no_cache asserts an empty cache and no HTTP upload right after run_crash returns, without waiting for the native daemon. Daemon crash handling is asynchronous, so those checks can pass before discard or upload finishes. The existing no-cache native pattern uses wait_for_daemon=True for this reason.

 

^{Reviewed by Cursor Bugbot for commit 719cd30. Configure here.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.75%. Comparing base (5bac907) to head (719cd30).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1935      +/-   ##
==========================================
+ Coverage   75.74%   75.75%   +0.01%     
==========================================
  Files          93       93              
  Lines       22128    22128              
  Branches     3939     3939              
==========================================
+ Hits        16760    16763       +3     
+ Misses       4484     4479       -5     
- Partials      884      886       +2     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.