Refactor tests

.eslintignore

@@ -1,2 +1,4 @@
 dist/**
 node_modules/**
+*.cjs
+*.mjs

.eslintrc.json

@@ -4,7 +4,8 @@
     "sourceType": "module"
   },
   "extends": [
-    "@myrotvorets/myrotvorets-ts"
+    "@myrotvorets/myrotvorets-ts",
+    "plugin:mocha/recommended"
   ],
   "env": {
     "es2022": true,

.mocharc.cjs

@@ -3,11 +3,10 @@ module.exports = {
     recursive: true,
     spec: ['test/**/*.test.mts'],
     'node-option': ['loader=ts-node/esm', 'no-warnings'],
-    // require: 'mocha.setup.mjs',
     reporter: 'mocha-multi',
     'reporter-option': [
         'spec=-',
         process.env.GITHUB_ACTIONS === 'true' ? 'mocha-reporter-gha=-' : null,
-        'mocha-reporter-sonarqube=test-report.xml'
+        process.env.SONARSCANNER === 'true' ? 'mocha-reporter-sonarqube=test-report.xml' : null,
     ].filter(Boolean),
 }

package.json

@@ -11,6 +11,7 @@
     "prepare": "npm run build",
     "pretest": "npm run lint",
     "test": "mocha",
+    "pretest:coverage": "npm run lint",
     "test:coverage": "c8 mocha"
   },
   "files": [
@@ -29,6 +30,7 @@
     "@types/supertest": "^2.0.10",
     "c8": "^8.0.1",
     "eslint-formatter-gha": "^1.2.0",
+    "eslint-plugin-mocha": "^10.2.0",
     "express": "^4.17.1",
     "express-openapi-validator": "^5.0.0",
     "mocha": "^10.2.0",

test/index.test.mts

@@ -1,6 +1,5 @@
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
-import { describe, it } from 'mocha';
 import express, { type Application, type NextFunction, type Request, type Response } from 'express';
 import request from 'supertest';
 import { installOpenApiValidator } from '../lib/index.mjs';
@@ -16,14 +15,14 @@ async function buildServer(install: boolean, env: string): Promise<Application>
         await installOpenApiValidator(join(dirname(fileURLToPath(import.meta.url)), 'openapi.yaml'), app, env);
     }
 
-    app.get('/test', (req: Request, res: Response): void => {
+    app.get('/test', (req, res): void => {
         res.json({
             s: req.query.s,
             debug: true,
         });
     });
 
-    app.get('/auth', (_req: Request, res: Response): unknown => res.status(204).end());
+    app.get('/auth', (_req, res): unknown => res.status(204).end());
 
     app.use((err: unknown, _req: Request, res: Response, _next: NextFunction) =>
         res.status((err as IWithStatus).status ?? 500).json(err),
@@ -32,65 +31,67 @@ async function buildServer(install: boolean, env: string): Promise<Application>
     return app;
 }
 
-describe('Without installOpenApiValidator', () => {
-    it('should return 200 for bad request', async (): Promise<unknown> => {
-        const server = await buildServer(false, '');
-        return request(server).get('/test').expect(200);
-    });
-});
-
-describe('With installOpenApiValidator', () => {
-    it('will not run security handlers', async (): Promise<unknown> => {
-        const server = await buildServer(true, 'test');
-        return request(server).get('/auth').expect(204);
+describe('installOpenApiValidator', function () {
+    describe('Without installOpenApiValidator', function () {
+        it('should return 200 for bad request', async function (): Promise<unknown> {
+            const server = await buildServer(false, '');
+            return request(server).get('/test').expect(200);
+        });
     });
 
-    describe('in test mode', () => {
-        // This one checks that `servers` section gets overwritten
-        // If `servers` is not overwritten, the URL won't match the base URL constraint
-        it('will validate all requests', async (): Promise<unknown> => {
+    describe('With installOpenApiValidator', function () {
+        it('will not run security handlers', async function (): Promise<unknown> {
             const server = await buildServer(true, 'test');
-            return request(server)
-                .get('/test')
-                .expect(400)
-                .expect(/\/query\/s/u);
+            return request(server).get('/auth').expect(204);
         });
 
-        it('will thoroughly validate requests', async (): Promise<unknown> => {
-            const server = await buildServer(true, 'test');
-            return request(server)
-                .get('/test?s=2012-13-31')
-                .expect(400)
-                .expect(/\/query\/s/u)
-                .expect(/must match format/u);
-        });
+        describe('in test mode', function () {
+            // This one checks that `servers` section gets overwritten
+            // If `servers` is not overwritten, the URL won't match the base URL constraint
+            it('will validate all requests', async function (): Promise<unknown> {
+                const server = await buildServer(true, 'test');
+                return request(server)
+                    .get('/test')
+                    .expect(400)
+                    .expect(/\/query\/s/u);
+            });
 
-        it('will validate responses', async (): Promise<unknown> => {
-            const server = await buildServer(true, 'test');
-            return request(server)
-                .get('/test?s=2012-12-31')
-                .expect(500)
-                .expect(/\/response\/debug/u);
-        });
-    });
+            it('will thoroughly validate requests', async function (): Promise<unknown> {
+                const server = await buildServer(true, 'test');
+                return request(server)
+                    .get('/test?s=2012-13-31')
+                    .expect(400)
+                    .expect(/\/query\/s/u)
+                    .expect(/must match format/u);
+            });
 
-    describe('in production mode', () => {
-        it('will validate all requests', async (): Promise<unknown> => {
-            const server = await buildServer(true, 'production');
-            return request(server)
-                .get('/test')
-                .expect(400)
-                .expect(/\/query\/s/u);
+            it('will validate responses', async function (): Promise<unknown> {
+                const server = await buildServer(true, 'test');
+                return request(server)
+                    .get('/test?s=2012-12-31')
+                    .expect(500)
+                    .expect(/\/response\/debug/u);
+            });
         });
 
-        it('will not thoroughly validate requests', async (): Promise<unknown> => {
-            const server = await buildServer(true, 'production');
-            return request(server).get('/test?s=2012-13-31').expect(200);
-        });
+        describe('in production mode', function () {
+            it('will validate all requests', async function (): Promise<unknown> {
+                const server = await buildServer(true, 'production');
+                return request(server)
+                    .get('/test')
+                    .expect(400)
+                    .expect(/\/query\/s/u);
+            });
+
+            it('will not thoroughly validate requests', async function (): Promise<unknown> {
+                const server = await buildServer(true, 'production');
+                return request(server).get('/test?s=2012-13-31').expect(200);
+            });
 
-        it('will not validate responses', async (): Promise<unknown> => {
-            const server = await buildServer(true, 'production');
-            return request(server).get('/test?s=2012-12-31').expect(200);
+            it('will not validate responses', async function (): Promise<unknown> {
+                const server = await buildServer(true, 'production');
+                return request(server).get('/test?s=2012-12-31').expect(200);
+            });
         });
     });
 });