Fix: Jest did not exit one second after the test run has completed ; …

…update mobile dependencies

api/__tests__/booking.test.ts

@@ -135,36 +135,34 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
-
-    // delete the supplier
-    await testHelper.deleteSupplier(SUPPLIER_ID)
-
-    // delete the location
-    await testHelper.deleteLocation(LOCATION_ID)
-
-    // delete the car
-    await Car.deleteMany({ _id: { $in: [CAR1_ID, CAR2_ID] } })
-
-    // delete drivers
-    // await User.deleteMany({ _id: { $in: [DRIVER1_ID, DRIVER2_ID] } })
-    const drivers = await User.find({ _id: { $in: [DRIVER1_ID, DRIVER2_ID] } })
-    expect(drivers.length).toBe(2)
-    for (const driver of drivers) {
-      if (driver.license) {
-        const license = path.join(env.CDN_LICENSES, driver.license)
-        if (await helper.exists(license)) {
-          await fs.unlink(license)
-        }
+  await testHelper.close()
+
+  // delete the supplier
+  await testHelper.deleteSupplier(SUPPLIER_ID)
+
+  // delete the location
+  await testHelper.deleteLocation(LOCATION_ID)
+
+  // delete the car
+  await Car.deleteMany({ _id: { $in: [CAR1_ID, CAR2_ID] } })
+
+  // delete drivers
+  // await User.deleteMany({ _id: { $in: [DRIVER1_ID, DRIVER2_ID] } })
+  const drivers = await User.find({ _id: { $in: [DRIVER1_ID, DRIVER2_ID] } })
+  expect(drivers.length).toBe(2)
+  for (const driver of drivers) {
+    if (driver.license) {
+      const license = path.join(env.CDN_LICENSES, driver.license)
+      if (await helper.exists(license)) {
+        await fs.unlink(license)
       }
-      await driver.deleteOne()
     }
-    await Notification.deleteMany({ user: { $in: [DRIVER1_ID, DRIVER2_ID] } })
-    await NotificationCounter.deleteMany({ user: { $in: [DRIVER1_ID, DRIVER2_ID] } })
-
-    await databaseHelper.close()
+    await driver.deleteOne()
   }
+  await Notification.deleteMany({ user: { $in: [DRIVER1_ID, DRIVER2_ID] } })
+  await NotificationCounter.deleteMany({ user: { $in: [DRIVER1_ID, DRIVER2_ID] } })
+
+  await databaseHelper.close()
 })
 
 //

api/__tests__/car.test.ts

@@ -4,7 +4,6 @@ import url from 'url'
 import path from 'path'
 import fs from 'node:fs/promises'
 import { nanoid } from 'nanoid'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import * as databaseHelper from '../src/common/databaseHelper'
 import app from '../src/app'
@@ -54,19 +53,17 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
+  await testHelper.close()
 
-    // delete suppliers
-    await testHelper.deleteSupplier(SUPPLIER1_ID)
-    await testHelper.deleteSupplier(SUPPLIER2_ID)
+  // delete suppliers
+  await testHelper.deleteSupplier(SUPPLIER1_ID)
+  await testHelper.deleteSupplier(SUPPLIER2_ID)
 
-    // delete locations
-    await testHelper.deleteLocation(LOCATION1_ID)
-    await testHelper.deleteLocation(LOCATION2_ID)
+  // delete locations
+  await testHelper.deleteLocation(LOCATION1_ID)
+  await testHelper.deleteLocation(LOCATION2_ID)
 
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 //

api/__tests__/country.test.ts

@@ -1,7 +1,6 @@
 import 'dotenv/config'
 import request from 'supertest'
 import { nanoid } from 'nanoid'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import app from '../src/app'
 import * as databaseHelper from '../src/common/databaseHelper'
@@ -39,10 +38,8 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
-    await databaseHelper.close()
-  }
+  await testHelper.close()
+  await databaseHelper.close()
 })
 
 //

api/__tests__/globalSetup.ts

@@ -1,6 +1,5 @@
 import 'dotenv/config'
 import bcrypt from 'bcrypt'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import * as env from '../src/config/env.config'
 import * as logger from '../src/common/logger'
@@ -28,10 +27,8 @@ export default async function globalSetup() {
         await admin.save()
         logger.info('globalSetup: Admin user created:', admin.id)
       }
-      if (mongoose.connection.readyState) {
-        await databaseHelper.close()
-        logger.info('Database connection closed')
-      }
+      await databaseHelper.close()
+      logger.info('Database connection closed')
     }
   } catch (err) {
     logger.error('Error while running global setup', err)

api/__tests__/index.test.ts

@@ -1,5 +1,4 @@
 import 'dotenv/config'
-import mongoose from 'mongoose'
 import * as databaseHelper from '../src/common/databaseHelper'
 import * as testHelper from './testHelper'
 import * as env from '../src/config/env.config'
@@ -21,9 +20,7 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 const createBookingIndex = async (expireAfterSeconds: number): Promise<void> => {

api/__tests__/location.test.ts

@@ -4,7 +4,7 @@ import url from 'url'
 import path from 'path'
 import fs from 'node:fs/promises'
 import { nanoid } from 'nanoid'
-import mongoose, { FlattenMaps } from 'mongoose'
+import { FlattenMaps } from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import app from '../src/app'
 import * as databaseHelper from '../src/common/databaseHelper'
@@ -71,10 +71,8 @@ afterAll(async () => {
   await LocationValue.deleteMany({ _id: { $in: [countryValue1Id, countryValue2Id] } })
   await Country.deleteOne({ _id: countryId })
 
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
-    await databaseHelper.close()
-  }
+  await testHelper.close()
+  await databaseHelper.close()
 })
 
 //

api/__tests__/middleware.test.ts

@@ -1,6 +1,5 @@
 import 'dotenv/config'
 import request from 'supertest'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import * as databaseHelper from '../src/common/databaseHelper'
 import app from '../src/app'
@@ -28,10 +27,8 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
-    await databaseHelper.close()
-  }
+  await testHelper.close()
+  await databaseHelper.close()
 })
 
 describe('POST /api/sign-in/backend', () => {

api/__tests__/miscellaneous.test.ts

@@ -1,5 +1,4 @@
 import 'dotenv/config'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import * as env from '../src/config/env.config'
 import * as databaseHelper from '../src/common/databaseHelper'
@@ -22,9 +21,7 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 const ADDITIONAL_DRIVER: bookcarsTypes.AdditionalDriver = {

api/__tests__/notification.test.ts

@@ -1,6 +1,5 @@
 import 'dotenv/config'
 import request from 'supertest'
-import mongoose from 'mongoose'
 import * as databaseHelper from '../src/common/databaseHelper'
 import * as testHelper from './testHelper'
 import Notification from '../src/models/Notification'
@@ -42,17 +41,15 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
+  await testHelper.close()
 
-    await testHelper.deleteSupplier(SUPPLIER_ID)
+  await testHelper.deleteSupplier(SUPPLIER_ID)
 
-    // clear admin user notifications and notification counter
-    await Notification.deleteMany({ user: ADMIN_USER_ID })
-    await NotificationCounter.deleteOne({ user: ADMIN_USER_ID })
+  // clear admin user notifications and notification counter
+  await Notification.deleteMany({ user: ADMIN_USER_ID })
+  await NotificationCounter.deleteOne({ user: ADMIN_USER_ID })
 
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 //

api/__tests__/stripe.test.ts

@@ -1,6 +1,5 @@
 import 'dotenv/config'
 import request from 'supertest'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import app from '../src/app'
 import * as databaseHelper from '../src/common/databaseHelper'
@@ -23,9 +22,7 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 describe('POST /api/create-checkout-session', () => {

api/__tests__/supplier.test.ts

@@ -4,7 +4,6 @@ import url from 'url'
 import path from 'path'
 import fs from 'node:fs/promises'
 import { nanoid } from 'nanoid'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import * as databaseHelper from '../src/common/databaseHelper'
 import * as testHelper from './testHelper'
@@ -116,18 +115,16 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
+  await testHelper.close()
 
-    // delete suppliers
-    await testHelper.deleteSupplier(SUPPLIER1_ID)
-    await testHelper.deleteSupplier(SUPPLIER2_ID)
+  // delete suppliers
+  await testHelper.deleteSupplier(SUPPLIER1_ID)
+  await testHelper.deleteSupplier(SUPPLIER2_ID)
 
-    // delete cars
-    await Car.deleteMany({ _id: { $in: [CAR1_ID, CAR2_ID] } })
+  // delete cars
+  await Car.deleteMany({ _id: { $in: [CAR1_ID, CAR2_ID] } })
 
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 //

api/__tests__/user.test.ts

@@ -4,7 +4,6 @@ import url from 'url'
 import path from 'path'
 import fs from 'node:fs/promises'
 import { nanoid } from 'nanoid'
-import mongoose from 'mongoose'
 import * as bookcarsTypes from ':bookcars-types'
 import app from '../src/app'
 import * as databaseHelper from '../src/common/databaseHelper'
@@ -58,13 +57,11 @@ beforeAll(async () => {
 // Closing and cleaning the database connection after running the test suite
 //
 afterAll(async () => {
-  if (mongoose.connection.readyState) {
-    await testHelper.close()
+  await testHelper.close()
 
-    await Token.deleteMany({ user: { $in: [ADMIN_ID] } })
+  await Token.deleteMany({ user: { $in: [ADMIN_ID] } })
 
-    await databaseHelper.close()
-  }
+  await databaseHelper.close()
 })
 
 //

api/src/common/databaseHelper.ts

@@ -57,9 +57,7 @@ export const connect = async (uri: string, ssl: boolean, debug: boolean): Promis
  * @returns {Promise<void>}
  */
 export const close = async (force: boolean = false): Promise<void> => {
-  if (mongoose.connection.readyState) {
-    await mongoose.connection.close(force)
-  }
+  await mongoose.connection.close(force)
 }
 
 /**