Codecov Report

Merging #1063 (8dadaca) into main (2ac97c8) will increase coverage by 1.70%.
The diff coverage is 83.80%.

❗ Current head 8dadaca differs from pull request most recent head 07e32a3. Consider uploading reports for the commit 07e32a3 to get more accurate results

@@            Coverage Diff             @@
##             main    #1063      +/-   ##
==========================================
+ Coverage   79.52%   81.22%   +1.70%     
==========================================
  Files         194      194              
  Lines        6349     6451     +102     
  Branches     1530     1564      +34     
==========================================
+ Hits         5049     5240     +191     
+ Misses        868      758     -110     
- Partials      432      453      +21     

Continue to review full report at Codecov.

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

Added the tests. Please, review @Tyrrrz @bruno-garcia as I'd like to start the integration on Unity side for these protocols.

Also, why danger fails again?

Please consider adding a changelog entry for the next release.

Instructions and example for changelog
Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:
Device and gpu protocols updated; public IJsonSerializable([#1063](https://github.com/getsentry/sentry-dotnet/pull/1063))
If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Perhaps using const names for the field names, for example:

internal const string ProcessorCountKey = "processor_count";
...
if (ProcessorCount is {} processorCount)           
 {                
writer.WriteNumber(ProcessorCountKey , processorCount);           
 }
...
var processorCount = json.GetPropertyOrNull(ProcessorCountKey)?.GetInt32();

Perhaps using const names for the field names, for example:

internal const string ProcessorCountKey = "processor_count";
...
if (ProcessorCount is {} processorCount)           
 {                
writer.WriteNumber(ProcessorCountKey , processorCount);           
 }
...
var processorCount = json.GetPropertyOrNull(ProcessorCountKey)?.GetInt32();

I was relying on the current style. I haven't seen such usage from within the project, so used directly.