Merge pull request #1223 from OneSignal/fg/sync-fix

Fix Sync Call

jest.config.js

@@ -8,6 +8,10 @@ module.exports = async () => {
     forceExit: true,
     bail: true,
     testEnvironment: 'jsdom',
+    moduleNameMapper: {
+      '^src/(.*)$': '<rootDir>/src/$1',
+      '^__test__/(.*)$': '<rootDir>/__test__/$1',
+    },
     // Run these files after jest has been
     // installed in the environment
     setupFilesAfterEnv: ['<rootDir>/__test__/jest/jest.setup.ts'], // use .js if you prefer JavaScript,

src/shared/api/OneSignalApiBase.ts

@@ -136,6 +136,12 @@ export class OneSignalApiBase {
       return isValidUuid(parts[1]);
     }
 
+    // special case for sync
+    if (url.startsWith('sync/')) {
+      const parts = url.split('/');
+      return isValidUuid(parts[1]);
+    }
+
     if (body && typeof body['app_id'] === 'string') {
       return isValidUuid(body['app_id']);
     }

src/shared/api/OneSignalApiSW.test.ts

@@ -0,0 +1,11 @@
+import { APP_ID } from '__test__/support/constants';
+import { OneSignalApiSW } from './OneSignalApiSW';
+
+// Temporary, can remove when adding new tests to ServiceWorker.test.ts
+describe('OneSignalApiSW', () => {
+  test('downloadServerAppConfig', async () => {
+    test.nock({});
+    const appConfig = await OneSignalApiSW.downloadServerAppConfig(APP_ID);
+    expect(appConfig).toBeDefined();
+  });
+});