map: add zoom level configuration validation (#64640) #216
vdeniaud
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/64640-map-cell-edit-zoom-validation"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
map: add zoom level configuration validation (#64640)to wip: map: add zoom level configuration validation (#64640)351444205dtoe9b03533f7e9b03533f7tob6446e6b22wip: map: add zoom level configuration validation (#64640)to map: add zoom level configuration validation (#64640)Nickel pour le code, par contre le test est assez peu homogène avec ce qu'on fait d'habitude, on peut voir en jabber si tu as des questions
@ -125,0 +137,4 @@min_zoom = cleaned_data.get('min_zoom')if min_zoom > max_zoom:raise ValidationError(_('Invalid zoom configuration : minimal zoom must be lower than maximal zoom'), code='invalid'Il n'y a pas d'espace avant le : en anglais (à retirer également plus bas)
@ -324,0 +336,4 @@manager_submit_cell(resp.forms[0], expect_errors=True)form_map = resp.forms[0]toformdict = lambda cell, names, testval: {Notre pratique quand il s'agit d'écrire des tests c'est de toujours privilégier la simplicité du code, en n'hésitant jamais à faire long et répétitif. Ici je te conseillerais de reprendre l'écriture de ce test, avec la contrainte de n'utiliser ni définition de fonction ni boucle for :)
@ -324,0 +357,4 @@resp = manager_submit_cell(form_map, True)assert resp.status_int == 200ret = resp.jsonassert len(ret['errorlist']) > 0Il faudrait vérifier explicitement quelle erreur est affichée, pour ça il n'y a effectivement pas beaucoup d'exemples mais je dirais de faire comme https://git.entrouvert.org/entrouvert/combo/src/branch/main/tests/test_manager.py#L1928 (mais cette façon là est également correcte https://git.entrouvert.org/entrouvert/combo/src/branch/main/tests/test_dataviz.py#L2088)
@ -324,0 +333,4 @@resp = app.get(f'/manage/pages/{page.pk}/')# submiting once to populate the formmanager_submit_cell(resp.forms[0], expect_errors=True)Pour ce point là c'est plutôt un bug, j'ai ouvert https://dev.entrouvert.org/issues/85720. En attendant un contournement plus simple c'est d'hériter de la fixture layer, comme ça la cellule ne sera pas désactivée et l'onglet Général apparaîtra
b6446e6b22to6e82a76fcd6e82a76fcdtobd205adfb0bd205adfb0toba39e977d0ba39e977d0toe303c352a1e303c352a1to44696dff8144696dff81toa717d3c4d6Merci pour la prise en compte des remarques, ça m'a l'air parfait !
a717d3c4d6to12a3d5b392