From 26be27f8149e76cb8f6ebc4053b83aa1a838ca2e Mon Sep 17 00:00:00 2001 From: dainty92 Date: Mon, 3 Mar 2025 09:43:13 +0100 Subject: [PATCH] fix: organisations.service.spec.ts --- .../organisations/tests/organisations.service.spec.ts | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/modules/organisations/tests/organisations.service.spec.ts b/src/modules/organisations/tests/organisations.service.spec.ts index edf599a6..5c817b44 100644 --- a/src/modules/organisations/tests/organisations.service.spec.ts +++ b/src/modules/organisations/tests/organisations.service.spec.ts @@ -265,11 +265,10 @@ describe('OrganisationsService', () => { describe('deleteorganisation', () => { it('should mark an organisation as deleted', async () => { const orgId = 'org-id'; - const mockOrganisation = { id: orgId, isDeleted: false } as Organisation; - // Ensure `findOneBy` is mocked correctly - jest.spyOn(organisationRepository, 'findOneBy').mockResolvedValue(mockOrganisation); - jest.spyOn(organisationRepository, 'update').mockResolvedValue({ affected: 1 } as any); + // Mock repository functions + organisationRepository.update = jest.fn().mockResolvedValue({ affected: 1 }); + organisationRepository.findOneBy = jest.fn().mockResolvedValue({ id: orgId, isDeleted: true }); const result = await service.deleteorganisation(orgId); @@ -287,8 +286,8 @@ describe('OrganisationsService', () => { { id: 'org2', name: 'Deleted Org', isDeleted: true }, ] as Organisation[]; - // Ensure `findBy` is mocked instead of `find` - jest.spyOn(organisationRepository, 'findBy').mockResolvedValue(mockOrganisations.filter(org => !org.isDeleted)); + // Mock `findBy` to only return non-deleted organisations + organisationRepository.findBy = jest.fn().mockResolvedValue(mockOrganisations.filter(org => !org.isDeleted)); const result = await service.getUserOrganisations('test-user-id');