Refactor Deserialization

[maciejwalkowiak]

📢 Type of change

• Bugfix
• New feature
• Enhancement
• Refactoring

📜 Description

Use single method to deserialize JSON.

💡 Motivation and Context

#971 (comment)

💚 How did you test it?

Unit tests.

📝 Checklist

• I reviewed submitted code
• I added tests to verify changes
• All tests passing
• No breaking changes

[marandaneto]

looks good, missing changelog though.

[codecov-io]

Codecov Report

Merging #1047 (be6042d) into main (3c87a48) will increase coverage by 0.01%.
The diff coverage is 87.50%.

@@             Coverage Diff              @@
##               main    #1047      +/-   ##
============================================
+ Coverage     72.95%   72.97%   +0.01%     
+ Complexity     1523     1520       -3     
============================================
  Files           161      161              
  Lines          5465     5457       -8     
  Branches        549      549              
============================================
- Hits           3987     3982       -5     
+ Misses         1189     1186       -3     
  Partials        289      289              

Impacted Files	Coverage Δ	Complexity Δ
sentry/src/main/java/io/sentry/NoOpSerializer.java	57.14% <ø> (+17.14%)	4.00 <0.00> (ø)
...ry/src/main/java/io/sentry/SentryEnvelopeItem.java	83.87% <50.00%> (ø)	16.00 <0.00> (ø)
sentry/src/main/java/io/sentry/GsonSerializer.java	96.82% <100.00%> (-0.24%)	10.00 <0.00> (-3.00)
sentry/src/main/java/io/sentry/OutboxSender.java	66.21% <100.00%> (ø)	14.00 <0.00> (ø)
...y/src/main/java/io/sentry/cache/CacheStrategy.java	49.61% <100.00%> (ø)	20.00 <0.00> (ø)
...y/src/main/java/io/sentry/cache/EnvelopeCache.java	49.09% <100.00%> (ø)	17.00 <0.00> (ø)

---

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 3c87a48...be6042d. Read the comment docs.

[marandaneto]

LGTM

[maciejwalkowiak]

@bruno-garcia perhaps you can also take a look at this one since @marandaneto asked to make it a part of the Performance PR too.