Commit 13d20aa6 authored by mkearney's avatar mkearney Committed by Commit bot

Slightly changed top-level redirect test

This test should pass, allowing extensions/index to redirect to extensions.

Review URL: https://codereview.chromium.org/886903002

Cr-Commit-Position: refs/heads/master@{#313989}
parent b89fda7b
...@@ -187,6 +187,10 @@ class IntegrationTest(unittest.TestCase): ...@@ -187,6 +187,10 @@ class IntegrationTest(unittest.TestCase):
'Rendered content length was %s vs template content length %s ' 'Rendered content length was %s vs template content length %s '
'when rendering %s' % (len(response.content), len(content), path)) 'when rendering %s' % (len(response.content), len(content), path))
# TODO(kalman): Hack to avoid failing redirects like extensions/index
# to extensions. Better fix would be to parse or whitelist the
# redirects.json files as part of this test.
if not path.endswith('/index'):
check_result(Handler(Request.ForTest(path)).Get()) check_result(Handler(Request.ForTest(path)).Get())
if path.startswith(('apps/', 'extensions/')): if path.startswith(('apps/', 'extensions/')):
......
...@@ -106,10 +106,9 @@ class RenderServletTest(unittest.TestCase): ...@@ -106,10 +106,9 @@ class RenderServletTest(unittest.TestCase):
len(ReadFile('%s%s.html' % (PUBLIC_TEMPLATES, html_file)))) len(ReadFile('%s%s.html' % (PUBLIC_TEMPLATES, html_file))))
def testIndexRender(self): def testIndexRender(self):
response = self._Render('extensions') self.assertEqual(200, self._Render('extensions').status)
self.assertEqual(200, response.status) self.assertEqual(('/extensions', False),
self.assertEqual(self._Render('extensions/index').content.ToString(), self._Render('extensions/index').GetRedirect())
response.content.ToString())
def testOtherRedirectsJsonRedirect(self): def testOtherRedirectsJsonRedirect(self):
response = self._Render('apps/webview_tag') response = self._Render('apps/webview_tag')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment