add test case for login then accept permissions

Author: fadi-george

__test__/unit/notifications/subscriptionmanager.test.ts

@@ -174,14 +174,22 @@ describe('LoginUserOperationExecutor', () => {
       await setPushToken(DUMMY_PUSH_TOKEN);
 
       const subscriptionModel = new SubscriptionModel();
-      subscriptionModel.setProperty('id', DUMMY_SUBSCRIPTION_ID);
+      subscriptionModel.setProperty(
+        'id',
+        DUMMY_SUBSCRIPTION_ID,
+        ModelChangeTags.HYDRATE,
+      );
       subscriptionModelStore.add(subscriptionModel, ModelChangeTags.HYDRATE);
 
       // perform operations with old onesignal id
       const executor = getExecutor();
 
       const loginOp = new LoginUserOperation(APP_ID, DUMMY_ONESIGNAL_ID);
-      loginOp.setProperty('externalId', DUMMY_EXTERNAL_ID);
+      loginOp.setProperty(
+        'externalId',
+        DUMMY_EXTERNAL_ID,
+        ModelChangeTags.HYDRATE,
+      );
 
       const createSubOp = new CreateSubscriptionOperation(
         mockSubscriptionOpInfo,

src/core/executors/LoginUserOperationExecutor.test.ts

@@ -52,14 +52,20 @@ describe('SubscriptionOperationExecutor', () => {
   });
 
   beforeEach(async () => {
+    setCreateSubscriptionResponse();
     subscriptionModelStore = OneSignal.coreDirector.subscriptionModelStore;
     newRecordsState = OneSignal.coreDirector.newRecordsState;
+    newRecordsState.records.clear();
 
     identityModelStore = OneSignal.coreDirector.identityModelStore;
     propertiesModelStore = OneSignal.coreDirector.propertiesModelStore;
     subscriptionsModelStore = OneSignal.coreDirector.subscriptionModelStore;
 
-    identityModelStore.model.onesignalId = DUMMY_ONESIGNAL_ID;
+    identityModelStore.model.setProperty(
+      'onesignal_id',
+      DUMMY_ONESIGNAL_ID,
+      ModelChangeTags.HYDRATE,
+    );
     buildUserService = new RebuildUserService(
       identityModelStore,
       propertiesModelStore,

src/core/executors/SubscriptionOperationExecutor.test.ts

@@ -13,6 +13,7 @@ import { PropertiesModelStore } from '../modelStores/PropertiesModelStore';
 import { SubscriptionModelStore } from '../modelStores/SubscriptionModelStore';
 import { NewRecordsState } from '../operationRepo/NewRecordsState';
 import { SetPropertyOperation } from '../operations/SetPropertyOperation';
+import { ModelChangeTags } from '../types/models';
 import { ExecutionResult } from '../types/operation';
 import { UpdateUserOperationExecutor } from './UpdateUserOperationExecutor';
 
@@ -49,6 +50,7 @@ describe('UpdateUserOperationExecutor', () => {
     identityModelStore.model.setProperty(
       IdentityConstants.ONESIGNAL_ID,
       DUMMY_ONESIGNAL_ID,
+      ModelChangeTags.HYDRATE,
     );
   });