[TECH] Suivi des tickets de refacto et d'écriture de tests #1014
Labels
Comments
This was referenced Jan 24, 2024
ivangabriele
added a commit
that referenced
this issue
Jan 30, 2024
- Conseil : checker les commits un par un pour ne pas mélanger renommages et diffs, en particulier le 3ème qui est le seul à contenir les changements de code. - J'ai préféré les TODO et any à des refactos dangereuses. Il n'y a normalement que peu de code "concret" modifié. - J'ai supprimé quelques utils et une story au passage. - Je ne comprends pas trop pourquoi on a un gros fichier JS pour les drapeaux, il n'y a pas de lib pour ça ? On ne pourrait pas tout simplement utiliser les emojis unicodes natifs (= gros gain de poids) ? ## Related Pull Requests & Issues - #964 - #1014 - Resolve #1155 ---- - [ ] Tests E2E (Cypress)
ivangabriele
added a commit
that referenced
this issue
Feb 9, 2024
## Taches effectuées - Migration des imports SVG en tant que composants pour les adapter à Vite. - Séparation des envs Frontend des autres via un fichier `/frontend/.env.example` (⚠️ **qu'il faut maintenant copier localement dans un `/frontend/.env`**). - Injection des variables d'environnement Frontend au runtime (et non au buildtime) via [`import-meta-env`](https://import-meta-env.org/), comme sur MonitorFish. [Voir l'ADR correspondant](https://github.com/MTES-MCT/monitorfish/blob/master/adrs/0001-frontend-runtime-env-var-injection.md). - Création d'un utilitaire `isPuppeteer()` pour différencier les conditions que l'on applique pour les tests Puppeteer de celles qu'on applique aux tests Cypress. Même principe qu'avec `isCypress()`. - Traduction à la volée des instructions de contribution legacy et déplacement vers le fichier `CONTRIBUTING.md`. - Début d'écriture des instructions de contribution mises à jour dans ce même fichier, en anglais, pour faciliter le partage éventuel avec d'autres équipes de l'UE. - Remplacement de la commande `npm run cypress:open` par `npm run test:e2e:open`. - Petite refacto qui déplace les constantes de `frontend/src/features/missions/MissionForm/sse.ts` dans un fichier séparé pour éviter un import cycle. ~~Je me suis aussi permis de désactiver un des tests des points d'întérêt parce qu'on est à près de 10% de failure, il y a au moins 4 bugs notables connus sur cette feature et Adeline a prévu de la revoir entièrement.~~  ## Notes - GitGuardian râle pour une clé de dév qui était déjà présente mais qui a été déplacée dans le nouveau ficher `fontend/.env.example`. - Il y aura un soucis avec monitor-ui concernant `cy.fill()` lorsqu'on passera à une der versions Rsuite `>5.45.0` qui casse la structure DOM des fields en assignant l'attribut `id` au wrapper (`div`) au lieu de le passer à l'input. Cet attribut est ciblé par attributs `for` des labels. Je ferai le fix. - Les tests e2e des missions sont localement instables à cause de `clickOutside()` manquants ou qui ne ferment pas correctement certains dropdowns. À investiguer ? ## Review - Ca serait idéal si vous pouviez tester que tout fonctionne bien localement chez vous. Env est assez léger pour tourner sur Chrome en mode dev chez moi contrairement à Fish. - Je vous conseille de cloner cette branche sous un autre dossier car le switch entre CRA et Vite n'est pas fun d'une branche à l'autre. ##⚠️ Derniers points à voir ensemble - Faire le tour des impications staging / prod concernant les variables d'environnement @louptheron @thoomasbro. - Il n'y a pas de `FRONTEND_MISSION_FORM_AUTO_SAVE_ENABLED` dans le docker compose de prod, c'est normal @louptheron ? Rien de problématique mais juste pour être sûr. ## Related Pull Requests & Issues - Resolve #964 - #1014 ---- - [ ] Tests E2E (Cypress)
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Ecriture de tests E2E :
Autres:
/monitorUipour utiliser les composants de Monitor-ui + suppression de storybook #1037envActionsne devrait pas figurer sur le type publiqueCreateOrUpdateMissionDataInput#879The text was updated successfully, but these errors were encountered: