Skip to content

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

Merged
loichenninger merged 14 commits into
mainfrom
fix/process-duplicates-part-2
May 13, 2026
Merged

fix: raw response to response process with 'interrogation id' duplicates#471
loichenninger merged 14 commits into
mainfrom
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
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)

@sonarqubecloud
Copy link
Copy Markdown

@loichenninger loichenninger marked this pull request as ready for review May 13, 2026 08:31
@loichenninger loichenninger merged commit fda98b5 into main May 13, 2026
9 of 10 checks passed
@loichenninger loichenninger deleted the fix/process-duplicates-part-2 branch May 13, 2026 08:32
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.

4 participants