-
Notifications
You must be signed in to change notification settings - Fork 4
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
fix: aggregate-scores when there is no na map #20
Conversation
Co-authored-by: Bertrand Keller <[email protected]>
Il me semble qu'on ne cherche pas à remplir une map : Mais à obtenir la liste des critères testés dans tous les audits réalisés. On veut un dictionnaire contenant la valeur Utiliser |
Voir si ce code corrige :
|
merci @bertrandkeller oui ça fonctionne ! |
Je me demande si on pourrait écrire un test automatisé pour vérifier certains de ces comportements un peu pointus. |
Tu penses à quoi ? Pour information. Le partial "Aggregate" fait tous les calculs. On pourrait l'appeler une seule fois dans le Header. Ainsi, tu pourrais avoir des éléments (sondes) à certains endroits pour vérifier des valeurs, indépendamment du type d'audit. |
Comme tu le dis, le partial aggregate a l'air de faire surtout de la logique de calculs sans contenir beaucoup de présentation. C'est en général de bons candidats pour tester unitairement. L'idée serait d'appliquer ce partial sur des jeux de données de test. Et vérifier que ce qui est généré est conforme à ce qu'on attend. Edit: Après reflexion, quelque chose de simple dans un premier temps serait probablement de faire un dépôt Git avec des projets d'exemples qui représentent un certain nombre de combinaisons possibles dans les csv. Par ex. le cas suivant ne fonctionne pas actuellement :
|
Oui ! C'est ce que je n'ai pas fait. J'ai plusieurs jeux de données. C'est possible de créer plusieurs dépôts, voire 1 seul dépôt. Et de lancer des CI à chaque push de master (ou créer une vraie branche de Avec Hugo, tu peux faire des partials qui rendent des valeurs. Tu peux donc rendre des objets. Que tu vas exploiter où tu en as envie sur le site. Ils rendent plus facile la création d'un JSON. Ensuite, tu peux créer des |
Un dépôt qui pourrait s'appeler Je pense qu'il est assez pratique de vérifier sur le contenu généré. J'aime bien l'idée de tester sur le contenu json. L'approche "approval testing" s'y prête bien. Un peu comme dans cette vidéo d'@emilybache : If Someone Was Doing Test-Driven Development, How Would You Tell? |
as mentioned in #19 there is a possible issue with non applicable map.
The test line 179
is not testing
nonapplicable
and the line after it is using it. So when it isnil
it breaks (see issue above).I made a workaround to make hugo generate reports but as we made an automated process with git submodules, it cannot be used. We update https://audits.test.iroco.co/ by hand for now.
I don't know the "clean" way to fix this, I saw that when the map is not empty we have the line 81:
So there seem to be a
thematique
key and I don't know how to use it.Couldn't we initialize all the necessary structure to be empty, and fill it when we have values?
Désolé pour l'anglais je me suis aperçu à la fin que c'était pas nécessaire.