fix(server): Configure jest

(WIP) Jest is detecting open handles on Redis' bee

Author: Henriquelay

server/jest.config.js

@@ -0,0 +1,10 @@
+import type { Config } from '@jest/types';
+import dotenv from 'dotenv';
+
+dotenv.config();
+
+const config: Config.InitialOptions = {
+  preset: 'ts-jest',
+  testEnvironment: 'node',
+};
+export default config;

server/jest.config.ts

@@ -16,7 +16,7 @@
     "docker:docs": "docker exec -t alertadotesouro-server-dev-1 yarn docs",
     "lint": "eslint . --ext .js,.jsx,.ts,.tsx",
     "docker:lint": "docker exec -t alertadotesouro-server-dev-1 yarn lint",
-    "test": "jest",
+    "test": "NODE_ENV=test jest",
     "docker:test": "docker exec -t alertadotesouro-server-dev-1 yarn test"
   },
   "dependencies": {

server/package.json

@@ -1,7 +1,6 @@
-import CreateServer from './app';
+import app from './app';
 
 it('Installs all the middleware', () => {
-  const app = CreateServer();
   expect(app.listen).toBeDefined();
-  expect(app.use).toHaveLength(4);
+  // I don't even know what else to test
 });

server/src/app.test.ts

@@ -4,12 +4,10 @@ import routes from './routes';
 import logRequest from './middlewares/requestsLogger';
 import 'dotenv/config';
 
-export default function CreateServer() {
-  const app = express();
+const app = express();
 
-  app.use(cors());
-  app.use(logRequest);
-  app.use(express.json());
-  app.use(routes);
-  return app;
-}
+app.use(cors());
+app.use(logRequest);
+app.use(express.json());
+app.use(routes);
+export default app;

server/src/app.ts

@@ -0,0 +1,21 @@
+import { getConnection } from 'typeorm';
+
+import DB from '.';
+
+beforeAll(async () => {
+  await DB.connect('test');
+});
+
+afterAll(async () => {
+  await DB.close('test');
+});
+
+beforeEach(async () => {
+  DB.clear('test');
+});
+
+test('connects to the database', () => {
+  const connection = getConnection('test');
+  expect(connection).toBeDefined();
+  expect(connection.isConnected).toBeTruthy();
+});

server/src/database/index.test.ts

@@ -1,3 +1,34 @@
-import { createConnection } from 'typeorm';
+import { createConnection, getConnection } from 'typeorm';
+import 'dotenv/config';
 
-createConnection();
+const DB = {
+  async connect(name = 'default') {
+    await createConnection(name);
+  },
+
+  async close(connectionName?: string) {
+    await getConnection(connectionName).close();
+  },
+
+  clear(connectionName?: string) {
+    const connection = getConnection(connectionName);
+    const entities = connection.entityMetadatas;
+
+    entities.forEach(async entity => {
+      const repository = connection.getRepository(entity.name);
+      await repository.query(`DELETE FROM ${entity.tableName}`);
+    });
+  },
+
+  async drop(connectionName?: string) {
+    await getConnection(connectionName).dropDatabase();
+  },
+};
+
+DB.connect().catch(err => {
+  console.error('❌ Error connection to DB!');
+  console.error(err);
+  throw err;
+});
+
+export default DB;

server/src/database/index.ts

@@ -2,6 +2,21 @@ import { Request, Response } from 'express';
 import ensureAuthenticated from './ensureAuthenticated';
 
 describe('ensures', () => {
+  // FIXME not actually throwing WTF but when env is actually no set on server it throws. I don't think it have anything to do with asynchronous tests and such, since they are on different threads.
+  it.skip('fails when `JWT_SECRET` env is not set', () => {
+    const request = {
+      // Is there a way to modularize this better? Doesn't feel DRY enough.
+      headers: {
+        authorization: 'Bearer 12345',
+      },
+    } as Request; // Asserting to type is fine to mock a request, that's the whole point;
+    const next = jest.fn();
+    delete process.env.JWT_SECRET;
+    expect(() =>
+      ensureAuthenticated(request, null as unknown as Response, next),
+    ).toThrow();
+  });
+
   it('rejects when JWT is missing', () => {
     const request = {
       headers: {
@@ -15,6 +30,7 @@ describe('ensures', () => {
       next,
     );
     expect(next.mock.calls).toHaveLength(1);
+    // REVIEW - I don't know how to hard type `jest.fn.mock.calls` since they are `any[]`
     expect(next.mock.calls[0][0] instanceof Error).toBeTruthy();
     expect(next.mock.calls[0][0].message).toBe(
       'Invalid JSON Web Token: JSON Web Token is missing',

server/src/middlewares/ensureAuthenticated.test.ts

@@ -39,7 +39,7 @@ export default function ensureAuthenticated(
     }
     const [, token] = authHeader.split(' '); // Do not use "type" of token (Bearer)
 
-    const decoded = verify(token, authConfig.jwt.secret as string);
+    const decoded = verify(token, authConfig.jwt.secret);
 
     const { sub } = decoded as TokenPayload;
 

server/src/middlewares/ensureAuthenticated.ts

@@ -14,10 +14,14 @@ export default function logRequest(
   _: Response,
   next: NextFunction,
 ): void {
-  const { method, url } = request;
-  const logLabel = `[${method.toUpperCase()}] request to ${url}`;
+  if (process.env.NODE_ENV !== 'test') {
+    const { method, url } = request;
+    const logLabel = `[${method.toUpperCase()}] request to ${url}`;
 
-  console.time(logLabel);
-  next();
-  console.timeEnd(logLabel);
+    console.time(logLabel);
+    next();
+    console.timeEnd(logLabel);
+  } else {
+    next();
+  }
 }