Add unit test

Author: EmilianoSanchez

CHANGES.txt

@@ -1,4 +1,4 @@
-2.8.1 (November 21, 2025)
+2.8.1 (November 25, 2025)
  - Updated the order of storage operations to prevent inconsistent states when using the `LOCALSTORAGE` storage type and the browser’s `localStorage` fails due to quota limits.
 
 2.8.0 (October 30, 2025)

src/storages/inLocalStorage/MySegmentsCacheInLocal.ts

@@ -4,6 +4,8 @@ import { AbstractMySegmentsCacheSync } from '../AbstractMySegmentsCacheSync';
 import type { MySegmentsKeyBuilder } from '../KeyBuilderCS';
 import { LOG_PREFIX, DEFINED } from './constants';
 import { StorageAdapter } from '../types';
+import { MySegmentsData } from '../../sync/polling/types';
+import { IMySegmentsResponse } from '../../dtos/types';
 
 export class MySegmentsCacheInLocal extends AbstractMySegmentsCacheSync {
 
@@ -16,7 +18,6 @@ export class MySegmentsCacheInLocal extends AbstractMySegmentsCacheSync {
     this.log = log;
     this.keys = keys;
     this.storage = storage;
-    // There is not need to flush segments cache like splits cache, since resetSegments receives the up-to-date list of active segments
   }
 
   protected addSegment(name: string): boolean {
@@ -70,9 +71,9 @@ export class MySegmentsCacheInLocal extends AbstractMySegmentsCacheSync {
     return n;
   }
 
-  registerSegments() {
+  resetSegments(segmentsData: MySegmentsData | IMySegmentsResponse) {
     try {
-      return super.registerSegments();
+      return super.resetSegments(segmentsData);
     } catch (e) {
       this.log.error(LOG_PREFIX + e);
       return false;

src/storages/inLocalStorage/__tests__/MySegmentsCacheInLocal.spec.ts

@@ -39,3 +39,16 @@ test.each(storages)('SEGMENT CACHE / in LocalStorage', (storage) => {
   expect(storage.getItem(PREFIX + '.user.largeSegment.mocked-segment-2')).toBe('1');
   expect(storage.getItem(PREFIX + '.user.largeSegment.mocked-segment')).toBe(null);
 });
+
+test('SEGMENT CACHE / Special case: localStorage failure should not throw an exception', () => {
+  const cache = new MySegmentsCacheInLocal(loggerMock, new KeyBuilderCS(PREFIX, 'user2'), localStorage);
+
+  // mock localStorage failure
+  const setItemSpy = jest.spyOn(localStorage, 'setItem').mockImplementation(() => { throw new Error('localStorage failure'); });
+  setItemSpy.mockClear();
+
+  expect(cache.resetSegments({ k: [{ n: 'mocked-segment' }, { n: 'mocked-segment-2' }], cn: 123 })).toBe(false);
+  expect(setItemSpy).toHaveBeenCalledTimes(1);
+
+  setItemSpy.mockRestore();
+});