@futdevpro/fsm-dynamo 1.16.5 → 1.16.7

package/src/_modules/data-handler/_models/data-list-handler.control-model.spec.ts

@@ -1,263 +1,263 @@
-import { DyFM_DataListHandler } from './data-list-handler.control-model';
-import { DyFM_Metadata } from '../../../_models/data-models/metadata.data-model';
-import { DyFM_DataListHandler_Settings } from './data-handler-settings.control-model';
-
-class TestMetadata extends DyFM_Metadata {
-  name?: string;
-}
-
-describe('| DyFM_DataListHandler', (): void => {
-  let handler: DyFM_DataListHandler<TestMetadata>;
-  let settings: DyFM_DataListHandler_Settings<TestMetadata>;
-
-  beforeEach(() => {
-    settings = {
-      name: 'TestListHandler',
-      get: async (loadBy?: string): Promise<TestMetadata[]> => {
-        return [
-          { _id: '1', name: 'item1' } as TestMetadata,
-          { _id: '2', name: 'item2' } as TestMetadata,
-        ];
-      },
-      setItem: async (item: TestMetadata): Promise<TestMetadata> => item,
-      deleteItem: async (id: string): Promise<void> => {},
-      defaultValue: [],
-    };
-
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-  });
-
-  it('| should create an instance with settings', (): void => {
-    expect(handler).toBeTruthy();
-    expect(handler.name).toBe('TestListHandler');
-  });
-
-  it('| should have dataList$ observable', (): void => {
-    expect(handler.dataList$).toBeTruthy();
-  });
-
-  it('| should have activeItem$ observable', (): void => {
-    expect(handler.activeItem$).toBeTruthy();
-  });
-
-  it('| should get empty dataList initially', (): void => {
-    expect(handler.dataList).toEqual([]);
-  });
-
-  it('| should add item to list', async (): Promise<void> => {
-    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
-    await handler.addItem(newItem);
-
-    expect(handler.dataList.length).toBeGreaterThan(0);
-  });
-
-  it('| should set active item', (): void => {
-    const item = { _id: '1', name: 'item1' } as TestMetadata;
-    handler.setActiveItem(item);
-
-    handler.activeItem$.subscribe(activeItem => {
-      expect(activeItem).toEqual(item);
-    });
-  });
-
-  it('| should clear data and active item', (): void => {
-    handler.clear();
-
-    expect(handler.dataList).toEqual([]);
-    handler.activeItem$.subscribe(activeItem => {
-      expect(activeItem).toBeNull();
-    });
-  });
-
-  it('| should remove item with deleteItem', async (): Promise<void> => {
-    // Test case for lines 202-216: removeItem with deleteItem
-    await handler.reloadData('test');
-    const deleteItemSpy = spyOn(handler as any, 'deleteItem').and.returnValue(Promise.resolve());
-    
-    await handler.removeItem('1');
-    
-    expect(deleteItemSpy).toHaveBeenCalledWith('1', undefined);
-    expect(handler.dataList.find(i => i._id === '1')).toBeUndefined();
-  });
-
-  it('| should remove item without deleteItem', async (): Promise<void> => {
-    // Test case for lines 202-216: removeItem without deleteItem
-    settings.deleteItem = undefined;
-    settings.noId = true;
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-    await handler.reloadData();
-    
-    await handler.removeItem('1');
-    
-    expect(handler.dataList.find(i => i._id === '1')).toBeUndefined();
-  });
-
-  it('| should clear active item when removing active item', async (): Promise<void> => {
-    // Test case for lines 213-215: clear active item when removing
-    await handler.reloadData('test');
-    handler.setActiveItem(handler.dataList[0]);
-    
-    await handler.removeItem(handler.dataList[0]._id);
-    
-    handler.activeItem$.subscribe(activeItem => {
-      expect(activeItem).toBeNull();
-    });
-  });
-
-  it('| should update item with setItem', async (): Promise<void> => {
-    // Test case for lines 228-247: updateItem with setItem
-    await handler.reloadData('test');
-    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve(handler.dataList[0]));
-    
-    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
-    await handler.updateItem(updatedItem);
-    
-    expect(setItemSpy).toHaveBeenCalledWith(updatedItem, undefined);
-    expect(handler.dataList.find(i => i._id === '1')?.name).toBe('updated');
-  });
-
-  it('| should update item without setItem', async (): Promise<void> => {
-    // Test case for lines 228-247: updateItem without setItem
-    settings.setItem = undefined;
-    settings.noId = true;
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-    await handler.reloadData();
-    
-    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
-    await handler.updateItem(updatedItem);
-    
-    expect(handler.dataList.find(i => i._id === '1')?.name).toBe('updated');
-  });
-
-  it('| should throw error when updating non-existent item', async (): Promise<void> => {
-    // Test case for lines 238-246: updateItem throws error when item not found
-    await handler.reloadData('test');
-    const nonExistentItem = { _id: '999', name: 'not found' } as TestMetadata;
-    
-    await expectAsync(handler.updateItem(nonExistentItem)).toBeRejected();
-  });
-
-  it('| should reloadData and set active item when different', async (): Promise<void> => {
-    // Test case for lines 153-166: reloadData override
-    await handler.reloadData('test');
-    handler.setActiveItem(handler.dataList[0]);
-    
-    const newLoadBy = 'test2';
-    settings.get = async (loadBy?: string): Promise<TestMetadata[]> => {
-      return [
-        { _id: '3', name: 'item3' } as TestMetadata,
-        { _id: '4', name: 'item4' } as TestMetadata,
-      ];
-    };
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-    
-    await handler.reloadData(newLoadBy);
-    
-    expect(handler.activeItem_BS.value).toBeDefined();
-  });
-
-  it('| should reloadData and not set active item when same', async (): Promise<void> => {
-    // Test case for lines 153-166: reloadData override with same loadBy
-    await handler.reloadData('test');
-    const originalActiveItem = handler.dataList[0];
-    handler.setActiveItem(originalActiveItem);
-    
-    await handler.reloadData('test');
-    
-    expect(handler.activeItem_BS.value).toEqual(originalActiveItem);
-  });
-
-  it('| should addItem with setItem and dontSendUpdate false', async (): Promise<void> => {
-    // Test case for lines 180-191: addItem with setItem
-    await handler.reloadData('test');
-    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve({ _id: '3', name: 'item3' }));
-    
-    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
-    await handler.addItem(newItem, undefined, false);
-    
-    expect(setItemSpy).toHaveBeenCalled();
-    expect(handler.activeItem_BS.value).toEqual(newItem);
-  });
-
-  it('| should addItem with setItem and dontSendUpdate true', async (): Promise<void> => {
-    // Test case for lines 180-191: addItem with dontSendUpdate
-    // When dontSendUpdate is true, setItem should NOT be called (line 183: if (this.setItem && !dontSendUpdate))
-    await handler.reloadData('test');
-    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve({ _id: '3', name: 'item3' }));
-    
-    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
-    await handler.addItem(newItem, undefined, true);
-    
-    expect(setItemSpy).not.toHaveBeenCalled();
-    expect(handler.activeItem_BS.value).toEqual(newItem);
-  });
-
-  it('| should addItem without setItem', async (): Promise<void> => {
-    // Test case for lines 180-191: addItem without setItem
-    settings.setItem = undefined;
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-    await handler.reloadData('test');
-    
-    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
-    await handler.addItem(newItem);
-    
-    expect(handler.dataList.find(i => i._id === '3')).toBeDefined();
-  });
-
-  it('| should removeItem when item not found', async (): Promise<void> => {
-    // Test case for lines 201-216: removeItem when index === -1
-    await handler.reloadData('test');
-    
-    await handler.removeItem('999');
-    
-    // Should not throw, just do nothing
-    expect(handler.dataList.length).toBe(2);
-  });
-
-  it('| should removeItem with updateBy parameter', async (): Promise<void> => {
-    // Test case for lines 201-216: removeItem with updateBy
-    await handler.reloadData('test');
-    const deleteItemSpy = spyOn(handler as any, 'deleteItem').and.returnValue(Promise.resolve());
-    
-    await handler.removeItem('1', 'updateByValue');
-    
-    expect(deleteItemSpy).toHaveBeenCalledWith('1', 'updateByValue');
-  });
-
-  it('| should removeItem with dontSendUpdate', async (): Promise<void> => {
-    // Test case for lines 201-216: removeItem with dontSendUpdate
-    await handler.reloadData('test');
-    settings.deleteItem = undefined;
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-    await handler.reloadData('test');
-    
-    await handler.removeItem('1', undefined, true);
-    
-    expect(handler.dataList.find(i => i._id === '1')).toBeUndefined();
-  });
-
-  it('| should updateItem with updateBy parameter', async (): Promise<void> => {
-    // Test case for lines 227-247: updateItem with updateBy
-    await handler.reloadData('test');
-    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve(handler.dataList[0]));
-    
-    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
-    await handler.updateItem(updatedItem, 'updateByValue');
-    
-    expect(setItemSpy).toHaveBeenCalledWith(updatedItem, 'updateByValue');
-  });
-
-  it('| should updateItem with dontSendUpdate', async (): Promise<void> => {
-    // Test case for lines 227-247: updateItem with dontSendUpdate
-    await handler.reloadData('test');
-    settings.setItem = undefined;
-    handler = new DyFM_DataListHandler<TestMetadata>(settings);
-    await handler.reloadData('test');
-    
-    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
-    await handler.updateItem(updatedItem, undefined, true);
-    
-    expect(handler.dataList.find(i => i._id === '1')?.name).toBe('updated');
-  });
-});
-
+import { DyFM_DataListHandler } from './data-list-handler.control-model';
+import { DyFM_Metadata } from '../../../_models/data-models/metadata.data-model';
+import { DyFM_DataListHandler_Settings } from './data-handler-settings.control-model';
+
+class TestMetadata extends DyFM_Metadata {
+  name?: string;
+}
+
+describe('| DyFM_DataListHandler', (): void => {
+  let handler: DyFM_DataListHandler<TestMetadata>;
+  let settings: DyFM_DataListHandler_Settings<TestMetadata>;
+
+  beforeEach(() => {
+    settings = {
+      name: 'TestListHandler',
+      get: async (loadBy?: string): Promise<TestMetadata[]> => {
+        return [
+          { _id: '1', name: 'item1' } as TestMetadata,
+          { _id: '2', name: 'item2' } as TestMetadata,
+        ];
+      },
+      setItem: async (item: TestMetadata): Promise<TestMetadata> => item,
+      deleteItem: async (id: string): Promise<void> => {},
+      defaultValue: [],
+    };
+
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+  });
+
+  it('| should create an instance with settings', (): void => {
+    expect(handler).toBeTruthy();
+    expect(handler.name).toBe('TestListHandler');
+  });
+
+  it('| should have dataList$ observable', (): void => {
+    expect(handler.dataList$).toBeTruthy();
+  });
+
+  it('| should have activeItem$ observable', (): void => {
+    expect(handler.activeItem$).toBeTruthy();
+  });
+
+  it('| should get empty dataList initially', (): void => {
+    expect(handler.dataList).toEqual([]);
+  });
+
+  it('| should add item to list', async (): Promise<void> => {
+    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
+    await handler.addItem(newItem);
+
+    expect(handler.dataList.length).toBeGreaterThan(0);
+  });
+
+  it('| should set active item', (): void => {
+    const item = { _id: '1', name: 'item1' } as TestMetadata;
+    handler.setActiveItem(item);
+
+    handler.activeItem$.subscribe(activeItem => {
+      expect(activeItem).toEqual(item);
+    });
+  });
+
+  it('| should clear data and active item', (): void => {
+    handler.clear();
+
+    expect(handler.dataList).toEqual([]);
+    handler.activeItem$.subscribe(activeItem => {
+      expect(activeItem).toBeNull();
+    });
+  });
+
+  it('| should remove item with deleteItem', async (): Promise<void> => {
+    // Test case for lines 202-216: removeItem with deleteItem
+    await handler.reloadData('test');
+    const deleteItemSpy = spyOn(handler as any, 'deleteItem').and.returnValue(Promise.resolve());
+    
+    await handler.removeItem('1');
+    
+    expect(deleteItemSpy).toHaveBeenCalledWith('1', undefined);
+    expect(handler.dataList.find(i => i._id === '1')).toBeUndefined();
+  });
+
+  it('| should remove item without deleteItem', async (): Promise<void> => {
+    // Test case for lines 202-216: removeItem without deleteItem
+    settings.deleteItem = undefined;
+    settings.noId = true;
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+    await handler.reloadData();
+    
+    await handler.removeItem('1');
+    
+    expect(handler.dataList.find(i => i._id === '1')).toBeUndefined();
+  });
+
+  it('| should clear active item when removing active item', async (): Promise<void> => {
+    // Test case for lines 213-215: clear active item when removing
+    await handler.reloadData('test');
+    handler.setActiveItem(handler.dataList[0]);
+    
+    await handler.removeItem(handler.dataList[0]._id);
+    
+    handler.activeItem$.subscribe(activeItem => {
+      expect(activeItem).toBeNull();
+    });
+  });
+
+  it('| should update item with setItem', async (): Promise<void> => {
+    // Test case for lines 228-247: updateItem with setItem
+    await handler.reloadData('test');
+    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve(handler.dataList[0]));
+    
+    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
+    await handler.updateItem(updatedItem);
+    
+    expect(setItemSpy).toHaveBeenCalledWith(updatedItem, undefined);
+    expect(handler.dataList.find(i => i._id === '1')?.name).toBe('updated');
+  });
+
+  it('| should update item without setItem', async (): Promise<void> => {
+    // Test case for lines 228-247: updateItem without setItem
+    settings.setItem = undefined;
+    settings.noId = true;
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+    await handler.reloadData();
+    
+    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
+    await handler.updateItem(updatedItem);
+    
+    expect(handler.dataList.find(i => i._id === '1')?.name).toBe('updated');
+  });
+
+  it('| should throw error when updating non-existent item', async (): Promise<void> => {
+    // Test case for lines 238-246: updateItem throws error when item not found
+    await handler.reloadData('test');
+    const nonExistentItem = { _id: '999', name: 'not found' } as TestMetadata;
+    
+    await expectAsync(handler.updateItem(nonExistentItem)).toBeRejected();
+  });
+
+  it('| should reloadData and set active item when different', async (): Promise<void> => {
+    // Test case for lines 153-166: reloadData override
+    await handler.reloadData('test');
+    handler.setActiveItem(handler.dataList[0]);
+    
+    const newLoadBy = 'test2';
+    settings.get = async (loadBy?: string): Promise<TestMetadata[]> => {
+      return [
+        { _id: '3', name: 'item3' } as TestMetadata,
+        { _id: '4', name: 'item4' } as TestMetadata,
+      ];
+    };
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+    
+    await handler.reloadData(newLoadBy);
+    
+    expect(handler.activeItem_BS.value).toBeDefined();
+  });
+
+  it('| should reloadData and not set active item when same', async (): Promise<void> => {
+    // Test case for lines 153-166: reloadData override with same loadBy
+    await handler.reloadData('test');
+    const originalActiveItem = handler.dataList[0];
+    handler.setActiveItem(originalActiveItem);
+    
+    await handler.reloadData('test');
+    
+    expect(handler.activeItem_BS.value).toEqual(originalActiveItem);
+  });
+
+  it('| should addItem with setItem and dontSendUpdate false', async (): Promise<void> => {
+    // Test case for lines 180-191: addItem with setItem
+    await handler.reloadData('test');
+    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve({ _id: '3', name: 'item3' }));
+    
+    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
+    await handler.addItem(newItem, undefined, false);
+    
+    expect(setItemSpy).toHaveBeenCalled();
+    expect(handler.activeItem_BS.value).toEqual(newItem);
+  });
+
+  it('| should addItem with setItem and dontSendUpdate true', async (): Promise<void> => {
+    // Test case for lines 180-191: addItem with dontSendUpdate
+    // When dontSendUpdate is true, setItem should NOT be called (line 183: if (this.setItem && !dontSendUpdate))
+    await handler.reloadData('test');
+    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve({ _id: '3', name: 'item3' }));
+    
+    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
+    await handler.addItem(newItem, undefined, true);
+    
+    expect(setItemSpy).not.toHaveBeenCalled();
+    expect(handler.activeItem_BS.value).toEqual(newItem);
+  });
+
+  it('| should addItem without setItem', async (): Promise<void> => {
+    // Test case for lines 180-191: addItem without setItem
+    settings.setItem = undefined;
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+    await handler.reloadData('test');
+    
+    const newItem = { _id: '3', name: 'item3' } as TestMetadata;
+    await handler.addItem(newItem);
+    
+    expect(handler.dataList.find(i => i._id === '3')).toBeDefined();
+  });
+
+  it('| should removeItem when item not found', async (): Promise<void> => {
+    // Test case for lines 201-216: removeItem when index === -1
+    await handler.reloadData('test');
+    
+    await handler.removeItem('999');
+    
+    // Should not throw, just do nothing
+    expect(handler.dataList.length).toBe(2);
+  });
+
+  it('| should removeItem with updateBy parameter', async (): Promise<void> => {
+    // Test case for lines 201-216: removeItem with updateBy
+    await handler.reloadData('test');
+    const deleteItemSpy = spyOn(handler as any, 'deleteItem').and.returnValue(Promise.resolve());
+    
+    await handler.removeItem('1', 'updateByValue');
+    
+    expect(deleteItemSpy).toHaveBeenCalledWith('1', 'updateByValue');
+  });
+
+  it('| should removeItem with dontSendUpdate', async (): Promise<void> => {
+    // Test case for lines 201-216: removeItem with dontSendUpdate
+    await handler.reloadData('test');
+    settings.deleteItem = undefined;
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+    await handler.reloadData('test');
+    
+    await handler.removeItem('1', undefined, true);
+    
+    expect(handler.dataList.find(i => i._id === '1')).toBeUndefined();
+  });
+
+  it('| should updateItem with updateBy parameter', async (): Promise<void> => {
+    // Test case for lines 227-247: updateItem with updateBy
+    await handler.reloadData('test');
+    const setItemSpy = spyOn(handler as any, 'setItem').and.returnValue(Promise.resolve(handler.dataList[0]));
+    
+    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
+    await handler.updateItem(updatedItem, 'updateByValue');
+    
+    expect(setItemSpy).toHaveBeenCalledWith(updatedItem, 'updateByValue');
+  });
+
+  it('| should updateItem with dontSendUpdate', async (): Promise<void> => {
+    // Test case for lines 227-247: updateItem with dontSendUpdate
+    await handler.reloadData('test');
+    settings.setItem = undefined;
+    handler = new DyFM_DataListHandler<TestMetadata>(settings);
+    await handler.reloadData('test');
+    
+    const updatedItem = { _id: '1', name: 'updated' } as TestMetadata;
+    await handler.updateItem(updatedItem, undefined, true);
+    
+    expect(handler.dataList.find(i => i._id === '1')?.name).toBe('updated');
+  });
+});
+