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

Unified Diff: appengine/components/components/auth/api_test.py

Issue 2836223002: auth: Add a page to setup web client ID for a service. (Closed)
Patch Set: nit Created 3 years, 8 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: appengine/components/components/auth/api_test.py
diff --git a/appengine/components/components/auth/api_test.py b/appengine/components/components/auth/api_test.py
index f2007881f16557e00e0fe80a05d90b5159ad2ca5..9bb31c30d92af7eb15b5771ee1e3127e3b6e3c5f 100755
--- a/appengine/components/components/auth/api_test.py
+++ b/appengine/components/components/auth/api_test.py
@@ -21,6 +21,7 @@ from google.appengine.ext import ndb
from components.auth import api
from components.auth import ipaddr
from components.auth import model
+from components import utils
from test_support import test_case
@@ -213,6 +214,7 @@ class AuthDBTest(test_case.TestCase):
def test_fetch_auth_db(self):
# Client IDs callback.
self.mock(api, '_additional_client_ids_cb', lambda: ['', 'cb_client_id'])
+ self.mock(api, 'get_web_client_id', lambda: 'web_client_id')
# Create AuthGlobalConfig.
global_config = model.AuthGlobalConfig(key=model.root_key())
@@ -276,6 +278,7 @@ class AuthDBTest(test_case.TestCase):
auth_db.ip_whitelists)
self.assertTrue(auth_db.is_allowed_oauth_client_id('1'))
self.assertTrue(auth_db.is_allowed_oauth_client_id('cb_client_id'))
+ self.assertTrue(auth_db.is_allowed_oauth_client_id('web_client_id'))
self.assertFalse(auth_db.is_allowed_oauth_client_id(''))
def test_get_secret(self):
@@ -791,6 +794,15 @@ class OAuthAccountsTest(test_case.TestCase):
api.extract_oauth_caller_identity()
+class AuthWebUIConfigTest(test_case.TestCase):
+ def test_works(self):
+ utils.clear_cache(api.get_web_client_id)
+ self.assertEqual('', api.get_web_client_id_uncached())
+ api.set_web_client_id('zzz')
+ self.assertEqual('zzz', api.get_web_client_id_uncached())
+ self.assertEqual('zzz', api.get_web_client_id())
+
+
if __name__ == '__main__':
if '-v' in sys.argv:
unittest.TestCase.maxDiff = None

Powered by Google App Engine
This is Rietveld 408576698