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
(4/9) RUM-2127 Add synthetics info to RUM Views #1717
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #1717 +/- ##
===========================================
- Coverage 83.41% 83.30% -0.11%
===========================================
Files 465 465
Lines 16092 16103 +11
Branches 2386 2389 +3
===========================================
- Hits 13422 13413 -9
- Misses 2025 2038 +13
- Partials 645 652 +7
|
@@ -903,77 +907,6 @@ internal class RumViewScopeTest { | |||
assertThat(result2).isNull() | |||
} | |||
|
|||
@Test | |||
fun `𝕄 send event 𝕎 handleEvent(StopView) on active view`( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we remove this test? I can see similar test is added with synthetics session, but then shouldn't this test assert user session type then instead of removing it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That must have been a cut-paste instead of a copy paste, fixed
@@ -697,6 +697,23 @@ internal open class RumViewScope( | |||
currentViewId | |||
) | |||
val replayStats = ViewEvent.ReplayStats(recordsCount = sessionReplayRecordsCount) | |||
val syntheticsAttribute = if ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we should extract these methods ?
b94fd27
to
e553692
Compare
e553692
to
33ebc18
Compare
This PR has been created automatically by the CI
Add synthetics info to RUM Views.
The info is read from the RUM context and converted into the relevant attributes at write time.