[Findit] Adding central config test class for unit tests

appengine/findit/handlers/config.py

Index: appengine/findit/handlers/config.py
diff --git a/appengine/findit/handlers/config.py b/appengine/findit/handlers/config.py
index 74883848bcde94072804cf7e2e4331020df83f12..e65f52e94f06cb78cc997e5ce66c84a34f4675fa 100644
--- a/appengine/findit/handlers/config.py
+++ b/appengine/findit/handlers/config.py
@@ -11,6 +11,8 @@ from base_handler import Permission
 from model import wf_config
 from waterfall import waterfall_config
 
+from google.appengine.api import users
+
 
 def _RemoveDuplicatesAndSort(elements):
   return list(set(elements))
@@ -243,6 +245,8 @@ class Configuration(BaseHandler):
       return self.CreateError(
           'New configuration settings is not properly formatted.', 400)
 
-    wf_config.FinditConfig.Get().Update(**new_config_dict)
+    wf_config.FinditConfig.Get().Update(users.get_current_user(),
+                                        users.IsCurrentUserAdmin(),

[stgao, 2016/04/04 22:07:23]

nit: Inconsistent with API. is_admin is a key-value arg, while user is a
positional arg.

The testing code seems to have different calling style too.

[lijeffrey, 2016/04/04 23:56:41]

Done.

+                                        **new_config_dict)
 
     return self.HandleGet()