fix unit tests

tests/units/services/data/entityManager.test.ts

@@ -187,18 +187,14 @@ describe('newInstance', () => {
 
     // @ts-ignore
     entityManager.save = vi.fn(
-      (model: typeof ApiResource, entity: ApiResource, permanent: boolean) =>
+      (entity: ApiResource, permanent: boolean) =>
         entity,
     )
 
     const result = entityManager.newInstance(DummyApiResource, properties)
 
     expect(repo.make).toHaveBeenCalledWith(properties)
-    expect(entityManager.save).toHaveBeenCalledWith(
-      DummyApiResource,
-      entity,
-      true,
-    )
+    expect(entityManager.save).toHaveBeenCalledWith(entity, true)
 
     expect(result.id).toEqual(properties.id)
   })
@@ -254,8 +250,8 @@ describe('save', () => {
     // @ts-ignore
     repo.save = vi.fn((record: Element) => entity)
 
-    const entity = new DummyApiResource()
-    entityManager.save(DummyApiResource, entity)
+    const entity = new DummyApiResource({ id: 1 })
+    entityManager.save(entity)
 
     expect(repo.save).toHaveBeenCalledWith(entity)
   })
@@ -536,7 +532,7 @@ describe('persist', () => {
     entityManager.removeTempAfterPersist = vi.fn()
     entityManager.makeProfileHash = vi.fn(async () => await 'azerty')
 
-    const result = await entityManager.persist(DummyApiModel, instance)
+    const result = await entityManager.persist(instance)
 
     // temp id should have been purged from the posted data
     expect(apiRequestService.post).toHaveBeenCalledWith(
@@ -593,7 +589,7 @@ describe('persist', () => {
     entityManager.removeTempAfterPersist = vi.fn()
     entityManager.makeProfileHash = vi.fn(async () => await 'azerty')
 
-    const result = await entityManager.persist(DummyApiModel, entity)
+    const result = await entityManager.persist(entity)
 
     expect(apiRequestService.put).toHaveBeenCalledWith(
       'api/dummyModel/1',
@@ -670,7 +666,7 @@ describe('delete', () => {
     // @ts-ignore
     repo.destroy = vi.fn((id: number) => null)
 
-    entityManager.delete(DummyApiModel, entity)
+    entityManager.delete(entity)
 
     expect(entityManager.getRepository).toHaveBeenCalledWith(DummyApiModel)
     expect(apiRequestService.delete).toHaveBeenCalledTimes(0)
@@ -696,7 +692,7 @@ describe('delete', () => {
     // @ts-ignore
     repo.destroy = vi.fn((id: number) => null)
 
-    await entityManager.delete(DummyApiModel, entity)
+    await entityManager.delete(entity)
 
     expect(entityManager.getRepository).toHaveBeenCalledWith(DummyApiModel)
     expect(apiRequestService.delete).toHaveBeenCalledWith('api/dummyModel/1')