Skip to content

RUM-11016: Fix clearUserInfo API by adapting it to the context queue#2847

Merged
satween merged 1 commit into
feature/v3from
nogorodnikov/rum-11016/fix-clearuserinfo-api
Sep 2, 2025
Merged

RUM-11016: Fix clearUserInfo API by adapting it to the context queue#2847
satween merged 1 commit into
feature/v3from
nogorodnikov/rum-11016/fix-clearuserinfo-api

Conversation

@0xnm

@0xnm 0xnm commented Sep 2, 2025

Copy link
Copy Markdown
Member

What does this PR do?

This PR adapts Datadog.clearUserInfo API to the context queue usage, it removes the flakiness caused by the concurrency.

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
0xnm requested review from a team as code owners September 2, 2025 09:02
@codecov-commenter

codecov-commenter commented Sep 2, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.60%. Comparing base (b03a1eb) to head (ab672e4).
⚠️ Report is 12 commits behind head on feature/v3.

Additional details and impacted files
@@              Coverage Diff               @@
##           feature/v3    #2847      +/-   ##
==============================================
- Coverage       70.69%   70.60%   -0.08%     
==============================================
  Files             810      810              
  Lines           29475    29476       +1     
  Branches         4945     4945              
==============================================
- Hits            20835    20811      -24     
- Misses           7282     7297      +15     
- Partials         1358     1368      +10     
Files with missing lines Coverage Δ
...n/com/datadog/android/core/internal/DatadogCore.kt 79.19% <100.00%> (+0.74%) ⬆️

... and 34 files with indirect coverage changes

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

@satween
satween merged commit 49b33da into feature/v3 Sep 2, 2025
26 checks passed
@satween
satween deleted the nogorodnikov/rum-11016/fix-clearuserinfo-api branch September 2, 2025 11:06
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