Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Mettre à jour des tests suite à la suppression de Bookshelf (PIX-10677) #10551

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Nov 14, 2024

🍂 Problème

Les PR #7752 et #8514 ont remplacé Bookshelf par Knex ❤️ Mais le updatedAt qui était mis à jour automatiquement par Bookshelf doit, avec Knex, être mis à jour manuellement et cela est parfois oublié dans les implémentations ou dans les mises à jour de code. Aussi il est maintenant nécessaire de systématiquement ajouté dans les tests que le updatedAt est toujours mis à jour.

🌰 Proposition

Ajouter les assertions manquantes sur le updatedAt partout où nécessaire.

On en a profité pour factoriser l'utilisation de sinon.useFakeTimers dans les fichiers de tests concernés.

🎃 Remarques

RAS

🪵 Pour tester

Vérification de la CI

Vérifier que la CI passe et que les modifications faites ne créent pas de warnings dans les logs d'exécution des tests qui pourraient par exemple être dus à la mise en place de fake timers. Il y a actuellement sur dev certains warnings dans les logs de tests, mais qui ne sont pas dus à cette PR, pour vérifier comparer l'exécution des tests sur dev et dans cette PR.

Vérification de non-régression dans Pix Orga

Cette PR modifie UserOrgaSettings et il convient donc de vérifier qu'il n'y a pas de régression là ou cet objet est utilisé.

  1. Se connecter à Pix Orga avec un compte ayant accès à plusieurs organisations (par exemple [email protected])
  2. Une fois connecté à Pix Orga, passer d'une organisation à l'autre (par exemple de l’organisation Collège House of The Dragon à l'organisation Lycée Joséphine Baker)
  3. Vérifier que la route /api/user-orga-settings/{id} est bien appelée et qu'il n'y a pas de régression

@lego-technix lego-technix requested a review from a team as a code owner November 14, 2024 18:37
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@lego-technix lego-technix changed the title Pix 10677 add missing assertions in tests after migrating from bookshelf [TECH] Mettre à jour des tests suite à la suppression de Bookshelf (PIX-10677) Nov 14, 2024
@lego-technix lego-technix self-assigned this Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants