Produits Entr'ouvert » Hobo

J'ai pas la situation : si order existe et est bien à None, ça va continuer de planter. Sans fatigue tu peux faire un « x.get('order') or 0 »

Faut que ça reste trié sinon les champs ne vont plus apparaitre dans le bon ordre dans authentic.

Mais on n'est pas du tout "en base" ici : « fields = hobo_json.get('profile', {}).get('fields') »

Désolé, je ne suis décidément pas clair. Voici ce que je teste :
le tri à posteriori est inutile/impossible comme on peut aussi le voir ici.

$ tox -e py2-coverage-hobo -- tests/test_settings_loaders.py
(Pdb) l
330      
331              fields = hobo_json.get('profile', {}).get('fields')
332              if fields:
333                  import pdb; pdb.set_trace()
334                  fields = [x for x in fields if not x['disabled']]
335  ->                ...

## bien, que l'attribut 'order' ne soit pas disponible pour faire le tri,
(Pdb) [x.get('order', 'undef') for x in fields]                          
['undef', 'undef', 'undef', 'undef', 'undef', 'undef', 'undef', 'undef', 'undef']

## c'est quand même déjà trié
(Pdb) [x.get('name') for x in fields]
[u'title', u'first_name', u'last_name', u'email', u'address', u'zipcode', u'city', u'phone', u'mobile']

Faut pas venir avec des undef dans les tests qui ne font qu'amener de nouvelles questions, surtout précédés d'un "integer field are not exported to hobo.json".

Le truc qui est bel et bien à tester c'est que les champs fournis dans le hobo.json sont bien dans l'ordre défini dans Hobo, dans la db.

Et oui ça terminera en "dans l'hobo.json les champs du profil sont dans une liste déjà correctement triée", ce qui était déjà le propos de #7269 que tu as trouvé, qui a "juste" oublié de retirer la référence à order à cet endroit. Vu la description de #7269 et du commit associé, j'imagine qu'il y a déjà eu à l'époque grande confusion entre l'ordre dans la liste et les attributs order. (peut-être en faisant #7253)

Bref, #7269 n'a pas nettoyé totalement, ce patch nettoie totalement, c'est mieux, mais il ne faudrait pas que l'ajout d'un test bizarre assure à son tour des difficultés de compréhension.

→, je dirais :

• commencer par vérifier l'ordre

env = get_hobo_json()
fields = env['profile']['fields']
assert [x['name'] for x in fields] == ['first_name', 'last_name', etc.]

• retirer la partie qui amène que confusion :

- # order is managed on database
- assert [x.get('name') for x in fields][0:11:10] == [u'title', u'mobile']

• expliciter qu'on intervertit deux champs

+    # swap title and mobile fields
+    title = AttributeDefinition.objects.get(name='title')
+    mobile = AttributeDefinition.objects.get(name='mobile')
+    (title.order, mobile.order) = (mobile.order, title.order)
+    title.save()
+    mobile.save()

• refaire le hobo.json, revérifier l'ordre.

• dégager une autre partie de confusion,

-    # integer field are not exported to hobo.json
-    assert [x.get('order', 'undef') for x in env['profile']['fields']] == ['undef',] * 11

• continuer et vérifier aussi les contenus de A2_*_FIELDS, comme tu as là.