@villedemontreal/mongo 8.2.0 → 9.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@villedemontreal/mongo",
-  "version": "8.2.0",
+  "version": "9.0.0",
   "description": "Utilities for Mongo / Mongoose",
   "main": "dist/src/index.js",
   "typings": "dist/src",
@@ -14,8 +14,8 @@
     "compile": "tsc --build",
     "show-tsconfig": "tsc --showConfig",
     "show-coverage": "open output/coverage/lcov-report/index.html",
-    "lint": "eslint -c .eslintrc.js --ext .ts src",
-    "lint-fix": "eslint -c .eslintrc.js --fix --ext .ts src",
+    "lint": "eslint",
+    "lint-fix": "eslint --fix",
     "ncu": "ncu --target minor -x @types/node",
     "ncu-fix": "ncu -u --target minor -x @types/node"
   },
@@ -33,39 +33,50 @@
   "author": "Ville de Montréal",
   "license": "MIT",
   "dependencies": {
-    "@villedemontreal/general-utils": "5.21.0",
-    "@villedemontreal/logger": "7.1.0",
+    "@villedemontreal/general-utils": "6.0.0",
+    "@villedemontreal/logger": "8.0.0",
     "app-root-path": "3.1.0",
-    "fs-extra": "11.3.2",
+    "fs-extra": "11.3.3",
     "http-status-codes": "2.3.0",
     "lodash": "4.17.21",
-    "mongodb": "6.20.0",
-    "mongodb-memory-server-core": "10.3.0",
-    "mongoose": "8.20.1",
+    "mongodb": "7.0.0",
+    "mongodb-memory-server-core": "11.0.1",
+    "mongoose": "9.0.2",
     "semver": "7.7.3"
   },
   "devDependencies": {
-    "@types/chai": "4.3.20",
+    "@eslint/eslintrc": "3.3.3",
+    "@eslint/js": "^9.39.2",
+    "@types/chai": "5.2.3",
     "@types/fs-extra": "11.0.4",
     "@types/http-status-codes": "1.2.0",
     "@types/lodash": "4.17.21",
     "@types/mocha": "10.0.10",
     "@types/semver": "7.7.1",
-    "@types/sinon": "17.0.4",
-    "@typescript-eslint/eslint-plugin": "5.62.0",
-    "@typescript-eslint/parser": "5.62.0",
-    "chai": "4.5.0",
-    "eslint": "8.57.1",
-    "eslint-config-prettier": "8.10.2",
+    "@types/sinon": "21.0.0",
+    "@typescript-eslint/eslint-plugin": "8.50.1",
+    "@typescript-eslint/parser": "8.50.1",
+    "@typescript-eslint/utils": "^8.50.1",
+    "chai": "6.2.2",
+    "eslint": "^9.39.2",
+    "eslint-config-prettier": "10.1.8",
     "eslint-plugin-prettier": "5.5.4",
+    "globals": "^16.5.0",
     "mocha": "11.7.5",
     "nyc": "17.1.0",
-    "prettier": "3.7.3",
-    "sinon": "21.0.0",
+    "prettier": "3.7.4",
+    "sinon": "21.0.1",
     "ts-node": "^10.9.2",
     "typescript": "5.9.3"
   },
   "override": {
     "mongodb": "$mongodb"
+  },
+  "config": {
+    "mongodbMemoryServer": {
+      "resumeDownload": true,
+      "maxRetries": 5,
+      "debug": false
+    }
   }
 }

package/src/config/configs.ts

@@ -11,7 +11,7 @@ export class Configs {
   private _loggerCreator: (name: string) => ILogger;
 
   constructor() {
-    this.libRoot = path.normalize(__dirname + '/../..');
+    this.libRoot = path.normalize(path.join(__dirname, '../..'));
     this.isWindows = os.platform() === 'win32';
   }
 

package/src/config/constants.ts

@@ -26,7 +26,7 @@ export class Constants {
 
   constructor() {
     // From the "dist/src/config" folder
-    this.libRoot = path.normalize(__dirname + '/../..');
+    this.libRoot = path.normalize(path.join(__dirname, '../..'));
     this.appRoot = appRoot;
   }
 

package/src/config/mongooseConfigs.ts

@@ -103,7 +103,6 @@ export class MongooseConfigs implements IMongooseConfigs {
    * Overrides default configurations using the ones passed
    * as parameters.
    */
-  // tslint:disable-next-line:cyclomatic-complexity
   constructor(overridingConfigs: IMongooseConfigs) {
     // ==========================================
     // Required configs

package/src/mongoClient.ts

@@ -98,6 +98,7 @@ export async function initMongoose(mongooseConfig: IMongooseConfigs): Promise<mo
 
   return new Promise<mongoose.Connection>((resolve, reject) => {
     doInitMongoose(mongooseConfigClean, resolve, reject).catch((err) => {
+      // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
       reject(`Error initializing Mongo: ${err}`);
     });
   });

package/src/mongoUpdater.test.ts

@@ -1,7 +1,3 @@
-// Ok for test files :
-// tslint:disable:max-func-body-length
-// tslint:disable:no-string-literal
-
 import { Timer } from '@villedemontreal/general-utils';
 import { assert } from 'chai';
 import * as MongoDb from 'mongodb';

package/src/mongoUpdater.ts

@@ -5,6 +5,7 @@ import * as MongoDb from 'mongodb';
 import * as semver from 'semver';
 import { constants as mongodbConstants } from './config/constants';
 import { createLogger } from './utils/logger';
+import * as path from 'path';
 
 const logger = createLogger('MongoUpdater');
 
@@ -95,8 +96,7 @@ export class MongoUpdater implements IMongoUpdater {
   public async installAppSchemaCollection(): Promise<any> {
     try {
       // Installing the "appSchema" collection.
-      // tslint:disable-next-line: prefer-template
-      logger.info(' > Installing the "' + this.appSchemaCollectionName + '" collection.');
+      logger.info(` > Installing the "${this.appSchemaCollectionName}" collection.`);
       const collection: MongoDb.Collection = await this.mongoDb.createCollection(
         this.appSchemaCollectionName,
       );
@@ -160,9 +160,8 @@ export class MongoUpdater implements IMongoUpdater {
     const appSchemaCollection: MongoDb.Collection = await this.getAppSchemaCollection();
 
     await appSchemaCollection.updateOne({}, { $set: { version: newVersion } });
-    // tslint:disable-next-line: prefer-template
     logger.info(
-      ' > MongoDB App Schema updagred from version ' + currentVersion + ' to version ' + newVersion,
+      ` > MongoDB App Schema updagred from version ${currentVersion} to version ${newVersion}`,
     );
   }
 
@@ -196,6 +195,7 @@ export class MongoUpdater implements IMongoUpdater {
               return false;
             });
         } catch (err2) {
+          // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
           return reject(err2);
         }
         return resolve(filesClean.sort(semver.compare));
@@ -210,21 +210,22 @@ export class MongoUpdater implements IMongoUpdater {
     );
     if (updateFileNames.length > 0) {
       for (const updateFileName of updateFileNames) {
-        logger.info(' > Pending app schema update: ' + updateFileName);
+        logger.info(` > Pending app schema update: ${updateFileName}`);
 
-        // tslint:disable-next-line: prefer-template
-        const updateFilePath = this.getAppSchemaFilesDirPath() + '/' + updateFileName;
+        const updateFilePath = path.join(this.getAppSchemaFilesDirPath(), updateFileName);
         let updateFunction: (db: MongoDb.Db) => Promise<void>;
         try {
-          updateFunction = require(updateFilePath).default;
+          const pkg = await import(updateFilePath);
+          updateFunction = pkg.default;
         } catch (e) {
+          // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
           return Promise.reject(e);
         }
 
         if (!isFunction(updateFunction)) {
+          // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
           return Promise.reject(
-            'The default export for an app schema update file must be a function! Was not for file : ' +
-              updateFilePath,
+            `The default export for an app schema update file must be a function! Was not for file : '${updateFilePath}'`,
           );
         }
 
@@ -377,9 +378,8 @@ export class MongoUpdater implements IMongoUpdater {
         // ==========================================
         currentAppSchemaVersion = await this.getAppSchemaVersion();
         if (semver.gte(currentAppSchemaVersion, targetVersion)) {
-          // tslint:disable-next-line: prefer-template
           logger.info(
-            ' > Current database app schema is up to date : ' + currentAppSchemaVersion + ').',
+            ` > Current database app schema is up to date : ${currentAppSchemaVersion}).`,
           );
           return;
         }
@@ -415,7 +415,7 @@ export class MongoUpdater implements IMongoUpdater {
   };
 
   protected getAppSchemaFilesDirPath(): string {
-    return mongodbConstants.appRoot + this.mongoSchemaUpdatesDirPath;
+    return path.join(mongodbConstants.appRoot, this.mongoSchemaUpdatesDirPath);
   }
 
   /**

package/src/mongoUtils.ts

@@ -2,7 +2,6 @@
 // Mongo utilities
 // ==========================================
 import {
-  ApiErrorAndInfo,
   createError,
   createInvalidParameterError,
   createNotFoundError,
@@ -19,6 +18,7 @@ import * as mocha from 'mocha';
 import { MongoMemoryReplSet, MongoMemoryServer } from 'mongodb-memory-server-core';
 import * as mongoose from 'mongoose';
 import { constants } from './config/constants';
+import * as path from 'path';
 
 /**
  * Mongo utilities
@@ -67,7 +67,7 @@ export class MongoUtils {
       // container, the server will not be downloaded
       // over and over.
       // ==========================================
-      const downloadDirPath = constants.appRoot + '/temp/mockServer';
+      const downloadDirPath = path.join(constants.appRoot, 'temp', 'mockServer');
       if (!fs.existsSync(downloadDirPath)) {
         fs.mkdirsSync(downloadDirPath);
       }
@@ -75,13 +75,13 @@ export class MongoUtils {
       // ==========================================
       // Data directory
       // ==========================================
-      const dataRootPath = downloadDirPath + '/data';
+      const dataRootPath = path.join(downloadDirPath, 'data');
       if (!fs.existsSync(dataRootPath)) {
         fs.mkdirsSync(dataRootPath);
       } else {
         await utils.clearDir(dataRootPath);
       }
-      const dataPath = `${dataRootPath}/${Date.now()}`;
+      const dataPath = path.join(dataRootPath, Date.now().toString());
       fs.mkdirsSync(dataPath);
 
       // ==========================================
@@ -214,10 +214,7 @@ export class MongoUtils {
    * @param publicMessage a public message to be used in the
    * generated error. Fopr example : "The user is invalid".
    */
-  public convertMongoOrMongooseErrorToApiError(
-    err: any,
-    publicMessage: string,
-  ): ApiErrorAndInfo | any {
+  public convertMongoOrMongooseErrorToApiError(err: any, publicMessage: string): any {
     if (!err) {
       return createServerError('Empty error object');
     }
@@ -308,15 +305,12 @@ export class MongoUtils {
     // ==========================================
     // Converts the "_id" property to "id"
     // ==========================================
-    // tslint:disable-next-line:no-string-literal
     pojoObj['id'] = pojoObj['_id'].toString();
-    // tslint:disable-next-line:no-string-literal
     delete pojoObj['_id'];
 
     // ==========================================
     // Removes the "__v"
     // ==========================================
-    // tslint:disable-next-line:no-string-literal
     delete pojoObj['__v'];
 
     return pojo;

package/src/plugins/pagination/index.test.ts

@@ -1,6 +1,3 @@
-// Some way of using chai requires disabling this rule:
-// tslint:disable:no-unused-expression
-
 import { IPaginatedResult } from '@villedemontreal/general-utils';
 import * as chai from 'chai';
 import * as mongoose from 'mongoose';

package/src/plugins/pagination/index.ts

@@ -93,6 +93,7 @@ export class PaginateBuilder {
           if (typeof callback === 'function') {
             return callback(error, null);
           }
+          // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
           reject(error);
         },
       );