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

test: added testing-library, and unit test exemple with usermenu #942

Merged
merged 3 commits into from
Mar 21, 2024

Conversation

MarcWadai
Copy link
Contributor

@MarcWadai MarcWadai commented Mar 21, 2024

Description

Here you will have the configuration for testing-library and an exemple of Tchap unit test

What changed

  • Path alias have been added
  • UserMenu unit test has been added
  • testing-library config

Kind of guideline to write a unit test (copy from notion)

  • Comparer avec le test original pour voir s’il y a des choses intéressante qui peuvent être récupérées. La plupart du temps non 🤪
  • Nos patches sont souvent des changements UI donc pour être sur qu’elles n’ont pas bouger, nous pouvons simplement utiliser un test de snapshot.
    • ce test snapshot peut capturer des changements visuels utilisé par les composants, mais pas en ce qui concerne la logique et les props
  • Faire un test sur les props d’entrées de votre composant
    • identifier les composants du sdk que vous utilisez
  • Faire un test sur le résultat de sortie de votre composant (ca peut aussi tout simplement correspondre au snapshot)
  • Ensuite pour vous pouvez ajouter de la logique spécifique à Tchap

@MarcWadai MarcWadai self-assigned this Mar 21, 2024
Copy link
Contributor

@estellecomment estellecomment left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trop cool :)

tsconfig.json Outdated Show resolved Hide resolved
@estellecomment
Copy link
Contributor

J'ai des doutes sur l'utilité des snapshots : des que element change qq chose au composant notre test casse.
Par exemple dans usermenu ils changent le nom des classes CSS, paf ca casse (alors que ca change rien du tout pour notre patch)
Par le passé quand ya des grosses upgrades tous les snapshots cassent mais ca nous apprend pas grand chose, donc on a arrêté les snapshots.

@MarcWadai MarcWadai merged commit e3771e2 into develop_tchap Mar 21, 2024
7 of 8 checks passed
@MarcWadai MarcWadai deleted the 937-study-unit-test-on-tchap-components branch March 21, 2024 15:24
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.

2 participants