dd-sdk-android icon indicating copy to clipboard operation
dd-sdk-android copied to clipboard

RUM-6171 use FileObserver to limit calls to filesystem

Open 0xnm opened this issue 7 months ago • 1 comments

What does this PR do?

The same as https://github.com/DataDog/dd-sdk-android/pull/2718, just branch moved directly to the repo.

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)

0xnm avatar Jun 12 '25 08:06 0xnm

Codecov Report

:x: Patch coverage is 91.83673% with 4 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 71.25%. Comparing base (ed7f4e1) to head (a3851b0). :warning: Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
...al/persistence/file/batch/BatchFileOrchestrator.kt 91.84% 0 Missing and 4 partials :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2720      +/-   ##
===========================================
+ Coverage    71.24%   71.25%   +0.01%     
===========================================
  Files          866      866              
  Lines        31730    31742      +12     
  Branches      5360     5364       +4     
===========================================
+ Hits         22604    22615      +11     
+ Misses        7602     7596       -6     
- Partials      1524     1531       +7     
Files with missing lines Coverage Δ
...al/persistence/file/batch/BatchFileOrchestrator.kt 93.41% <91.84%> (+0.51%) :arrow_up:

... and 33 files with indirect coverage changes

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

codecov-commenter avatar Jun 12 '25 09:06 codecov-commenter

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 71.46% (-0.01%)
View detailed report

This comment will be updated automatically if new data arrives.
* Fix with Cursor requires Datadog plugin ≥v2.17.0
🔗 Commit SHA: a3851b0 | Docs | Datadog PR Page | Was this helpful? Give us feedback!