Extensions Docs Server: Apps samples page

chrome/common/extensions/docs/server2/integration_test.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import logging
 import os
 from StringIO import StringIO
 import unittest
 
 import appengine_memcache as memcache
 import handler
 from handler import Handler
+import url_constants
 
 KNOWN_FAILURES = [
+  # Apps samples fails because it requires fetching data from github.com.
+  'samples.html',

[not at google - send to devlin, 2012/08/09 07:52:36]

... except now we're not testing the extensions either.

This paths need to be qualified, methinks. Like ['apps/samples.html',
'apps/app.html'].

[cduvall, 2012/08/09 19:20:08]

Done.

[not at google - send to devlin, 2012/08/10 06:02:18]

It would be nice to actually test this... and for that we'll need to fake out a
few more parts of appengine, set up some fake paths, and check in some abridged
version of the github zip.

Do in a follow-up, though (but file bug and put here).

[cduvall, 2012/08/10 21:17:47]

Done. Bug is: http://crbug.com/141910

   # Exception in schema compiler (model.py). See http://crbug.com/141279.
-  'app.html',
+  'app.html'
 ]
 
 class _MockResponse(object):
   def __init__(self):
     self.status = 200
     self.out = StringIO()
 
   def set_status(self, status):
     self.status = status
 
 class _MockRequest(object):
   def __init__(self, path):
     self.headers = {}
     self.path = path
 
 class IntegrationTest(unittest.TestCase):
   def testAll(self):
+    logging.getLogger().setLevel(logging.ERROR)

[not at google - send to devlin, 2012/08/09 07:52:36]

remove logging

[cduvall, 2012/08/09 19:20:08]

This isn't a log, it just sets the logging level so that only ERROR level logs
are shown when we're running this test, so there's not unnecessary output.

     base_path = os.path.join('templates', 'public')
     for path, dirs, files in os.walk(base_path):
       for name in files:
         filename = os.path.join(path, name)
         if name in KNOWN_FAILURES or '.' in path or name.startswith('.'):
           continue
         request = _MockRequest(filename.split('/', 2)[-1])
         response = _MockResponse()
         Handler(request, response, local_path='../..').get()
         self.assertEqual(200, response.status)
@@ skipping 12 matching lines @@
       request = _MockRequest('extensions/samples.html')
       request.headers['Accept-Language'] = lang + ';q=0.8'
       response = _MockResponse()
       Handler(request, response, local_path='../..').get()
       self.assertEqual(200, response.status)
       self.assertTrue(response.out.getvalue())
 
   def testWarmupRequest(self):
     for branch in ['dev', 'trunk', 'beta', 'stable']:
       handler.BRANCH_UTILITY_MEMCACHE.Set(
-          branch + '.' +  handler.OMAHA_PROXY_URL,
+          branch + '.' +  url_constants.OMAHA_PROXY_URL,
           'local',
           memcache.MEMCACHE_BRANCH_UTILITY)
     request = _MockRequest('_ah/warmup')
     response = _MockResponse()
     Handler(request, response, local_path='../..').get()
     self.assertEqual(200, response.status)
     # Test that the pages were rendered by checking the size of the output.
     # In python 2.6 there is no 'assertGreater' method.
     self.assertTrue(len(response.out.getvalue()) > 500000)
 
 if __name__ == '__main__':
   unittest.main()