Skip to content

RUM-13518: Fix Compose checkbox not sending RUM action#3077

Merged
jonathanmos merged 1 commit into
developfrom
jmoskovich/rum-13518/fix-checkbox-action
Dec 29, 2025
Merged

RUM-13518: Fix Compose checkbox not sending RUM action#3077
jonathanmos merged 1 commit into
developfrom
jmoskovich/rum-13518/fix-checkbox-action

Conversation

@jonathanmos

Copy link
Copy Markdown
Member

What does this PR do?

• Modifies LayoutNodeUtils to identify Compose checkboxes as clickable
• Add stub classes for Compose elements to make tests more comprehensive

Motivation

Checkbox taps were not being sent as RUM actions.

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)

@jonathanmos jonathanmos changed the title RUM-13518: Fix Compose checkbox not sending Rum action RUM-13518: Fix Compose checkbox not sending RUM action Dec 25, 2025
@datadog-datadog-prod-us1

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

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 66.45% (+0.13%)

View detailed report

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

@codecov-commenter

codecov-commenter commented Dec 25, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.46%. Comparing base (4f0f65a) to head (cc075ef).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3077      +/-   ##
===========================================
+ Coverage    71.40%   71.46%   +0.06%     
===========================================
  Files          881      881              
  Lines        32388    32388              
  Branches      5462     5462              
===========================================
+ Hits         23126    23145      +19     
+ Misses        7727     7698      -29     
- Partials      1535     1545      +10     
Files with missing lines Coverage Δ
.../android/compose/internal/utils/LayoutNodeUtils.kt 64.29% <ø> (+21.43%) ⬆️

... and 39 files with indirect coverage changes

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

@jonathanmos
jonathanmos marked this pull request as ready for review December 25, 2025 11:02
@jonathanmos
jonathanmos requested a review from a team as a code owner December 25, 2025 11:02
0xnm
0xnm previously approved these changes Dec 26, 2025
@ambushwork

Copy link
Copy Markdown
Member

@codex review

@ambushwork

Copy link
Copy Markdown
Member

LGTM!

ambushwork
ambushwork previously approved these changes Dec 26, 2025
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

ℹ️ 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".

@jonathanmos
jonathanmos dismissed stale reviews from ambushwork and 0xnm via cc075ef December 28, 2025 09:05
@jonathanmos
jonathanmos force-pushed the jmoskovich/rum-13518/fix-checkbox-action branch from 25da177 to cc075ef Compare December 28, 2025 09:05
@jonathanmos
jonathanmos merged commit 96f099f into develop Dec 29, 2025
26 checks passed
@jonathanmos
jonathanmos deleted the jmoskovich/rum-13518/fix-checkbox-action branch December 29, 2025 08:43
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.

4 participants