Refactor how span status is peristed and sent to sdk span from wrappers

[bidetofevil]

Goal

Set the status and status description on the SDK span when setStatus() is called from EmbSpan. We still won't set it to OK when EmbraceSpan.end() is called without an ErrorCode, as this value is not useful when using the Embrace API. When an ErrorCode is used, we set the status to ERROR automatically.

Testing

Fix up existing tests.

[bidetofevil]

• Refactor how span status is peristed and sent to sdk span from wrappers #967 👈
• Don't set span status to OK on every successful trace stopping #966
• master

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

Join @bidetofevil and the rest of your teammates on Graphite

[fractalwrench]

LGTM

[codecov]

Codecov Report

Attention: Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.

Project coverage is 80.96%. Comparing base (070a69a) to head (1eed8f4).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #967      +/-   ##
==========================================
+ Coverage   80.94%   80.96%   +0.01%     
==========================================
  Files         444      444              
  Lines       11824    11814      -10     
  Branches     1807     1805       -2     
==========================================
- Hits         9571     9565       -6     
+ Misses       1458     1456       -2     
+ Partials      795      793       -2     

Files	Coverage Δ
.../android/embracesdk/internal/payload/SpanMapper.kt	62.00% <100.00%> (-0.97%)	⬇️
...oid/embracesdk/internal/spans/EmbraceExtensions.kt	85.45% <100.00%> (+0.70%)	⬆️
...mbrace/android/embracesdk/opentelemetry/EmbSpan.kt	96.15% <100.00%> (+4.48%)	⬆️
...android/embracesdk/spans/PersistableEmbraceSpan.kt	100.00% <100.00%> (ø)
...droid/embracesdk/internal/spans/EmbraceSpanImpl.kt	86.56% <90.00%> (-0.05%)	⬇️

... and 2 files with indirect coverage changes