Skip to content

[TECH] ♻️ Utilise un vrai builder pour le candidat dans le contexte enrolment (PIX-23680) - #16944

Draft
yaf wants to merge 1 commit into
devfrom
PIX-23680-upgrade-candidate-builder
Draft

[TECH] ♻️ Utilise un vrai builder pour le candidat dans le contexte enrolment (PIX-23680)#16944
yaf wants to merge 1 commit into
devfrom
PIX-23680-upgrade-candidate-builder

Conversation

@yaf

@yaf yaf commented Jul 24, 2026

Copy link
Copy Markdown
Member

🪧 Problème

On constate que c’est pénible dans :

  • tous les tests
    • se souvenir des valeurs à mettre à des attributs pour faire en sorte que les modèles utilisés reflètent un état métier spécifique
  • intégration/acceptance
    • créer à la fois le modèle domaine mais aussi les données sous-jacentes en BDD
    • en particulier pour les plus nouveaux, se souvenir quand utiliser le domainBuilder VS databaseBuilder

Ces builders n’en portent que le nom, et sont frustrant à l’usage.

https://1024pix.atlassian.net/wiki/spaces/DC/pages/6276775948/BACK+Builders

🌈 Proposition

Pour adresser toutes les frustrations énoncées ci-dessus, on a proposé une revisite du domainBuilder pour que ça ressemble à un Builder, à savoir une entité sur laquelle on peut chaîner des méthodes expressives afin de construire un modèle dans un état spécifique. Ce Builder propose également une méthode permettant d’insérer les données sous-jacentes en BDD.

Ici, c'est le builder de candidat du contexte enrolment qui est migré.

✊ Remarques

🎉 Pour tester

@yaf yaf self-assigned this Jul 24, 2026
@pix-bot-github

Copy link
Copy Markdown

@yaf
yaf force-pushed the PIX-23680-upgrade-candidate-builder branch from 95a79da to 24321db Compare July 24, 2026 18:49
@yaf
yaf marked this pull request as ready for review July 27, 2026 06:58
@yaf
yaf requested a review from a team as a code owner July 27, 2026 06:58
@yaf
yaf force-pushed the PIX-23680-upgrade-candidate-builder branch 2 times, most recently from d3fe5f7 to f801c36 Compare July 27, 2026 07:07
@yaf
yaf marked this pull request as draft July 27, 2026 07:54
@yaf
yaf force-pushed the PIX-23680-upgrade-candidate-builder branch 2 times, most recently from 54ed5b1 to 75e1c53 Compare July 27, 2026 15:26
@yaf
yaf force-pushed the PIX-23680-upgrade-candidate-builder branch from 75e1c53 to 73f1a95 Compare July 27, 2026 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants