-
Notifications
You must be signed in to change notification settings - Fork 161
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
Refactorise le templatetag "joinby" #4653
Conversation
Ce commit refactorise le templatetag "joinby" et en modifie l'utilisation. La documentation et le template qui l'utilise ("templates/tutorialv2/view/content_online.html") sont donc mis à jour en même temps.
Ce commit refactorise le templatetag "joinby" et en modifie l'utilisation. La documentation et le template qui l'utilise ("templates/tutorialv2/view/content_online.html") sont donc mis à jour en même temps.
…into improve-joinby
…improve-joinby
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça me semble facile d’écrire un petit test pour ça :-)
Oui, je bosse dessus en ce moment même. Tu pense que devrais le rajouter à cette PR ou dans une autre ? :) |
Dans cette PR c’est très bien. |
…improve-joinby
zds/utils/tests/test_misc.py
Outdated
@@ -56,3 +57,48 @@ def test_remove_url_scheme(self): | |||
for element in oracle: | |||
# as we are not in py3 we do not have subTest method. so we use a bare for loop. | |||
self.assertEquals(remove_url_scheme(element.given), element.expected) | |||
|
|||
|
|||
class TemplateTagsTest(TestCase): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Faut surtout tester les cas particuliers qui risquent de planter, comme joinby([])
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah oui, bien vu. Cependant, ce cas est prévu par le " if not values: return '' ". ;)
…improve-joinby
…improve-joinby
…improve-joinby
Désolé pour le retard, j'avais un peu oublié cette PR. |
par contre va falloir faire un rebase, il y a des conflits |
Cette PR refactorise le templatetag "joinby" afin d'améliorer son rendu. Elle change son fonctionnement, et la doc et le template qui l'utilisait sont modifiés également. Elle fait suite à une remarque de @artragis ici, et elle améliore au passage la gestion des multi-auteurs dans les citations. Désolé pour les commits de merge disgracieux.
QA