more method renames

Author: fadi-george

.vscode/settings.json

@@ -26,6 +26,7 @@
           "  newName",
           ");",
           "if (edit) await vscode.workspace.applyEdit(edit);",
+          "await new Promise(resolve => setTimeout(resolve, 100));",
           "await vscode.commands.executeCommand('workbench.action.files.saveAll');"
         ]
       }

__test__/support/environment/TestEnvironment.ts

@@ -32,7 +32,7 @@ export class TestEnvironment {
   ) {
     mockJsonp();
     const oneSignal = initOSGlobals(config);
-    OneSignal._coreDirector._operationRepo.queue = [];
+    OneSignal._coreDirector._operationRepo._queue = [];
 
     if (config.initOneSignalId) {
       updateIdentityModel('onesignal_id', ONESIGNAL_ID);

__test__/unit/push/registerForPush.test.ts

@@ -32,7 +32,7 @@ describe('Register for push', () => {
     const promise = OneSignal.User.PushSubscription.optIn();
 
     expect(spy).not.toHaveBeenCalled();
-    OneSignalEvent.trigger(OneSignal.EVENTS.SDK_INITIALIZED);
+    OneSignalEvent._trigger(OneSignal.EVENTS.SDK_INITIALIZED);
     await promise;
     expect(OneSignal._initialized).toBe(true);
     expect(spy).toHaveBeenCalledTimes(1);

__test__/unit/pushSubscription/nativePermissionChange.test.ts

@@ -50,7 +50,7 @@ describe('Notification Types are set correctly on subscription change', () => {
       });
       await setDbPermission('granted');
 
-      await MainHelper.checkAndTriggerNotificationPermissionChanged();
+      await MainHelper._checkAndTriggerNotificationPermissionChanged();
       expect(triggerNotificationSpy).not.toHaveBeenCalled();
     });
 
@@ -74,7 +74,7 @@ describe('Notification Types are set correctly on subscription change', () => {
         permChangeStringListener,
       );
 
-      await MainHelper.checkAndTriggerNotificationPermissionChanged();
+      await MainHelper._checkAndTriggerNotificationPermissionChanged();
 
       // should update the db
       const dbPermission = await getOptionsValue<NotificationPermission>(

package.json

@@ -84,7 +84,7 @@
     },
     {
       "path": "./build/releases/OneSignalSDK.page.es6.js",
-      "limit": "48.11 kB",
+      "limit": "47.88 kB",
       "gzip": true
     },
     {

src/core/CoreModuleDirector.ts

@@ -117,7 +117,7 @@ export class CoreModuleDirector {
     SubscriptionModel | undefined
   > {
     logMethodCall('CoreModuleDirector.getPushSubscriptionModelByCurrentToken');
-    const pushToken = await MainHelper.getCurrentPushToken();
+    const pushToken = await MainHelper._getCurrentPushToken();
     if (pushToken) {
       return this._getSubscriptionOfTypeWithToken(
         SubscriptionChannel.Push,

src/core/controllers/CustomEventController.ts

@@ -22,7 +22,7 @@ export class CustomEventController implements ICustomEventController {
   }
 
   _sendCustomEvent(event: ICustomEvent): void {
-    const appId = MainHelper.getAppId();
+    const appId = MainHelper._getAppId();
     const identityModel = this._identityModelStore.model;
 
     const op = new TrackCustomEventOperation({
@@ -33,6 +33,6 @@ export class CustomEventController implements ICustomEventController {
       event,
     });
 
-    this._opRepo.enqueue(op);
+    this._opRepo._enqueue(op);
   }
 }

src/core/listeners/IdentityModelStoreListener.ts

@@ -26,7 +26,7 @@ export class IdentityModelStoreListener extends SingletonModelStoreListener<Iden
     _oldValue: unknown,
     newValue: unknown,
   ): Operation {
-    const appId = MainHelper.getAppId();
+    const appId = MainHelper._getAppId();
     if (newValue != null && typeof newValue === 'string') {
       return new SetAliasOperation(
         appId,

src/core/listeners/ModelStoreListener.ts

@@ -32,7 +32,7 @@ export abstract class ModelStoreListener<TModel extends Model> {
 
     const operation = this._getAddOperation(model);
     if (operation != null) {
-      this._opRepo.enqueue(operation);
+      this._opRepo._enqueue(operation);
     }
   }
 
@@ -48,7 +48,7 @@ export abstract class ModelStoreListener<TModel extends Model> {
       args.newValue,
     );
     if (operation != null) {
-      this._opRepo.enqueue(operation);
+      this._opRepo._enqueue(operation);
     }
   }
 
@@ -59,7 +59,7 @@ export abstract class ModelStoreListener<TModel extends Model> {
 
     const operation = await this._getRemoveOperation(model);
     if (operation != null) {
-      this._opRepo.enqueue(operation);
+      this._opRepo._enqueue(operation);
     }
   }
 

src/core/listeners/PropertiesModelStoreListener.ts

@@ -28,7 +28,7 @@ export class PropertiesModelStoreListener extends SingletonModelStoreListener<Pr
     _oldValue: unknown,
     newValue: unknown,
   ): Operation | null {
-    const appId = MainHelper.getAppId();
+    const appId = MainHelper._getAppId();
 
     return new SetPropertyOperation(
       appId,