chore: optimize user engagement event record logic (#26)

Co-authored-by: xiaoweii <[email protected]>

Author: zhu-xiaowei

src/tracker/PageViewTracker.ts

@@ -62,7 +62,9 @@ export class PageViewTracker extends BaseTracker {
 				previousPageTitle !== currentPageTitle
 			) {
 				this.provider.scrollTracker?.enterNewPage();
-				this.recordUserEngagement();
+				if (previousPageUrl !== '') {
+					this.recordUserEngagement();
+				}
 				this.trackPageView(previousPageUrl, previousPageTitle);
 				this.trackSearchEvents();
 
@@ -110,7 +112,6 @@ export class PageViewTracker extends BaseTracker {
 	}
 
 	recordUserEngagement(isImmediate = false) {
-		if (this.lastScreenStartTimestamp === 0) return;
 		this.lastEngageTime = this.getLastEngageTime();
 		if (
 			this.provider.configuration.isTrackUserEngagementEvents &&

test/tracker/PageViewTracker.test.ts

@@ -139,9 +139,14 @@ describe('PageViewTracker test', () => {
 			pageViewTracker,
 			'trackPageViewForSPA'
 		);
+		const userEngagementMock = jest.spyOn(
+			pageViewTracker,
+			'recordUserEngagement'
+		);
 		(context.configuration as any).pageType = PageType.SPA;
 		pageViewTracker.setUp();
 		expect(trackPageViewForSPAMock).toBeCalled();
+		expect(userEngagementMock).not.toBeCalled();
 	});
 
 	test('test environment is not supported for sessionStorage', () => {