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

Ajoute les types de scolarités "maternelle" et "primaire" #4487

Merged

Conversation

Shamzic
Copy link
Contributor

@Shamzic Shamzic commented Jul 22, 2024

cf #4424

@Shamzic Shamzic linked an issue Jul 22, 2024 that may be closed by this pull request
4 tasks
@Shamzic Shamzic changed the title Ajoute les scolarité "maternelle" et "primaire" Ajoute les scolarités "maternelle" et "primaire" Jul 22, 2024
@github-actions github-actions bot added this to the BC actuel milestone Jul 22, 2024
@Shamzic Shamzic changed the title Ajoute les scolarités "maternelle" et "primaire" Ajoute les types de scolarités "maternelle" et "primaire" Jul 22, 2024
@Shamzic Shamzic marked this pull request as ready for review July 22, 2024 12:56
Copy link
Contributor

@jenovateurs jenovateurs left a comment

Choose a reason for hiding this comment

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

Ajouter un test unitaire dans unit/openfisca/mapping/dispatch-individuals.spec.ts
afin de vérifier que la valeur primaire et/ou maternelle est bien mappé en utilisant la constante mis en place.
Le test n'a pas été fait pour les autres cas mais je pense que ça peut être intéressant à faire pour cet ajout spécifique. Qu'en penses-tu ?

Et pourquoi pas ajouter un test cypress si on a une aide compatible avec cette situation dans famille.

@Shamzic
Copy link
Contributor Author

Shamzic commented Jul 25, 2024

Ajouter un test unitaire dans unit/openfisca/mapping/dispatch-individuals.spec.ts afin de vérifier que la valeur primaire et/ou maternelle est bien mappé en utilisant la constante mis en place. Le test n'a pas été fait pour les autres cas mais je pense que ça peut être intéressant à faire pour cet ajout spécifique. Qu'en penses-tu ?

J'ai ajouté un test unitaire, tu me diras si c'est à cela que tu pensais

Et pourquoi pas ajouter un test cypress si on a une aide compatible avec cette situation dans famille.

C'est possible d'en ajouter un dans la PR pour l'aide à la rentrée scolaire

@Shamzic Shamzic requested a review from jenovateurs July 25, 2024 11:59
expect(result.foyers_fiscaux.conjoint.declarants).toEqual([
situation.conjoint.id,
])
})
it("checks scolarite enfant_a_charge", function () {
expect(result.individus.demandeur.enfant_a_charge.scolarite).toEqual(
Copy link
Contributor

Choose a reason for hiding this comment

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

Quelle est la différence entre enfants: [{ id: "e1" }],
et situation.demandeur.enfant_a_charge.scolarite
J'ai l'impression que la première option est la plus probable non ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est corrigé

Copy link
Contributor

@jenovateurs jenovateurs left a comment

Choose a reason for hiding this comment

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

Screenshot 2024-07-29 at 15 37 14 Est-ce que cette question a aussi été géré ? @Shamzic

@Shamzic
Copy link
Contributor Author

Shamzic commented Jul 30, 2024

Est-ce que cette question a aussi été géré ? @Shamzic

C'est l'objectif de cette tâche oui

@Shamzic Shamzic requested a review from jenovateurs July 30, 2024 14:49
Copy link
Contributor

@jenovateurs jenovateurs left a comment

Choose a reason for hiding this comment

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

Approuvé avec la validation de ma suggestion @Shamzic

@Shamzic Shamzic merged commit da71489 into main Jul 31, 2024
24 checks passed
@Shamzic Shamzic deleted the 4424-ajouter-lcole-primaire-sur-la-question-de-la-scolarit branch July 31, 2024 12:26
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.

Ajouter l'école primaire sur la question de la scolarité
3 participants