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 une option pour afficher des infos de debug dans le rapport Litteralis #1139

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

florimondmanca
Copy link
Collaborator

@florimondmanca florimondmanca commented Jan 8, 2025

Motivation

Lorsqu'une erreur, un warning ou une notice est loggée dans l'intégration Litteralis, on ajoute des informations de contexte comme l'ID de l'emprise, la valeur qui a posé problème, etc

Mais jusqu'ici elles étaient seulement mises dans le log, qui est assez peu pratique à utiliser

Description des changements

Cette PR ajoute une option --debug qui affiche ces infos supplémentaires en JSON directement dans le rapport d'intégration

Exemple

Remarques
---------

Mesures de type non-supporté par DiaLog : 14 (dans 10 arrêtés)
  Arrêtés :
    24-AT-0665 (debug: [{"name":"Interdiction de stationnement","idemprise":685939}])
    24-AT-0658 (debug: [{"name":"Interdiction de stationnement","idemprise":686236}])
    24-AT-0660 (debug: [{"name":"Interdiction de stationnement","idemprise":686240},{"name":"D\u00e9viation","idemprise":686240}])
    24-AT-0661 (debug: [{"name":"Interdiction de stationnement","idemprise":686241}])

Sans le debug on aurait seulement:

Remarques
---------

Mesures de type non-supporté par DiaLog : 14 (dans 10 arrêtés)
  Arrêtés :
    24-AT-0665
    24-AT-0658
    24-AT-0660
    24-AT-0661

Et donc on ne sait pas quelles mesures ont posé problème ni dans quelles emprises on devrait aller fouiller pour en savoir +

@florimondmanca florimondmanca force-pushed the feat/litteralis-debug branch 2 times, most recently from 9478ce7 to 12c0a1a Compare January 8, 2025 14:01
@florimondmanca florimondmanca marked this pull request as ready for review January 8, 2025 14:02
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.86%. Comparing base (5016a1e) to head (df8cf61).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1139   +/-   ##
=========================================
  Coverage     98.86%   98.86%           
- Complexity     1831     1843   +12     
=========================================
  Files           365      366    +1     
  Lines          7916     7956   +40     
=========================================
+ Hits           7826     7866   +40     
  Misses           90       90           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

2 participants