test: Async safe log write to console

[philipphofmann]

Ensure setting SENTRY_ASYNC_SAFE_LOG_ALSO_WRITE_TO_CONSOLE = 1 doesn't get committed by accident.

#skip-changelog

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.689%. Comparing base (8bd2b82) to head (b6e222b).
Report is 4 commits behind head on main.

Additional details and impacted files

@@              Coverage Diff              @@
##              main     #5094       +/-   ##
=============================================
- Coverage   92.787%   92.689%   -0.099%     
=============================================
  Files          676       676               
  Lines        83881     83876        -5     
  Branches     30537     29453     -1084     
=============================================
- Hits         77831     77744       -87     
- Misses        5952      6037       +85     
+ Partials        98        95        -3     

see 23 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8bd2b82...b6e222b. Read the comment docs.

🚀 New features to boost your workflow:

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

[philprime]

m: While I do like this approach in using tests to cause the failure of CI, where else is this SENTRY_ASYNC_SAFE_LOG_ALSO_WRITE_TO_CONSOLE used and where else can it become a problem?
What if it is not set on SentryTests but set on Sentry?

I believe I need more context here.

[philipphofmann]

I did this as a follow up on: #5091 (comment). I added a clarification comment. Is it clearer now @philprime?

[philprime]

I don't want to block this PR longer therefore LGTM approved, but I want to point out that it's not clear where the flag needs to be set, therefore I don't know how to test this.

[philipphofmann]

Thanks for the feedback, @philprime . Is it clear now b6e222b?

[philprime]

Makes sense now, thank you!