add prefix to log methods

Author: fadi-george

__test__/unit/push/registerForPush.test.ts

@@ -31,7 +31,7 @@ describe('Register for push', () => {
     expect(spy).not.toHaveBeenCalled();
     OneSignalEvent.trigger(OneSignal.EVENTS.SDK_INITIALIZED);
     await promise;
-    expect(Log.error).toHaveBeenCalled();
+    expect(Log._error).toHaveBeenCalled();
     expect(OneSignal.initialized).toBe(true);
     expect(spy).toHaveBeenCalledTimes(1);
   });
@@ -42,7 +42,7 @@ describe('Register for push', () => {
     (global as any).OneSignal._initCalled = false;
 
     await InitHelper.registerForPushNotifications();
-    expect(Log.error).toHaveBeenCalled();
+    expect(Log._error).toHaveBeenCalled();
     expect(spy).toHaveBeenCalledTimes(1);
   });
 });

package.json

@@ -84,12 +84,12 @@
     },
     {
       "path": "./build/releases/OneSignalSDK.page.es6.js",
-      "limit": "51.964 kB",
+      "limit": "51.91 kB",
       "gzip": true
     },
     {
       "path": "./build/releases/OneSignalSDK.sw.js",
-      "limit": "14.07 kB",
+      "limit": "14.04 kB",
       "gzip": true
     },
     {

src/core/executors/CustomEventOperationExecutor.ts

@@ -34,7 +34,7 @@ export class CustomEventsOperationExecutor implements IOperationExecutor {
   }
 
   async execute(operations: Operation[]): Promise<ExecutionResponse> {
-    Log.debug(
+    Log._debug(
       `CustomEventsOperationExecutor(operations: ${JSON.stringify(
         operations,
       )})`,

src/core/executors/IdentityOperationExecutor.ts

@@ -40,7 +40,7 @@ export class IdentityOperationExecutor implements IOperationExecutor {
   }
 
   async execute(operations: Operation[]): Promise<ExecutionResponse> {
-    Log.debug(
+    Log._debug(
       `IdentityOperationExecutor(operations: ${JSON.stringify(operations)})`,
     );
 

src/core/executors/LoginUserOperationExecutor.ts

@@ -62,7 +62,7 @@ export class LoginUserOperationExecutor implements IOperationExecutor {
   }
 
   async execute(operations: Operation[]): Promise<ExecutionResponse> {
-    Log.debug(
+    Log._debug(
       `LoginUserOperationExecutor(operation: ${JSON.stringify(operations)})`,
     );
     const startingOp = operations[0];
@@ -124,11 +124,11 @@ export class LoginUserOperationExecutor implements IOperationExecutor {
       }
 
       case ExecutionResult.FAIL_CONFLICT:
-        Log.debug(`Handling 409 for externalId: ${loginUserOp.externalId}`);
+        Log._debug(`Handling 409 for externalId: ${loginUserOp.externalId}`);
         return this.createUser(loginUserOp, operations);
 
       case ExecutionResult.FAIL_NORETRY:
-        Log.error(
+        Log._error(
           `Recovering from SetAlias failure for externalId: ${loginUserOp.externalId}`,
         );
         return this.createUser(loginUserOp, operations);

src/core/executors/RefreshUserOperationExecutor.ts

@@ -52,7 +52,7 @@ export class RefreshUserOperationExecutor implements IOperationExecutor {
   }
 
   async execute(operations: Operation[]): Promise<ExecutionResponse> {
-    Log.debug(
+    Log._debug(
       `RefreshUserOperationExecutor(operation: ${JSON.stringify(operations)})`,
     );
 

src/core/executors/SubscriptionOperationExecutor.ts

@@ -53,7 +53,7 @@ export class SubscriptionOperationExecutor implements IOperationExecutor {
   }
 
   async execute(operations: Operation[]): Promise<ExecutionResponse> {
-    Log.debug(`SubscriptionOperationExecutor(operations: ${operations})`);
+    Log._debug(`SubscriptionOperationExecutor(operations: ${operations})`);
 
     const startingOp = operations[0];
     if (startingOp instanceof CreateSubscriptionOperation)

src/core/executors/UpdateUserOperationExecutor.ts

@@ -73,7 +73,7 @@ export class UpdateUserOperationExecutor implements IOperationExecutor {
   }
 
   async execute(operations: Operation[]): Promise<ExecutionResponse> {
-    Log.debug(`UpdateUserOperationExecutor(operation: ${operations})`);
+    Log._debug(`UpdateUserOperationExecutor(operation: ${operations})`);
 
     const { appId, onesignalId, propertiesObject, refreshDeviceMetadata } =
       this.processOperations(operations);

src/core/modelRepo/OperationModelStore.ts

@@ -27,7 +27,7 @@ export class OperationModelStore extends ModelStore<Operation> {
 
   create(jsonObject?: { name?: string } | null): Operation | null {
     if (jsonObject === null) {
-      Log.error('null jsonObject sent to OperationModelStore.create');
+      Log._error('null jsonObject sent to OperationModelStore.create');
       return null;
     }
 
@@ -96,15 +96,15 @@ export class OperationModelStore extends ModelStore<Operation> {
   } {
     const operationName = object?.name;
     if (!operationName) {
-      Log.error("jsonObject must have 'name' attribute");
+      Log._error("jsonObject must have 'name' attribute");
       return false;
     }
 
     const excluded = new Set<string>([OPERATION_NAME.LOGIN_USER]);
 
     // Must have onesignalId if it is not one of the excluded operations above
     if (!object.onesignalId && !excluded.has(operationName)) {
-      Log.error(
+      Log._error(
         `${operationName} jsonObject must have 'onesignalId' attribute`,
       );
       return false;

src/core/operationRepo/OperationRepo.test.ts

@@ -21,7 +21,7 @@ import {
 import { NewRecordsState } from './NewRecordsState';
 import { OperationQueueItem, OperationRepo } from './OperationRepo';
 
-vi.spyOn(Log, 'error').mockImplementation((msg) => {
+vi.spyOn(Log, '_error').mockImplementation((msg) => {
   if (typeof msg === 'string' && msg.includes('Operation execution failed'))
     return '';
   return msg;