Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUM-7332: Add instrumented test for Compose Session Replay #2469

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ambushwork
Copy link
Member

What does this PR do?

Add instrumented test for Compose Session Replay

Motivation

RUM-7332

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)

@ambushwork ambushwork force-pushed the yl/compose/instrumental-test branch from 26c3d32 to e07743a Compare December 30, 2024 13:00
@ambushwork ambushwork marked this pull request as ready for review December 30, 2024 13:17
@ambushwork ambushwork requested review from a team as code owners December 30, 2024 13:17
@codecov-commenter
Copy link

codecov-commenter commented Dec 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.97%. Comparing base (f61a42c) to head (2d982f2).
Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2469      +/-   ##
===========================================
+ Coverage    69.95%   69.97%   +0.02%     
===========================================
  Files          786      786              
  Lines        29298    29298              
  Branches      4889     4889              
===========================================
+ Hits         20493    20500       +7     
+ Misses        7439     7436       -3     
+ Partials      1366     1362       -4     

see 36 files with indirect coverage changes

jonathanmos
jonathanmos previously approved these changes Dec 31, 2024
Copy link
Member

@jonathanmos jonathanmos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We should probably at some point convert all the instrumented tests over to use FGM privacy levels

@ambushwork ambushwork force-pushed the yl/compose/instrumental-test branch from e07743a to 209c8a0 Compare December 31, 2024 11:19
@ambushwork ambushwork force-pushed the yl/compose/instrumental-test branch from 209c8a0 to 2d982f2 Compare December 31, 2024 12:41
)

@Test
@Ignore("To fix in RUM-7899: The test is flaky among different APIS")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
@Ignore("To fix in RUM-7899: The test is flaky among different APIS")
@Ignore("To fix in RUM-7899: The test is flaky among different APIs")

.padding(16.dp)
.clip(RoundedCornerShape(16.dp))
.background(Color.Black),
onClick = {}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should it be empty?

Comment on lines +30 to +37
Checkbox(checked = true, onCheckedChange = {})
Checkbox(checked = false, onCheckedChange = {})

Switch(checked = true, onCheckedChange = {})
Switch(checked = false, onCheckedChange = {})

RadioButton(selected = true, onClick = {})
RadioButton(selected = false, onClick = {})
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should all these be empty? or it is just to make them respond to click events?

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.

5 participants