Fix auto termination propagation

[fractalwrench]

Goal

Fixes a few places where the AutoTermination enum wasn't getting passed through. I also removed it as a parameter from recordCompletedSpan as it doesn't make sense in the context of that function. I debated removing it from recordSpan but believe it makes sense to keep it there given that the code block could potentially run for a long time.

Testing

Relied on existing test coverage. An upcoming PR will include the integration test that originally caught these problems.

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 15 lines in your changes missing coverage. Please review.

Project coverage is 85.31%. Comparing base (2f8ad69) to head (e7a6db8).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
.../io/embrace/android/embracesdk/spans/TracingApi.kt	44.00%	14 Missing ⚠️
...main/java/io/embrace/android/embracesdk/Embrace.kt	80.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1749      +/-   ##
==========================================
- Coverage   85.41%   85.31%   -0.11%     
==========================================
  Files         464      464              
  Lines       10771    10783      +12     
  Branches     1596     1596              
==========================================
- Hits         9200     9199       -1     
- Misses        857      870      +13     
  Partials      714      714              

Files with missing lines	Coverage Δ
...android/embracesdk/internal/spans/EmbraceTracer.kt	100.00% <ø> (ø)
...main/java/io/embrace/android/embracesdk/Embrace.kt	82.27% <80.00%> (ø)
.../io/embrace/android/embracesdk/spans/TracingApi.kt	77.92% <44.00%> (-15.83%)	⬇️

[bidetofevil]

LGTM

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Files