Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6147)

Unified Diff: chrome/common/extensions/docs/server2/render_servlet_test.py

Issue 311793002: Docserver: Fix broken redirect on code.google.com/chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: also test http Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/docs/server2/render_servlet_test.py
diff --git a/chrome/common/extensions/docs/server2/render_servlet_test.py b/chrome/common/extensions/docs/server2/render_servlet_test.py
index d4b97db52b2fcdf9ca38b27c516e0c5134523dd3..330f5002e4b04f1b06e93240e85f6a3c4614f45e 100755
--- a/chrome/common/extensions/docs/server2/render_servlet_test.py
+++ b/chrome/common/extensions/docs/server2/render_servlet_test.py
@@ -19,8 +19,8 @@ class _RenderServletDelegate(RenderServlet.Delegate):
class RenderServletTest(unittest.TestCase):
- def _Render(self, path, headers=None):
- return RenderServlet(Request.ForTest(path, headers=headers),
+ def _Render(self, path, headers=None, host=None):
+ return RenderServlet(Request.ForTest(path, headers=headers, host=host),
_RenderServletDelegate()).Get()
def testExtensionAppRedirect(self):
@@ -34,6 +34,16 @@ class RenderServletTest(unittest.TestCase):
Response.Redirect('/extensions/storage', permanent=True),
self._Render('%s/extensions/storage' % channel))
+ def testOldHostsRedirect(self):
+ self.assertEqual(
+ Response.Redirect('https://developer.chrome.com/extensions',
+ permanent=False),
+ self._Render('/chrome/extensions', host='http://code.google.com'))
+ self.assertEqual(
+ Response.Redirect('https://developer.chrome.com/extensions',
+ permanent=False),
+ self._Render('/chrome/extensions', host='https://code.google.com'))
+
def testNotFound(self):
def create_404_response(real_path):
real_404 = self._Render(real_path)
« no previous file with comments | « chrome/common/extensions/docs/server2/render_servlet.py ('k') | chrome/common/extensions/docs/server2/servlet.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698