up

Author: raju-opti

lib/odp/odp_manager.browser.ts

@@ -0,0 +1,11 @@
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+
+import { createOdpManager } from './odp_manager_factory.browser';
+import { getMockRequestHandler } from '../tests/mock/mock_request_handler';
+
+describe('createOdpManager', () => {
+  it('should use BrowserRequestHandler with the provided timeout as the segment request handler', () => {
+    const odpManager = createOdpManager({ segmentsApiTimeout: 1000 });
+    expect((odpManager as any).segmentRequestHandler.timeout).toBe(1000);
+  });
+});

lib/odp/odp_manager.node.ts

@@ -6,15 +6,13 @@ import { getOdpManager, OdpManagerOptions } from './odp_manager_factory';
 export const BROWSER_DEFAULT_API_TIMEOUT = 10_000;
 
 export const createOdpManager = (options: OdpManagerOptions): OdpManager => {
-  let defaultRequestHandler = new BrowserRequestHandler({ timeout: BROWSER_DEFAULT_API_TIMEOUT });
-
-  const segmentRequestHandler = options.segmentsApiTimeout !== undefined  ?
-    new BrowserRequestHandler({ timeout: options.segmentsApiTimeout }) :
-    defaultRequestHandler;
+  const segmentRequestHandler = new BrowserRequestHandler({ 
+    timeout: options.segmentsApiTimeout || BROWSER_DEFAULT_API_TIMEOUT,
+  });
 
-  const eventRequestHandler = options.eventApiTimeout !== undefined ?
-    new BrowserRequestHandler({ timeout: options.eventApiTimeout }) :
-    defaultRequestHandler;
+  const eventRequestHandler = new BrowserRequestHandler({ 
+    timeout: options.eventApiTimeout || BROWSER_DEFAULT_API_TIMEOUT,
+  });
 
   return getOdpManager({
     ...options,