Skip to content

[DRAFT] Cronet: APM support dogfooding#3169

Draft
satween wants to merge 5 commits intodevelopfrom
tvaleev/feature/cronet-apm-snapshot
Draft

[DRAFT] Cronet: APM support dogfooding#3169
satween wants to merge 5 commits intodevelopfrom
tvaleev/feature/cronet-apm-snapshot

Conversation

@satween
Copy link
Contributor

@satween satween commented Feb 6, 2026

What does this PR do?

This is the PR only for dogfooding. It wouldn't be merged and review is not required.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@datadog-datadog-prod-us1

This comment has been minimized.

@codecov-commenter
Copy link

codecov-commenter commented Feb 6, 2026

Codecov Report

❌ Patch coverage is 75.10149% with 184 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.25%. Comparing base (19c6623) to head (fa47b6c).

Files with missing lines Patch % Lines
...android/trace/internal/DatadogPropagationHelper.kt 63.95% 43 Missing and 10 partials ⚠️
...ndroid/trace/internal/ApmNetworkInstrumentation.kt 58.20% 35 Missing and 16 partials ⚠️
...oid/cronet/internal/DatadogCronetRequestContext.kt 89.22% 11 Missing ⚠️
...net/internal/DatadogRequestFinishedInfoListener.kt 42.11% 9 Missing and 2 partials ⚠️
...trace/internal/net/ApmNetworkInstrumentationExt.kt 69.23% 4 Missing and 4 partials ⚠️
...android/cronet/internal/CronetUrlRequestWrapper.kt 14.29% 3 Missing and 3 partials ⚠️
...g/android/cronet/internal/CronetHttpRequestInfo.kt 81.48% 4 Missing and 1 partial ⚠️
.../android/cronet/internal/DatadogRequestCallback.kt 88.37% 0 Missing and 5 partials ⚠️
...tadog/android/cronet/internal/DatadogUrlRequest.kt 70.59% 0 Missing and 5 partials ⚠️
.../instrumentation/network/HttpRequestInfoBuilder.kt 0.00% 4 Missing ⚠️
... and 12 more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3169      +/-   ##
===========================================
+ Coverage    71.03%   71.25%   +0.22%     
===========================================
  Files          912      926      +14     
  Lines        33557    34169     +612     
  Branches      5642     5790     +148     
===========================================
+ Hits         23836    24345     +509     
- Misses        8154     8190      +36     
- Partials      1567     1634      +67     
Files with missing lines Coverage Δ
...droid/flags/openfeature/internal/FlagsClientExt.kt 85.71% <ø> (ø)
...otlin/com/datadog/android/rum/_RumInternalProxy.kt 68.18% <100.00%> (+1.52%) ⬆️
...m/configuration/RumInstrumentationConfiguration.kt 100.00% <100.00%> (ø)
.../android/rum/internal/monitor/DatadogRumMonitor.kt 88.89% <ø> (+0.14%) ⬆️
...roid/rum/internal/net/RumNetworkInstrumentation.kt 81.48% <100.00%> (ø)
...om/datadog/trace/core/propagation/B3HttpCodec.java 6.38% <ø> (ø)
...tadog/trace/core/propagation/DatadogHttpCodec.java 13.68% <ø> (ø)
...m/datadog/trace/core/propagation/W3CHttpCodec.java 12.99% <ø> (ø)
...om/datadog/android/trace/ApmNetworkTracingScope.kt 100.00% <100.00%> (ø)
...datadog/android/trace/DeterministicTraceSampler.kt 100.00% <100.00%> (ø)
... and 31 more

... and 28 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants