Skip to content

Conversation

@faridkhouda
Copy link
Collaborator

@faridkhouda faridkhouda force-pushed the unit-tests/date-holidays branch 2 times, most recently from c41c658 to 621f2fa Compare December 26, 2024 09:30
@faridkhouda faridkhouda self-assigned this Dec 26, 2024
@tonytang-planity
Copy link

Je pense pas que ça fasse ce que j'attends parce que tu utilises des mocks.
Je voudrai que tu teste la fonctione qui permet de récupérer les jours fériés pour les pays qu'on utilise et vérifier que ça retourne les véritables dates.
Par exemple, pour FR, il faut que ça retourne le 25 décembre, le 1er janvicer, etc.

Pareil pour les autres country code.

@faridkhouda
Copy link
Collaborator Author

faridkhouda commented Dec 27, 2024

Je pense pas que ça fasse ce que j'attends parce que tu utilises des mocks. Je voudrai que tu teste la fonctione qui permet de récupérer les jours fériés pour les pays qu'on utilise et vérifier que ça retourne les véritables dates. Par exemple, pour FR, il faut que ça retourne le 25 décembre, le 1er janvicer, etc.

Pareil pour les autres country code.

@tonytang-planity, Si j'ai bien compris tu veux que je focus sur la function getHolidays(year?: string | number | Date, lang?: string): HolidaysTypes.Holiday[] ? Par exemple, vérifier qu’elle retourne les correct public holidays comme le 1er janvier pour FR ?
Concernant le date format, vu que c’est handled côté app, Est-ce que je dois uniquement valider les valeurs brutes retournées par getHolidays, ou bien vérifier que le format attendu est respecté ?

@tonytang-planity
Copy link

Yes c'est ça. Pas besoin de valider le format de la date

@faridkhouda faridkhouda force-pushed the unit-tests/date-holidays branch from b0f30f4 to 4244ef9 Compare January 6, 2025 17:17
@faridkhouda faridkhouda marked this pull request as draft January 6, 2025 17:17
@faridkhouda
Copy link
Collaborator Author

faridkhouda commented Jan 6, 2025

@MalouPro , @MahaKHARRAT Capture d’écran 2025-01-07 à 08 47 25

🛠️ Tests for the German (DE) and the Belgium (BE) public holidays are still in progress.

@faridkhouda faridkhouda force-pushed the unit-tests/date-holidays branch 3 times, most recently from d30f48f to 2582830 Compare January 7, 2025 09:33
@faridkhouda faridkhouda force-pushed the unit-tests/date-holidays branch from 2582830 to 1dcb22c Compare January 8, 2025 17:42
@faridkhouda faridkhouda marked this pull request as ready for review January 8, 2025 17:43
@faridkhouda faridkhouda requested a review from MalouPro January 8, 2025 17:43
@MalouPro MalouPro merged commit fffcc9c into master Jan 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants