No tests are run on any GitHub actions

[fuhrmanator]

The actions are not set up to run tests on a PR (this is not good -- need to run the tests on PRs).

[fuhrmanator]

Tests need to run also before deploying.

npm test in the client directory has lots of failing tests.

npm test in the server directory only runs two tests, lots of failures.

Need more tests (that pass!) before merging PRs and deploying.

[fuhrmanator]

I added front-end tests for PR: https://github.com/ets-cfuhrman-pfe/EvalueTonSavoir/blob/main/.github/workflows/frontend-tests.yml

It's not integrated into the Actions that deploy. It would be good to "call" that action before building/deploying the front-end to docker hub. It should work the same way for the back-end.

[fuhrmanator]

Le back-end est un casse-tête pour faire les tests:

• les "models" (classes du domaine) sont tous singletons (à éviter), module.exports = new X() -- un singleton est comme une façade - un modèle (pour moi) représente des instances de chaque objet du domaine. Impossible de raisonner sur l'état de quelque chose qui a été créé (un nouveau User par exemple, car il y a juste un objet Users (au pluriel!) et on doit faire des requêtes dans MongoDB pour trouver un User individuel).
• les "models" ont toute la logique de BD dedans (on ne peut rouler les tests sans la BD, sauf si on fait des mocks).

Pour résoudre ces problèmes, je propose pour le back-end de refactoriser les models:

• ne plus faire de singleton
• injecter la bd par le constructeur (permet de mocker plus facilement)
• éventuellement appliquer le patron Service Layer

ChatGPT/Copilot est ton ami.