refactor: removed used code

- removed unused code
- refactored reset methode

Author: Sunita Prajapati

packages/plugins/plugin-amplitudeSession/package.json

@@ -48,7 +48,7 @@
     "@segment/analytics-react-native": "^2.18.0"
   },
   "devDependencies": {
-    "@segment/analytics-react-native": "^2.18.0",
+    "@segment/analytics-react-native": "^2.21.4",
     "@segment/analytics-rn-shared": "workspace:^",
     "@segment/sovran-react-native": "^1.1.0",
     "@types/jest": "^29.5.8",

packages/plugins/plugin-amplitudeSession/src/AmplitudeSessionPlugin.tsx

@@ -163,11 +163,7 @@ export class AmplitudeSessionPlugin extends EventPlugin {
     this.sessionId = -1;
     this.eventSessionId = -1;
     this.lastEventTime = -1;
-    await AsyncStorage.multiRemove([
-      SESSION_ID_KEY,
-      EVENT_SESSION_ID_KEY,
-      LAST_EVENT_TIME_KEY,
-    ]);
+    await AsyncStorage.removeItem(SESSION_ID_KEY);
   }
 
   private insertSession = (event: SegmentEvent) => {
@@ -193,7 +189,6 @@ export class AmplitudeSessionPlugin extends EventPlugin {
 
   private onBackground = () => {
     this.lastEventTime = Date.now();
-    //this.saveSessionData();
   };
 
   private onForeground = () => {
@@ -245,8 +240,6 @@ export class AmplitudeSessionPlugin extends EventPlugin {
       this.eventSessionId === -1 ? newSessionId : this.eventSessionId;
     this.lastEventTime = newSessionId;
 
-    //await this.saveSessionData();
-
     console.log(`[AmplitudeSession] startNewSession -> ${newSessionId}`);
 
     await this.trackSessionStart(newSessionId);
@@ -291,14 +284,6 @@ export class AmplitudeSessionPlugin extends EventPlugin {
       storedEventSessionId != null ? Number(storedEventSessionId) : -1;
   }
 
-  // private async saveSessionData() {
-  //   await AsyncStorage.setItem(SESSION_ID_KEY, this.sessionId.toString());
-  //   await AsyncStorage.setItem(
-  //     LAST_EVENT_TIME_KEY,
-  //     this.lastEventTime.toString()
-  //   );
-  // }
-
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   private disableAllIntegrations(integrations?: Record<string, any>) {
     // eslint-disable-next-line @typescript-eslint/no-explicit-any

packages/plugins/plugin-amplitudeSession/src/__tests__/AmplitudeSessionPlugin.test.ts

@@ -681,11 +681,9 @@ describe('AmplitudeSessionPlugin', () => {
       expect(plugin.sessionId).toBe(-1);
       expect(plugin.lastEventTime).toBe(-1);
       expect(plugin.eventSessionId).toBe(-1);
-      expect(mockAsyncStorage.multiRemove).toHaveBeenCalledWith([
-        'previous_session_id',
-        'event_session_id',
-        'last_event_time',
-      ]);
+      expect(mockAsyncStorage.removeItem).toHaveBeenCalledWith(
+        'previous_session_id'
+      );
     });
   });