Track thermal state

[gabriellanata]

📜 Description

Adds tracking of device thermal state

💡 Motivation and Context

Thermal state is an important measurement because it can explain device throttling that commonly cause app hangs.

💚 How did you test it?

Added to existing tests and ran tests locally

📝 Checklist

You have to check all boxes before merging:

• I reviewed the submitted code.
• I added tests to verify the changes.
• No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
• I updated the docs if needed.
• Review from the native team if needed.
• No breaking change or entry added to the changelog.
• No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

[brustolin]

LGTM

[brustolin]

Some of the new tests are failing.

[gabriellanata]

@brustolin Seems like the failures are unrelated to my changes: Error cloning certificates git repo, please make sure you have access to the repository - see instructions above Is there something i should do about that?

[armcknight]

Yeah those permission errors happen whenever a PR comes in from a fork, because the fork doesn't have access to the secrets in our CI settings.

But there are some related failures, like https://github.com/getsentry/sentry-cocoa/actions/runs/10649320635/job/29561450624?pr=4305#step:11:38

[gabriellanata]

Ok i think the test is fixed

[armcknight]

I pushed a couple quick commits, but everything else here looks great, thanks for adding this. Confirmed it shows up in a test issue:

[codecov]

Codecov Report

Attention: Patch coverage is 77.77778% with 8 lines in your changes missing coverage. Please review.

Project coverage is 91.683%. Comparing base (8c9d1d4) to head (a627040).
Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
Sources/Sentry/SentryExtraContextProvider.m	52.941%	8 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##              main     #4305       +/-   ##
=============================================
+ Coverage   91.671%   91.683%   +0.011%     
=============================================
  Files          617       617               
  Lines        50213     50237       +24     
  Branches     18129     18158       +29     
=============================================
+ Hits         46031     46059       +28     
+ Misses        4089      4084        -5     
- Partials        93        94        +1     

Files with missing lines	Coverage Δ
...ntryTestUtils/TestSentryNSProcessInfoWrapper.swift	100.000% <100.000%> (ø)
Sources/Sentry/SentryClient.m	98.591% <100.000%> (+0.002%)	⬆️
Sources/Sentry/SentryNSProcessInfoWrapper.mm	62.962% <100.000%> (+4.629%)	⬆️
...Tests/Helper/SentryExtraContextProviderTests.swift	100.000% <100.000%> (ø)
Tests/SentryTests/SentryClientTests.swift	97.566% <100.000%> (ø)
Sources/Sentry/SentryExtraContextProvider.m	80.000% <52.941%> (-20.000%)	⬇️

... and 5 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 8c9d1d4...a627040. Read the comment docs.

[gabriellanata]

cc @brustolin

[brustolin]

Thanks very much @gabriellanata