-
Notifications
You must be signed in to change notification settings - Fork 130
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
✨ [RUMF-794] Add isActive attribute to view events #648
Conversation
fc4014d
to
35580e3
Compare
packages/rum/src/domain/rumEventsCollection/view/trackViews.spec.ts
Outdated
Show resolved
Hide resolved
packages/rum/src/domain/rumEventsCollection/view/trackViews.spec.ts
Outdated
Show resolved
Hide resolved
packages/rum/src/domain/rumEventsCollection/view/trackViews.spec.ts
Outdated
Show resolved
Hide resolved
packages/rum/src/domain/rumEventsCollection/view/trackViews.spec.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Bastien Caudan <bastien.caudan@datadoghq.com>
;({ handler, getViewEvent } = spyOnViews()) | ||
|
||
setupBuilder = setup() | ||
.withFakeClock() |
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.
NIT: the fake clock is not used here, it could be removed
Codecov Report
@@ Coverage Diff @@
## master #648 +/- ##
=======================================
Coverage 87.75% 87.75%
=======================================
Files 55 55
Lines 2531 2531
Branches 524 524
=======================================
Hits 2221 2221
Misses 310 310
Continue to review full report at Codecov.
|
Motivation
Adds a new attribute
isActive
to view events, indicating if the view is currently active or not.Changes
Testing
I have gone over the contributing documentation.