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

Unified Diff: appengine/config_service/ui/test/config-ui/config-set_test.html

Issue 2990713002: config_service: change error message for error 403. (Closed)
Patch Set: Nit: forgot to revert URLs for local deployment Created 3 years, 5 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/config_service/ui/test/config-ui/config-set_test.html
diff --git a/appengine/config_service/ui/test/config-ui/config-set_test.html b/appengine/config_service/ui/test/config-ui/config-set_test.html
index 978e4c0a327e3e4f8f648fd4eb70ec7e0962aac4..b73c3605a6947772a2dabd1b5a1190536b673ae4 100644
--- a/appengine/config_service/ui/test/config-ui/config-set_test.html
+++ b/appengine/config_service/ui/test/config-ui/config-set_test.html
@@ -293,8 +293,8 @@
assert.equal(request.status, 403);
config_set.addEventListener('fetchError', function() {
assert.equal(config_set.isLoading, false);
- assert.equal(config_set.errorMessage, "Authorization required to view this config set." +
- " Please sign in.");
+ assert.equal(config_set.errorMessage, "Access denied. If you are not signed in, please do so. " +
+ "Otherwise, try again later.");
Sergey Berezin 2017/07/26 22:35:50 Shouldn't the tests be updated with the new messag
ayanaadylova 2017/07/26 22:49:07 Done.
done();
}.bind(config_set));
});
« no previous file with comments | « appengine/config_service/ui/src/config-ui/front-page.html ('k') | appengine/config_service/ui/test/index.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698