Produits Entr'ouvert » w.c.s.

A priori pas lié à cette nouvelle section (un for x in formdata.iter_target_datas(): print(x) de test est tout à fait rapide).

[0.07] getting flat keys
[0.77] getting flat keys -- done
[0.77] iterating on flat keys
[1.43] iterating on flat keys -- done

Je vois bien que cette partie du code fonctionne :

            # cache during request
            if not hasattr(request, 'live_card_cache'):
                request.live_card_cache = {}
            cache_key = '%s-%s' % (self._field.data_source['type'], self.raw)
            carddata = request.live_card_cache.get(cache_key)
            if carddata is not None:
                if isinstance(carddata, Exception):
                    raise carddata
                return LazyFormData(carddata)

mais peut-être que je suis trop vieux, j'aurais ajouté un else au cas où le cache n'est pas encore là, comme :

            # cache during request
            cache_key = '%s-%s' % (self._field.data_source['type'], self.raw)
            if not hasattr(request, 'live_card_cache'):
                request.live_card_cache = {}
            else:
                carddata = request.live_card_cache.get(cache_key)
                if carddata is not None:
                    if isinstance(carddata, Exception):
                        raise carddata
                    return LazyFormData(carddata)

Ensuite concernant le test il y a un truc qui m'échappe : il fonctionne aussi très bien sans cache... Que teste-t-on exactement en demandant deux fois context['form_var_foo_live_name'] ?

Ensuite concernant le test il y a un truc qui m'échappe : il fonctionne aussi très bien sans cache...

Oui, c'est qu'idéalement il y aurait mock/whatever et pouvoir vérifier qu'il n'y a pas d'appel, mais au final trop galère.

Modification poussée dans la branche + aussi mock dans le test (pour juste la partie KeyError en fait ça va) pour voir que get_by_urlname n'est pas appelé la seconde fois.