Skip to content

RUM-13082: Fall back to DdRumContentProvider creation time#3055

Merged
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/RUM-13082/fallback-to-DdRumContentProvider
Dec 16, 2025
Merged

RUM-13082: Fall back to DdRumContentProvider creation time#3055
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/RUM-13082/fallback-to-DdRumContentProvider

Conversation

@aleksandr-gringauz

@aleksandr-gringauz aleksandr-gringauz commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

See description in RUM-13082.

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

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)

@aleksandr-gringauz aleksandr-gringauz changed the title RUM-13082: Fall back to DdRumContentProvider creation time if Process… RUM-13082: Fall back to DdRumContentProvider creation time Dec 15, 2025
….getStartElapsedRealtime returns a strange value
@aleksandr-gringauz
aleksandr-gringauz force-pushed the aleksandr-gringauz/RUM-13082/fallback-to-DdRumContentProvider branch from 1f8e310 to 872aab9 Compare December 15, 2025 15:06
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Dec 15, 2025

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 71.46%

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 872aab9 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.17%. Comparing base (ed7f4e1) to head (872aab9).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3055      +/-   ##
===========================================
- Coverage    71.24%   71.17%   -0.07%     
===========================================
  Files          866      866              
  Lines        31730    31730              
  Branches      5360     5361       +1     
===========================================
- Hits         22604    22581      -23     
- Misses        7602     7617      +15     
- Partials      1524     1532       +8     
Files with missing lines Coverage Δ
.../core/internal/time/DefaultAppStartTimeProvider.kt 100.00% <100.00%> (ø)
...va/com/datadog/android/rum/DdRumContentProvider.kt 87.50% <100.00%> (ø)

... and 47 files with indirect coverage changes

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

@aleksandr-gringauz
aleksandr-gringauz marked this pull request as ready for review December 15, 2025 20:16
@aleksandr-gringauz
aleksandr-gringauz requested a review from a team as a code owner December 15, 2025 20:16
@aleksandr-gringauz
aleksandr-gringauz merged commit 0266a14 into develop Dec 16, 2025
26 checks passed
@aleksandr-gringauz
aleksandr-gringauz deleted the aleksandr-gringauz/RUM-13082/fallback-to-DdRumContentProvider branch December 16, 2025 07:20
@aleksandr-gringauz

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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.

3 participants