flag for ConfigAPI.getconfig

radarlabs · Jul 19, 2023 · 3cb5b34 · 3cb5b34
1 parent 40e2255
commit 3cb5b34
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 5 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -92,7 +92,11 @@ class Radar {
       Logger.info(`using options: ${JSON.stringify(options)}`);
     }
 
-    ConfigAPI.getConfig();
+    // NOTE(jasonl): this allows us to run jest tests
+    // without having to mock the ConfigAPI.getConfig call
+    if (!(window as any)?.RADAR_TEST_ENV) {
+      ConfigAPI.getConfig();
+    }
   }
 
   public static clear() {

diff --git a/test/api/trips.test.ts b/test/api/trips.test.ts
@@ -11,6 +11,8 @@ describe('Trips', () => {
 
   beforeEach(() => {
     Radar.initialize('prj_test_pk_123');
+    Radar.setUserId(userId);
+    jest.spyOn(Http, 'request');
   });
 
   afterEach(() => {
@@ -19,10 +21,6 @@ describe('Trips', () => {
   });
 
   describe('startTrip', () => {
-    beforeEach(() => {
-      Radar.setUserId(userId);
-      jest.spyOn(Http, 'request');
-    });
 
     describe('called without tripOptions', () => {
       it('should include tripOptions each set to unknown', async () => {

diff --git a/test/radar.test.ts b/test/radar.test.ts
@@ -11,6 +11,7 @@ import ContextAPI from '../src/api/context';
 import SearchAPI from '../src/api/search';
 import TrackAPI from '../src/api/track';
 import RoutingAPI from '../src/api/routing';
+import ConfigAPI from '../src/api/config';
 
 import { latitude, longitude } from './common';
 
@@ -51,6 +52,15 @@ describe('Radar', () => {
   const units = 'imperial';
 
   describe('initialize', () => {
+    beforeEach(() => {
+      (window as any).RADAR_TEST_ENV = false;
+    });
+
+    afterEach(() => {
+      (window as any).RADAR_TEST_ENV = true;
+      jest.restoreAllMocks();
+    });
+
     describe('no key is provided', () => {
       it('should throw RadarPublishableKeyError', () => {
         try {
@@ -77,19 +87,23 @@ describe('Radar', () => {
 
     describe('test publishableKey is provided', () => {
       it('should initialize SDK in a test environment', () => {
+        jest.spyOn(ConfigAPI, 'getConfig');
         Radar.initialize('_my_test_pk_123');
         const options = Config.get();
         expect(options.publishableKey).toEqual('_my_test_pk_123');
         expect(options.live).toEqual(false);
+        expect(ConfigAPI.getConfig).toHaveBeenCalledTimes(1);
       });
     });
 
     describe('live publishableKey is provided', () => {
       it('should initialize SDK in a live environment', () => {
+        jest.spyOn(ConfigAPI, 'getConfig');
         Radar.initialize('_my_live_pk_123');
         const options = Config.get();
         expect(options.publishableKey).toEqual('_my_live_pk_123');
         expect(options.live).toEqual(true);
+        expect(ConfigAPI.getConfig).toHaveBeenCalledTimes(1);
       });
     });
   });