Add refresh support to the console page.

PRESUBMIT.py

Index: PRESUBMIT.py
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index e0f0dfb0b16a73b7112080989db6f4755c2ab51f..63e49791ebcdaedb96f8a863394be7fd5b6e851b 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -16,12 +16,12 @@ Setup instructions:
 
 To run the tests directly:
   export PYTHONPATH="../google_appengine/"
-  nosetests --with-gae --gae-lib-root=../
+  nosetests --with-gae --gae-lib-root=../google_appengine/
 
 To interact with the environment:
   (insert a line: import code; code.interact(locals=local()))
   export PYTHONPATH="../google_appengine/"
-  nosetests --with-gae --gae-lib-root=../ -s
+  nosetests --with-gae --gae-lib-root=../google_appengine/ -s
 """
 
 
@@ -72,10 +72,14 @@ def CommonChecks(input_api, output_api):
         input_api.os_path.join(sdk_path, 'lib'),
         input_api.os_path.join(sdk_path, 'lib', 'simplejson'),
     ] + sys.path
+    disabled_warnings = [
+        'E1101',  # has no member

[M-A Ruel, 2012/05/29 18:46:33]

If possible, you should try fixing it, it's a good warning.

[cmp, 2012/05/29 19:38:03]

I disabled this because it gives false positives due to webapp2 and AppEngine's
SDK:

************* Module app_test
E1101: 43,16:GaeTestCase.clear_datastore: Instance of 'APIProxyStubMap' has no
'_APIProxyStubMap__stub_map' member
E1101: 47,12:GaeTestCase.clear_datastore: Instance of 'APIProxyStubMap' has no
'_APIProxyStubMap__stub_map' member
************* Module base_page
E1101: 65,42:BasePage.InitializeTemplate: Instance of 'BasePage' has no
'request' member
E1101: 66,44:BasePage.InitializeTemplate: Instance of 'BasePage' has no
'request' member
E1101: 79,4:BasePage.DisplayTemplate: Instance of 'BasePage' has no 'response'
member
E1101: 88,4:BasePage.DisplayTemplate: Instance of 'BasePage' has no 'response'
member
************* Module handler
E1101: 24,11:PageAction.get: Instance of 'PageAction' has no 'request' member
E1101: 26,13:PageAction.get: Instance of 'PageAction' has no 'request' member
E1101: 33,6:PageAction.get: Instance of 'PageAction' has no 'error' member
E1101: 36,4:PageAction.get: Instance of 'PageAction' has no 'response' member
E1101: 39,7:PageAction.get: Instance of 'PageAction' has no 'request' member
E1101: 46,34:PageAction.get: Instance of 'PageAction' has no 'request' member
E1101: 54,8:PageAction.get: Instance of 'PageAction' has no 'response' member
E1101: 55,8:PageAction.get: Instance of 'PageAction' has no 'response' member
E1101: 61,4:PageAction.get: Instance of 'PageAction' has no 'response' member
E1101: 62,4:PageAction.get: Instance of 'PageAction' has no 'response' member
E1101: 63,4:PageAction.get: Instance of 'PageAction' has no 'response' member
E1101: 78,4:MainAction.get: Instance of 'MainAction' has no 'redirect' member

You suggested elsewhere not explicitly marking each site but to use a general
exemption, so I used this instead.  I would like to keep it, let me know if you
have suggestions for an alternative approach.

+    ]
     results.extend(input_api.canned_checks.RunPylint(
         input_api,
         output_api,
-        black_list=black_list))
+        black_list=black_list,
+        disabled_warnings=disabled_warnings))
   finally:
     sys.path = backup_sys_path
   return results