break up sw static class methods

Author: fadi-george

package.json

@@ -89,7 +89,7 @@
     },
     {
       "path": "./build/releases/OneSignalSDK.sw.js",
-      "limit": "13.58 kB",
+      "limit": "13.442 kB",
       "gzip": true
     },
     {

src/entries/worker.ts

@@ -1,8 +1,7 @@
 /**
  * New clients will only be including this entry file, which will result in a reduced service worker size.
  */
-import { OneSignalServiceWorker } from '../sw/serviceWorker/ServiceWorker';
+import { run } from '../sw/serviceWorker/ServiceWorker';
 
-// Expose this class to the global scope
-declare const self: ServiceWorkerGlobalScope;
-self.OneSignalWorker = OneSignalServiceWorker;
+// The run() is already called in ServiceWorker.ts, but importing it ensures it's not tree-shaken
+void run;

src/global.d.ts

@@ -41,7 +41,6 @@ declare global {
 
   interface WorkerGlobalScope {
     OneSignal: _OneSignal;
-    OneSignalWorker: typeof import('./sw/serviceWorker/ServiceWorker').OneSignalServiceWorker;
     _workerMessenger: import('./sw/serviceWorker/WorkerMessengerSW').WorkerMessengerSW;
     shouldLog: boolean;
   }

src/sw/serviceWorker/ServiceWorker.test.ts

@@ -31,12 +31,20 @@ import type {
   UpsertOrDeactivateSessionPayload,
 } from 'src/shared/session/types';
 import { NotificationType } from 'src/shared/subscriptions/constants';
+import { getAppId } from './ServiceWorker';
+
+// Mock webhook notification events
+vi.mock('../webhooks/notifications/webhookNotificationEvent', () => ({
+  notificationClick: vi.fn(),
+  notificationDismissed: vi.fn(),
+  notificationWillDisplay: vi.fn(),
+}));
+
 import {
   notificationClick,
   notificationDismissed,
   notificationWillDisplay,
 } from '../webhooks/notifications/webhookNotificationEvent';
-import { OneSignalServiceWorker } from './ServiceWorker';
 
 declare const self: ServiceWorkerGlobalScope;
 
@@ -114,7 +122,7 @@ describe('ServiceWorker', () => {
       // @ts-expect-error - search is readonly but we need to set it for testing
       self.location.search = '?appId=some-app-id';
 
-      const appId = await OneSignalServiceWorker._getAppId();
+      const appId = await getAppId();
       expect(appId).toBe('some-app-id');
     });
   });