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

Unified Diff: dashboard/dashboard/update_test_suites_test.py

Issue 2350113002: Add request-level caching for privileged queries. (Closed)
Patch Set: Addressed review comments Created 4 years, 3 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
« no previous file with comments | « dashboard/dashboard/testing_common.py ('k') | dashboard/dashboard/utils_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dashboard/dashboard/update_test_suites_test.py
diff --git a/dashboard/dashboard/update_test_suites_test.py b/dashboard/dashboard/update_test_suites_test.py
index 62181b5eed28f680969b5d50bb098fc0d3f15716..bf97604d8f9e3f59f2205ae5752f985065b2383b 100644
--- a/dashboard/dashboard/update_test_suites_test.py
+++ b/dashboard/dashboard/update_test_suites_test.py
@@ -28,6 +28,7 @@ class ListTestSuitesTest(testing_common.TestCase):
datastore_hooks.InstallHooks()
testing_common.SetIsInternalUser('internal@chromium.org', True)
self.UnsetCurrentUser()
+ self.PatchDatastoreHooksRequest()
def testFetchCachedTestSuites_NotEmpty(self):
# If the cache is set, then whatever's there is returned.
@@ -67,6 +68,8 @@ class ListTestSuitesTest(testing_common.TestCase):
})
def testPost_ForcesCacheUpdate(self):
+ request = webapp2.get_request()
+ request.registry['privileged_cached'] = False
key = update_test_suites._NamespaceKey(
update_test_suites._LIST_SUITES_CACHE_KEY)
stored_object.Set(key, {'foo': 'bar'})
« no previous file with comments | « dashboard/dashboard/testing_common.py ('k') | dashboard/dashboard/utils_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698