Extensions Docs Server: Apps samples page

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

Index: chrome/common/extensions/docs/server2/integration_test.py
diff --git a/chrome/common/extensions/docs/server2/integration_test.py b/chrome/common/extensions/docs/server2/integration_test.py
index 6f0244c39e983bdd40b828b519def5242ccbce4c..4effc42799c1cfc77faa478f5f70d02af4462630 100755
--- a/chrome/common/extensions/docs/server2/integration_test.py
+++ b/chrome/common/extensions/docs/server2/integration_test.py
@@ -3,6 +3,7 @@
 # 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
@@ -10,10 +11,13 @@ 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):
@@ -31,6 +35,7 @@ class _MockRequest(object):
 
 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:
@@ -63,7 +68,7 @@ class IntegrationTest(unittest.TestCase):
   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')