-
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-738] migrate view to v2 format (experimental) #586
Conversation
Codecov Report
@@ Coverage Diff @@
## bcaudan/resources #586 +/- ##
=====================================================
+ Coverage 87.95% 88.58% +0.63%
=====================================================
Files 47 48 +1
Lines 2382 2391 +9
Branches 493 495 +2
=====================================================
+ Hits 2095 2118 +23
+ Misses 287 273 -14
Continue to review full report at Codecov.
|
packages/rum/src/boot/rum.spec.ts
Outdated
@@ -10,7 +10,7 @@ import { | |||
SESSION_KEEP_ALIVE_INTERVAL, | |||
THROTTLE_VIEW_UPDATE_PERIOD, | |||
View, | |||
} from '../domain/rumEventsCollection/view/viewCollection' | |||
} from '../domain/rumEventsCollection/view/trackViews' |
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.
This naming is not really satisfying since trackXxx and xxxCollection means pretty much the same thing 😕
I'd be willing to go with that for now and wait the end of the migration to have a better overview and see what we can do.
@@ -114,6 +83,34 @@ describe('rum session', () => { | |||
expect(subsequentRequests[0].view.id).not.toEqual(initialRequests[0].view.id) | |||
}) | |||
|
|||
it('when the session is renewed, a new view event should be sent', () => { |
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.
Some of the remaining tests are integration tests, it could be interesting to see what we want to do with them at some point.
Motivation
Following #570, migrating view to v2 format
Changes
Testing
unit tests, staging
I have gone over the contributing documentation.