test: fixing test and linting errors

Author: Christian Jeschke

jest-preset.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line @typescript-eslint/no-require-imports
 const preset = require('./lib').default;
 
 module.exports = preset;

readme.md

@@ -134,10 +134,7 @@ describe('insert', () => {
   let db;
 
   beforeAll(async () => {
-    connection = await MongoClient.connect(global.__MONGO_URI__, {
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-    });
+    connection = await MongoClient.connect(global.__MONGO_URI__, {});
     db = await connection.db();
   });
 

src/environment.ts

@@ -7,6 +7,7 @@ import type {EnvironmentContext} from '@jest/environment';
 import type {JestEnvironmentConfig} from '@jest/environment';
 import {getMongodbMemoryOptions, isMongoMemoryReplSetOptions} from './helpers';
 
+// eslint-disable-next-line @typescript-eslint/no-require-imports
 const debug = require('debug')('jest-mongodb:environment');
 
 module.exports = class MongoEnvironment extends TestEnvironment {
@@ -49,9 +50,9 @@ module.exports = class MongoEnvironment extends TestEnvironment {
     await super.teardown();
   }
 
-  // @ts-ignore
+  // @ts-expect-error - runScript has incompatible type definitions
   runScript(script) {
-    // @ts-ignore
+    // @ts-expect-error - parent class method call
     return super.runScript(script);
   }
 };

src/helpers.ts

@@ -17,6 +17,7 @@ export function getMongodbMemoryOptions(
   cwd: string
 ): MongoMemoryReplSetOpts | MongoMemoryServerOpts | undefined {
   try {
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
     const {mongodbMemoryServerOptions}: Config = require(resolve(cwd, configFile));
 
     return mongodbMemoryServerOptions;
@@ -32,24 +33,26 @@ export function getMongodbMemoryOptions(
 
 export function getMongoURLEnvName(cwd: string) {
   try {
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
     const {mongoURLEnvName}: Config = require(resolve(cwd, configFile));
 
     return mongoURLEnvName || 'MONGO_URL';
-  } catch (e) {
+  } catch {
     return 'MONGO_URL';
   }
 }
 
 export function shouldUseSharedDBForAllJestWorkers(cwd: string) {
   try {
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
     const {useSharedDBForAllJestWorkers}: Config = require(resolve(cwd, configFile));
 
     if (typeof useSharedDBForAllJestWorkers === 'undefined') {
       return true;
     }
 
     return useSharedDBForAllJestWorkers;
-  } catch (e) {
+  } catch {
     return true;
   }
 }

src/setup.ts

@@ -11,7 +11,8 @@ import {
 } from './helpers';
 import {isMongoMemoryReplSetOptions} from './helpers';
 
-const debug = require('debug')('jest-mongodb:setup');
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const debug = require('debug')('jest-mongodb:environment');
 
 module.exports = async (config: JestEnvironmentConfig['globalConfig']) => {
   const globalConfigPath = join(config.rootDir, 'globalConfig.json');

src/teardown.ts

@@ -2,7 +2,8 @@ import {join} from 'path';
 import {unlink} from 'fs';
 import type {JestEnvironmentConfig} from '@jest/environment';
 
-const debug = require('debug')('jest-mongodb:teardown');
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const debug = require('debug')('jest-mongodb:environment');
 
 module.exports = async function (config: JestEnvironmentConfig['globalConfig']) {
   const globalConfigPath = join(config.rootDir, 'globalConfig.json');

test/mongo-aggregate.test.ts

@@ -8,12 +8,7 @@ describe('insert', () => {
   let db: Db;
 
   beforeAll(async () => {
-    // @ts-ignore
-    connection = await MongoClient.connect(uri, {
-      // @ts-ignore
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-    });
+    connection = await MongoClient.connect(uri, {});
     db = await connection.db();
   });
 

test/mongo-insert.test.ts

@@ -8,12 +8,7 @@ describe('insert', () => {
   let db: Db;
 
   beforeAll(async () => {
-    // @ts-ignore
-    connection = await MongoClient.connect(uri, {
-      // @ts-ignore
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-    });
+    connection = await MongoClient.connect(uri, {});
     db = await connection.db();
   });
 

test/mongo-parallelism.test.ts

@@ -9,12 +9,7 @@ describe('parallelism: first worker', () => {
   let db: Db;
 
   beforeAll(async () => {
-    // @ts-ignore
-    connection = await MongoClient.connect(uri, {
-      // @ts-ignore
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-    });
+    connection = await MongoClient.connect(uri, {});
     db = await connection.db();
   });
 

test/mongo-parallelism2.test.ts

@@ -9,12 +9,7 @@ describe('parallelism: second worker', () => {
   let db: Db;
 
   beforeAll(async () => {
-    // @ts-ignore
-    connection = await MongoClient.connect(uri, {
-      // @ts-ignore
-      useNewUrlParser: true,
-      useUnifiedTopology: true,
-    });
+    connection = await MongoClient.connect(uri, {});
     db = await connection.db();
   });