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

fix: ANT-2621 - Refacto hooks and add unit tests #34

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

marlenetienne
Copy link
Collaborator

@marlenetienne marlenetienne commented Jan 13, 2025

https://gopro-tickets.rte-france.com/browse/ANT-2621

Hooks =>

  • ajout de test unitaire pour l'ensensemble des hook sauf le suivant : useDateFormatter (non utilisé dans le projet)
  • utilisation des hooks du design system si possible (ex: useRdsId)
  • sécurisation des entrées et sorties des hooks existants personnalisés avec useRef et useCallback (ex: useStudyTableDisplay)
  • création de hook personnalisé =>

Composants design system =>

  • remplacement des composants par ceux du design system (ex: RdsTextTooltip au lieux de StdTextTooltip)

Pinned projects =>

  • Création d'un context / reducer pour la gestion des projets épinglés => utilisation au niveau des pages Home et projects + suppression des props destinées à forcer la récupération des projets épinglés (reloadPinnedProject, isReloadPinnedProject)
  • utilisation du dispatch pour mettre la liste des projets épinglés à jour

Endpoint =>

  • création d'un fichier rassemblant tous les endpoints utilisés dans les API (à compléter dans le cadre du ticket suivant ANT-2622)

https://gopro-tickets.rte-france.com/browse/ANT-2622

Services =>

  • création de fichiers regroupant les services par sujet (study, pinned projects, projects)
  • ajout de test unitaire

@marlenetienne marlenetienne force-pushed the fix/ANT-2621_refacto_hooks branch from 8e16dba to c4a8446 Compare January 22, 2025 14:44
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
61.5% Coverage on New Code (required ≥ 80%)
4.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant