Permalink
Browse files

remove 2 of my comments, add 1 comment

  • Loading branch information...
1 parent 5cb99a9 commit b73f5d52f2f9563da54dfa073e92ad272ac1275e Bayle Shanks committed Feb 1, 2011
Showing with 2 additions and 6 deletions.
  1. +0 −6 pyramid/tests/test_config.py
  2. +2 −0 pyramid/tests/test_url.py
@@ -1859,9 +1859,6 @@ def test_add_route_wsgi(self):
def dummyapp(environ, start_response):
""" """
-
- # since add_route_wsgi is just one line, this test is
- # kind of meaningless
config = self._makeOne(autocommit=True)
route = config.add_route_wsgi('name', 'path*subpath', dummyapp)
self._assertRoute(config, 'name', 'path*subpath')
@@ -1888,9 +1885,6 @@ def __call__(self, status, headers):
self.status = status
self.headers = headers
-
- # since add_route_wsgi is just one line, this test is
- # kind of meaningless
config = self._makeOne(autocommit=True)
route = config.add_route_wsgi('name', '/{a}/2/3*subpath', dummyapp)
@@ -249,6 +249,8 @@ def test_with__route_name(self):
_anchor=u"foo", _route_name='bar')
self.assertEqual(result,
'http://example.com:5432/1/2/3/extra1/extra2?a=1#foo')
+ #note: the reason that this matches even though _route_name is 'bar' is
+ # that the DummyRoutesMapper ignores the route name
class TestRoutePath(unittest.TestCase):
def setUp(self):

0 comments on commit b73f5d5

Please sign in to comment.