ci: Fixed mongodb teardown issue in tests

Author: buddh4

common/config/rush/browser-approved-packages.json

@@ -10,6 +10,10 @@
       "name": "@ckpack/vue-color",
       "allowedCategories": [ "core", "lib" ]
     },
+    {
+      "name": "@css-inline/css-inline-linux-x64-gnu",
+      "allowedCategories": [ "core" ]
+    },
     {
       "name": "@ctrl/tinycolor",
       "allowedCategories": [ "lib" ]
@@ -314,6 +318,10 @@
       "name": "ts-jest",
       "allowedCategories": [ "core", "feature", "lib" ]
     },
+    {
+      "name": "tsconfig-paths",
+      "allowedCategories": [ "core" ]
+    },
     {
       "name": "tslib",
       "allowedCategories": [ "application", "core", "feature", "lib" ]

common/config/rush/pnpm-lock.yaml

@@ -1,15 +1,5 @@
-import log from 'why-is-node-running';
+import 'tsconfig-paths/register';
 import 'reflect-metadata';
-import { useDayJsDateTimeAdapter } from '@lyvely/dates';
+import { setupTests } from '@/testing';
 
-useDayJsDateTimeAdapter();
-
-afterAll(async () => {
-  //normal cleanup things
-  await new Promise((resolve) => {
-    setTimeout(() => {
-      log();
-      resolve('');
-    }, 500);
-  });
-});
+export default () => setupTests();

packages/core/api/jest.setup.ts

@@ -0,0 +1,3 @@
+import { tearDownTests } from '@/testing';
+
+export default async () => await tearDownTests();

packages/core/api/jest.teardown.ts

@@ -9,7 +9,7 @@
   "types": "dist/index.d.ts",
   "scripts": {
     "build": "gulp clean && tsc -p tsconfig.build.json && tsc-alias -p tsconfig.build.json && gulp copyAssets",
-    "test": "jest --silent",
+    "test": "jest",
     "lint": "eslint -c .eslintrc.cjs \"src/**/*.ts\" --fix",
     "format": "prettier src --write",
     "prettier": "prettier src --check",
@@ -104,14 +104,14 @@
     "depcheck": "^1.4",
     "dependency-cruiser": "^14.1",
     "bullmq": "5.1.11",
-    "why-is-node-running": "^2.2.2",
-    "cross-env": "^7.0"
+    "cross-env": "^7.0",
+    "tsconfig-paths": "^4.2.0"
   },
   "jest": {
+    "testEnvironment": "node",
     "rootDir": ".",
-    "setupFilesAfterEnv": [
-      "<rootDir>/jest.setup.ts"
-    ],
+    "globalSetup": "<rootDir>/jest.setup.ts",
+    "globalTeardown": "<rootDir>/jest.teardown.ts",
     "testRegex": "[^cypress]*\\.spec\\.ts$",
     "moduleFileExtensions": [
       "js",
@@ -138,7 +138,6 @@
     "collectCoverageFrom": [
       "src/**/*.ts"
     ],
-    "coverageDirectory": "./coverage",
-    "testEnvironment": "node"
+    "coverageDirectory": "./coverage"
   }
 }

packages/core/api/package.json

@@ -38,10 +38,6 @@ describe('AppConfigService', () => {
     await testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   describe('getAppConfig', () => {
     it('inject module config', async () => {
       const config: IAppConfig<{ test: { someConfig: boolean } }> = appConfigService.getAppConfig(<any>{});

packages/core/api/src/app-config/services/app-config.service.spec.ts

@@ -3,7 +3,7 @@ import { promises as fsPromises } from 'fs';
 import { join, dirname, isAbsolute } from 'path';
 import { merge } from '@lyvely/common';
 import { Logger } from '@nestjs/common';
-import { MongoClient } from 'mongodb';
+//import { MongoClient } from 'mongodb';
 import { COLLECTION_CONFIG } from './config.constants';
 
 const logger = new Logger('loadConfigs');
@@ -39,7 +39,7 @@ export const loadConfigs = (
 
     const { uri, dbName } = mergedConfig.mongodb;
     logger.log(`Load db config from uri: ${uri} dbName: ${dbName}`);
-    const client = new MongoClient(uri);
+    /* const client = new MongoClient(uri);
 
     try {
       await client.connect();
@@ -50,7 +50,8 @@ export const loadConfigs = (
       return dbConfig || {};
     } finally {
       await client.close();
-    }
+    }*/
+    return {};
   };
 
   return () => {

packages/core/api/src/config/lyvely-config.loader.ts

@@ -53,10 +53,6 @@ describe('content module', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   it('should be defined', () => {
     expect(contentService).toBeDefined();
     expect(contentTypeRegistry).toBeDefined();

packages/core/api/src/content/content.module.spec.ts

@@ -38,10 +38,6 @@ describe('content dao', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   async function createTestContent(user: User, profile: Profile, testData = 'Testing...') {
     const testContent = new TestContent(
       { profile, user },

packages/core/api/src/content/daos/content.dao.spec.ts

@@ -53,10 +53,6 @@ describe('ContentReadPolicy', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   it('test custom content read policy', async () => {
     class TestContent extends Content {
       value: string;

packages/core/api/src/content/policies/content-read.policy.spec.ts

@@ -38,10 +38,6 @@ describe('ContentScore', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   it('should be defined', () => {
     expect(ExtendedTestContentScoreModel).toBeDefined();
   });

packages/core/api/src/content/schemas/content-score.schema.spec.ts

@@ -21,6 +21,10 @@ describe('Content Model', () => {
     profile = userAndProfile.profile;
   });
 
+  afterEach(async () => {
+    await testingModule.afterEach();
+  });
+
   describe('constructor', () => {
     it('content meta', async () => {
       const content = new Content({ profile, user });

packages/core/api/src/content/schemas/content.schema.spec.ts

@@ -26,10 +26,6 @@ describe('ContentPermissionsService', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   function registerTestPermission(data?: Partial<IProfilePermission>) {
     registerPermissions([
       {

packages/core/api/src/content/services/content-permissions.service.spec.ts

@@ -54,10 +54,6 @@ describe('ContentScoreService', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   it('should be defined', () => {
     expect(contentScoreService).toBeDefined();
   });

packages/core/api/src/content/services/content-score.service.spec.ts

@@ -34,10 +34,6 @@ describe('content dao', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   async function createTestContent(user: User, profile: Profile, archived = false) {
     const testContent = new TestContent(
       { profile, user },

packages/core/api/src/content/services/content.service.spec.ts

@@ -1,5 +1,5 @@
 import { ModuleRegistry } from './components';
-import { afterAllTests, afterEachTest, createCoreTestingModule } from './testing';
+import { afterEachTest, createCoreTestingModule } from './testing';
 import { TestingModule } from '@nestjs/testing';
 
 describe('CoreModule', () => {
@@ -17,10 +17,6 @@ describe('CoreModule', () => {
     await afterEachTest(TEST_KEY, testingModule);
   });
 
-  afterAll(async () => {
-    await afterAllTests();
-  });
-
   it('registry is defined', () => {
     expect(moduleRegistry).toBeDefined();
   });

packages/core/api/src/core/core.module.spec.ts

@@ -1,7 +1,7 @@
 import { InjectModel, Prop, Schema, SchemaFactory } from '@nestjs/mongoose';
 import { AbstractDao } from './abstract.dao';
 import { Model, TObjectId } from './db.type';
-import { createCoreTestingModule, afterEachTest, afterAllTests } from '../testing/core-test.util';
+import { createCoreTestingModule, afterEachTest } from '../testing/core-test.util';
 import { ModelDefinition } from '@nestjs/mongoose/dist/interfaces';
 import { Injectable } from '@nestjs/common';
 import { TestingModule } from '@nestjs/testing';
@@ -107,10 +107,6 @@ describe('AbstractDao', () => {
     await afterEachTest(TEST_KEY, testingModule);
   });
 
-  afterAll(async () => {
-    await afterAllTests();
-  });
-
   it('should be defined', () => {
     expect(dao).toBeDefined();
   });

packages/core/api/src/core/db/abstract.dao.discriminator-transformation.spec.ts

@@ -7,7 +7,6 @@ import {
   EventTester,
   getObjectId,
   afterEachTest,
-  afterAllTests,
 } from '../testing/core-test.util';
 import { ModelDefinition } from '@nestjs/mongoose/dist/interfaces';
 import { Injectable } from '@nestjs/common';
@@ -88,10 +87,6 @@ describe('AbstractDao', () => {
     await afterEachTest(TEST_KEY, testingModule);
   });
 
-  afterAll(async () => {
-    await afterAllTests();
-  });
-
   it('should be defined', () => {
     expect(dao).toBeDefined();
   });

packages/core/api/src/core/db/abstract.dao.spec.ts

@@ -5,7 +5,6 @@ import {
   createCoreTestingModule,
   EventTester,
   afterEachTest,
-  afterAllTests,
 } from '../testing/core-test.util';
 import { ModelDefinition } from '@nestjs/mongoose/dist/interfaces';
 import { Injectable } from '@nestjs/common';
@@ -123,10 +122,6 @@ describe('AbstractDao Transformations', () => {
     await afterEachTest(TEST_KEY, testingModule);
   });
 
-  afterAll(async () => {
-    await afterAllTests();
-  });
-
   describe('transform', () => {
     it('transform on findById', async () => {
       const v1Model = await ModelV1.create(new TestEntityV1({ numberField: 3 }));

packages/core/api/src/core/db/abstract.dao.transformation.spec.ts

@@ -1,5 +1,5 @@
 import { UrlGenerator } from './url-generator.service';
-import { afterAllTests, afterEachTest, createCoreTestingModule } from '../testing';
+import { afterEachTest, createCoreTestingModule } from '../testing';
 import { TestingModule } from '@nestjs/testing';
 
 describe('UrlGenerator', () => {
@@ -17,10 +17,6 @@ describe('UrlGenerator', () => {
     await afterEachTest(TEST_KEY, testingModule);
   });
 
-  afterAll(async () => {
-    await afterAllTests();
-  });
-
   it('should be defined', () => {
     expect(urlGenerator).toBeDefined();
   });

packages/core/api/src/core/services/url-generator.service.spec.ts

@@ -8,7 +8,6 @@ import { FeatureType } from "@lyvely/interface";
 describe('ProfileGuard', () => {
   let testingModule: ILyvelyTestingModule;
   let featureGuard: FeatureGuard;
-  //let featureRegistry: FeatureRegistry;
   let context: ExecutionContext;
 
   const TEST_KEY = 'feature-guard';
@@ -23,9 +22,6 @@ describe('ProfileGuard', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
   describe('canActivate()', () => {
     it('enabled class level global feature', async () => {
       registerFeatures([

packages/core/api/src/features/guards/feature.guard.spec.ts

@@ -28,10 +28,6 @@ describe('StorageService', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   class TestProvider extends StorageProvider<any> {
     async upload(upload: FileUpload): Promise<void> {
       /** Nothing to do **/

packages/core/api/src/files/services/storage.service.spec.ts

@@ -17,10 +17,6 @@ describe('I18n', () => {
     return testingModule.afterEach();
   });
 
-  afterAll(async () => {
-    return testingModule.afterAll();
-  });
-
   it('should be defined', () => {
     expect(i18n).toBeDefined();
   });

packages/core/api/src/i18n/components/i18n.component.spec.ts

@@ -1,15 +1,20 @@
-import { buildTest } from '@/testing';
+import { buildTest, ILyvelyTestingModule } from '@/testing';
 import { i18nITestPlugin } from '../testing';
 import { I18n } from '../components';
 
 describe('I18nModuleLoader', () => {
   let i18n: I18n;
+  let testingModule: ILyvelyTestingModule;
 
   beforeEach(async () => {
-    const test = await buildTest('I18nModuleLoader').plugins([i18nITestPlugin]).compile();
-    i18n = test.get(I18n);
+    testingModule = await buildTest('I18nModuleLoader').plugins([i18nITestPlugin]).compile();
+    i18n = testingModule.get(I18n);
   }, 50000);
 
+  afterEach(async () => {
+    await testingModule.afterEach();
+  });
+
   it('Test module prefix', async () => {
     const translation = i18n.t('i18n.test.hello', { locale: 'de' });
     expect(translation).toEqual('Hallo');