fix: removed unnecesary jest.resetmodules() in test cases to prevent …

…sequelize from failing
internxt · Dec 28, 2023 · 5ddc88e · 5ddc88e
1 parent b60afe2
commit 5ddc88e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git 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
@@ -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,
     });