feat(reportingLogger): Introduce default user agent and URL handling in ReportingLogger, with corresponding tests for fallback behavior

Author: gtamanaha

src/logging/reportingLogger.ts

@@ -14,6 +14,8 @@ export class ReportingLogger implements IReportingLogger {
     private readonly integration: string = 'mp-wsdk';
     private readonly rateLimiter: IRateLimiter;
     private readonly DEFAULT_ACCOUNT_ID: string = 'no-account-id-set';
+    private readonly DEFAULT_USER_AGENT: string = 'no-user-agent-set';
+    private readonly DEFAULT_URL: string = 'no-url-set';
 
     constructor(
         private baseUrl: string,
@@ -97,11 +99,11 @@ export class ReportingLogger implements IReportingLogger {
     }
 
     private getUrl(): string {
-        return window.location.href;
+        return window?.location?.href ?? this.DEFAULT_URL;
     }
 
     private getUserAgent(): string {
-        return window.navigator.userAgent;
+        return window?.navigator?.userAgent ?? this.DEFAULT_USER_AGENT;
     }
 
     private sendLogToServer(logRequest: LogRequest) {

test/jest/reportingLogger.spec.ts

@@ -101,6 +101,17 @@ describe('ReportingLogger', () => {
         const fetchCall = mockFetch.mock.calls[0];
         expect(fetchCall[1].headers['rokt-account-id']).toBe('no-account-id-set');
     });
+
+    it('uses default user agent when user agent is empty', () => {
+        logger = new ReportingLogger(baseUrl, sdkVersion, accountId);
+        delete (globalThis as any).navigator;
+        delete (globalThis as any).location;
+        logger.error('msg');
+        expect(mockFetch).toHaveBeenCalled();
+        const fetchCall = mockFetch.mock.calls[0];
+        const body = JSON.parse(fetchCall[1].body);
+        expect(body).toMatchObject({ deviceInfo: 'no-user-agent-set', url: 'no-url-set' });
+    });
 });
 
 describe('RateLimiter', () => {