Defer session properties load until it's really needed

[bidetofevil]

Goal

Similar to UserInfo, SessionProperties were also being initialized premature. Defer this to when it's actually read and synchronize access.

Testing

Existing tests cover this.

[bidetofevil]

• Add fallback start time values for cases where process creation time is unavailable #564
• Initialize native crash monitoring in a background thread #540
• Defer aspects of API service load until necessary #539
• Defer session properties load until it's really needed #538 👈
• Defer loading of user info until when it's really necessary #537
• Synchronize, optimize, and safely delay native module load #536
• Add NDK service tracing #535
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 87.37864% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 79.65%. Comparing base (5e18d90) to head (b3b9481).
Report is 6 commits behind head on master.

❗ Current head b3b9481 differs from pull request most recent head abcc1ea. Consider uploading reports for the commit abcc1ea to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #538      +/-   ##
==========================================
+ Coverage   79.45%   79.65%   +0.19%     
==========================================
  Files         403      403              
  Lines       10861    10897      +36     
  Branches     1602     1607       +5     
==========================================
+ Hits         8630     8680      +50     
+ Misses       1585     1572      -13     
+ Partials      646      645       -1     

Files	Coverage Δ
...cesdk/anr/ndk/EmbraceNativeThreadSamplerService.kt	90.83% <100.00%> (+2.79%)	⬆️
.../io/embrace/android/embracesdk/ndk/NativeModule.kt	100.00% <100.00%> (+21.95%)	⬆️
...embracesdk/anr/ndk/NativeThreadSamplerInstaller.kt	57.14% <50.00%> (+0.89%)	⬆️
...roid/embracesdk/capture/user/EmbraceUserService.kt	72.85% <93.10%> (+6.19%)	⬆️
.../android/embracesdk/internal/SharedObjectLoader.kt	64.28% <61.53%> (-21.43%)	⬇️
...sdk/session/properties/EmbraceSessionProperties.kt	83.33% <89.13%> (+1.85%)	⬆️

... and 6 files with indirect coverage changes

[fractalwrench]

LGTM

[bidetofevil]

Merge activity

• Mar 11, 11:07 AM EDT: @bidetofevil started a stack merge that includes this pull request via Graphite.
• Mar 11, 11:12 AM EDT: Graphite rebased this pull request as part of a merge.
• Mar 11, 11:13 AM EDT: @bidetofevil merged this pull request with Graphite.