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

[QA] Comparaison des valeurs JSON indépendamment de l'ordre des clés #3466

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Dec 17, 2024

Ticket

#3455

Description

Les valeurs JSON avec un contenu identique mais un ordre de clés différent sont actuellement considérées comme différentes dans la logique de comparaison.

Ancienne valeurs

{
    "arrete_type": "Arrêté L.511-19 - Insalubrité",
    "arrete_numero": "2024/DD13/00664",
    "arrete_mainlevee_date": "01/08/2024",
    "arrete_mainlevee_numero": "2024-DD13-00173"
}

Nouvelles valeurs

{
    "arrete_numero": "2024/DD13/00664",
    "arrete_type": "Arrêté L.511-19 - Insalubrité",
    "arrete_mainlevee_date": "01/08/2024",
    "arrete_mainlevee_numero": "2024-DD13-00173"
}

Les valeurs sont identiques mais l'ordre est différent.
N'ayant pas la main sur ce que UnitOfWork de doctrine renvoie, il serait plus sur de trier les clés de tableau avant de comparer.

Changements apportés

  • Ajout d'un tri de tableau

Pré-requis

make worker-stop && make worker-start
make mock-stop && make mock-start

Tests

  • Lancer plusieurs fois la commande make sync-sish et vérifier l'absence d'historique sur la table intervention

TNR

  • Modifier la superficie et vérifier que l'objet typeCompositionLogement a bien été historisé
  • Modifier allocataire et vérifier que l'objet SituationFoyer a bien été historisé
  • Modifier assurance contacté et vérifier que l'objet informationProcedure a bien été historisé
  • Modifier nombre étage et vérifier que l'objet informationComplementaire a bien été historisé

@sfinx13 sfinx13 changed the title fix compare array #3455 [QA] Comparaison des valeurs JSON indépendamment de l'ordre des clés Dec 17, 2024
Copy link

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

ok

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

ok tests et lecture

@hmeneuvrier hmeneuvrier merged commit 9e41a51 into develop Dec 19, 2024
4 checks passed
@sfinx13 sfinx13 deleted the feature/3455-historique-json branch December 19, 2024 19:13
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