Produits Entr'ouvert » Authentic 2

C'est moche que ça casse dans un seul environnement, ça sent mauvais.

Dans 0002,

+        if (not self.username
+            and not self.email
+            and not (self.first_name
+                     or self.last_name)):

+            username_qs = qs
+            if not app_settings.A2_USERNAME_IS_UNIQUE:
+                username_qs = qs.filter(ou=self.ou)

Sinon de manière générale, j'aimerais que les tests ne soient pas cassés entre deux commits (coucou git rebase --exec). Dans le cas présent c'est sympa pour la relecture d'avoir plein de commits, mais 0001 et 0005 pourraient disparaître et être mis dans les commits qui cassent les-dits tests, ça aide aussi à la compréhension de ce qui est recherché dans ces commits je trouve.

Valentin Deniaud a écrit :

Dans 0002,
[...]
Chacun sa logique mais moi ça m'a demandé trop de temps de cerveau, j'aurais écris ça juste à coup de and not ou juste à coup de not or, pas un mix des deux (ou encore avec any()).

C'est parce que c'est pas la bonne condition.

[...]
La première ligne ne sert à rien (et faire pareil pour email_qs plus bas tant qu'à faire, où l'affectation sert bien à quelque chose mais on peut s'en passer en faisant comme ça).

Si je l'enlève ça ne marche plus dans le cas app_settings.A2_USERNAME_IS_UNIQUE is True par contre j'ai fait pareil coté email, email_qs = qs.filter.... pour la symétrie.

Sinon de manière générale, j'aimerais que les tests ne soient pas cassés entre deux commits (coucou git rebase --exec). Dans le cas présent c'est sympa pour la relecture d'avoir plein de commits, mais 0001 et 0005 pourraient disparaître et être mis dans les commits qui cassent les-dits tests, ça aide aussi à la compréhension de ce qui est recherché dans ces commits je trouve.

C'est très bien que les tests cassent entre deux commits sinon on ne voit pas ce qui est réparé, c'est le principe d'un test, un test qui ne foire jamais ça ne teste rien. Sans la progression des commits je ne vois pas comment tu aurais pu comprendre 0005 (ça c'est assez con mais on y peut rien si les libs python3 ne sont pas adaptés à unicode, ça va vite devenir inutile) ou 0004, faut voir ce que ça casse pour comprendre ce que ça corrige.

C'est le seul truc que j'ai appris pendant mes 3 ans de thèse (c'est dire...), avant de corriger un truc on fait un test qui marche pas.

Branche à jour.