- The ``repoze.bfg.url.route_url`` function inappropriately passed

  along ``_query`` and/or ``_anchor`` arguments to the
  ``mapper.generate`` function, resulting in blowups.

2 files changed, 4 insertions, 6 deletions

diff --git a/repoze/bfg/tests/test_url.py b/repoze/bfg/tests/test_url.py
index 86c1cdf11..0f5050cd4 100644
--- a/repoze/bfg/tests/test_url.py
+++ b/repoze/bfg/tests/test_url.py
@@ -176,7 +176,7 @@ class TestRouteUrl(unittest.TestCase):
         mapper.raise_exc = KeyError
         self.assertRaises(KeyError, self._callFUT, 'flub', request, a=1)
 
-    def test_malthe_wiggy_add_your_bug_here(self):
+    def test_generate_doesnt_receive_query_or_anchor(self):
         from repoze.bfg.interfaces import IRoutesMapper
         mapper = DummyRoutesMapper(result='')
         from zope.component import getSiteManager
@@ -184,6 +184,7 @@ class TestRouteUrl(unittest.TestCase):
         sm.registerUtility(mapper, IRoutesMapper)
         request = DummyRequest()
         result = self._callFUT('flub', request, _query=dict(name='some_name'))
+        self.assertEqual(mapper.kw, {}) # shouldnt have anchor/query
         self.assertEqual(result, 'http://example.com:5432?name=some_name')
 
 class TestStaticUrl(unittest.TestCase):
@@ -266,7 +267,8 @@ class DummyRoutesMapper:
     def get_routes(self):
         return self.routes
         
-    def generate(self, *route_args, **newargs):
+    def generate(self, *route_args, **kw):
+        self.kw = kw
         if self.raise_exc:
             raise self.raise_exc
         return self.result
diff --git a/repoze/bfg/tests/test_urldispatch.py b/repoze/bfg/tests/test_urldispatch.py
index 5e84b88b2..84484b692 100644
--- a/repoze/bfg/tests/test_urldispatch.py
+++ b/repoze/bfg/tests/test_urldispatch.py
@@ -35,10 +35,6 @@ class TestRoute(unittest.TestCase):
         route = self._makeOne(':path')
         self.assertEqual(route.generate({'path':'abc'}), '/abc')
 
-    def test_generate_with_query(self):
-        route = self._makeOne('foo')
-        self.assertEqual(route.generate({'_query': dict(foo=1)}), '/?foo=1')
-
 class RoutesMapperTests(unittest.TestCase):
     def setUp(self):
         testing.setUp()