From 5ddc88ec6846ee7bdcf7ccbe32490a482ad76433 Mon Sep 17 00:00:00 2001
From: Andres Pinto <apinto@shokworks.io>
Date: Thu, 28 Dec 2023 18:43:24 -0400
Subject: [PATCH] fix: removed unnecesary jest.resetmodules() in test cases to
 prevent sequelize from failing

---
 src/modules/folder/folder.e2e-spec.ts   | 6 ------
 src/modules/folder/folder.repository.ts | 3 +--
 2 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/modules/folder/folder.e2e-spec.ts b/src/modules/folder/folder.e2e-spec.ts
index 584faa6c8..9313d0f8e 100644
--- a/src/modules/folder/folder.e2e-spec.ts
+++ b/src/modules/folder/folder.e2e-spec.ts
@@ -39,13 +39,8 @@ describe('Folder module', () => {
   let userTestRerpository: UserTestRepository;
 
   beforeAll(async () => {
-    jest.resetModules();
-
     folderTestRepository = new FolderTestRepository(sequelizeTest);
     userTestRerpository = new UserTestRepository(sequelizeTest);
-
-    await sequelizeTest.authenticate();
-    await sequelizeTest.sync();
     user = await userTestRerpository.getPrincipalUser();
     userToken = userTestRerpository.generateToken(user, getEnv().secrets.jwt);
 
@@ -211,7 +206,6 @@ describe('Folder module', () => {
   });
 
   describe('Delete /folders - Delete Folders', () => {
-    // TODO: fix this test in the pipeline, in local it runs fine
     it.skip('should delete all orphan folders', async () => {
       const uuid = v4();
 
diff --git a/src/modules/folder/folder.repository.ts b/src/modules/folder/folder.repository.ts
index 750926edd..825a4d058 100644
--- a/src/modules/folder/folder.repository.ts
+++ b/src/modules/folder/folder.repository.ts
@@ -439,8 +439,7 @@ export class SequelizeFolderRepository implements FolderRepository {
           [Op.not]: null,
         },
       },
-      //  Sequelize throws an unknown error on e2e tests additionlOrders is passed here.
-      order: process.env.NODE_ENV === 'test' ? null : additionalOrders,
+      order: additionalOrders,
       limit,
       offset,
     });