Skip to content

fix: raw response to response process with 'interrogation id' duplicates#471

Draft
nsenave wants to merge 11 commits intomainfrom
fix/process-duplicates-part-2
Draft

fix: raw response to response process with 'interrogation id' duplicates#471
nsenave wants to merge 11 commits intomainfrom
fix/process-duplicates-part-2

Conversation

@nsenave
Copy link
Copy Markdown
Contributor

@nsenave nsenave commented May 6, 2026

Le problème : quand on a des doublon d'identifiant d'interrogation dans les réponses brutes, on peut avoir ce genre de situation

  • interrogationId: 1, processDate: 08/04/2026 (a été processed)
  • interrogationId: 1, processDate: null (est arrivée après, pas encore processed)

(processed = intégrée dans la table des response depuis la table des rawResponse)

Dans ce cas là, le service qui process va convertir les deux réponses brutes avec l'id 1 en réponses "survey unit", alors qu'une des deux a déjà été processed

Uniquement le cas "raw response". Je ferai le fix pour le données legacy "Lunatic raw data" dans une autre PR


NB : historique git / github

Suite de #445

  • J'ai repris la branche fix/process-duplicates, mise à jour à partir de main, puis créé fix/process-duplicates-part-1
  • J'ai rassemblé tous les changements de fix/process-duplicates-part-1 dans un squashed commit ici pour y voir plus clair

Squashed commits from branch 'fix/process-duplicates-part-1'
@nsenave nsenave self-assigned this May 6, 2026
@nsenave nsenave changed the title feat: process duplicate ids fix: raw response to response process with 'interrogation id' duplicates May 6, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

issue: On va passer à des mocks mockito à l'avenir, vaut mieux éviter de faire des stubs maison pour des tests unitaires

@nsenave
Copy link
Copy Markdown
Contributor Author

nsenave commented May 7, 2026

@Hajarel-moukh @alexisszmundy Si vous voulez bien je vous laisse régler les derniers conflits et remplacer les stubs par du mock, je vais passer à d'autres sujet

(si vous voulez que je finisse cette PR faites moi signe)

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