Produits Entr'ouvert » w.c.s.

Un test qui fait planter :

diff --git a/tests/test_variadic_url.py b/tests/test_variadic_url.py
index 447bbfd..f8f4dd0 100644
--- a/tests/test_variadic_url.py
+++ b/tests/test_variadic_url.py
@@ -64,6 +64,10 @@ def test_url_server():
     assert get_variadic_url('[url]/foobar',
             {'url': 'http://www.example.net'}) == 'http://www.example.net/foobar'

+def test_url_server_qs_without_slash():
+    assert get_variadic_url('[url]?foo=bar',
+            {'url': 'http://www.example.net/'}) == 'http://www.example.net/?foo=bar'
+
 def test_url_server_more():
     assert get_variadic_url('[url]/foobar/json?toto',
             {'url': 'http://www.example.net'}) == 'http://www.example.net/foobar/json?toto'

donne :

E       assert 'http://www.example.net/' == 'http://www.example.net/?foo=bar'
E         - http://www.example.net/
E         + http://www.example.net/?foo=bar
E         ?                        ++++++++

J'en suis à regarder et à penser que ça se joue ici :

323                 if p2.path and not path:
324                     path, query = p2.path, p2.query

où on oublie de penser qu'il y a peut-être une query existante, qu'on oublie gentilment (foo=bar dans le test)