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

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

Issue 2989833002: config_service: include type of config to the config-set card (Closed)
Patch Set: Change config-set cards. Created 3 years, 4 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 unified diff | Download patch
« no previous file with comments | « appengine/config_service/ui/src/config-ui/config-set-card.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!-- 1 <!--
2 Copyright 2017 The LUCI Authors. All rights reserved. 2 Copyright 2017 The LUCI Authors. All rights reserved.
3 Use of this source code is governed under the Apache License, Version 2.0 3 Use of this source code is governed under the Apache License, Version 2.0
4 that can be found in the LICENSE file. 4 that can be found in the LICENSE file.
5 --> 5 -->
6 6
7 <!doctype html> 7 <!doctype html>
8 <html lang="en"> 8 <html lang="en">
9 <head> 9 <head>
10 <meta charset="utf-8"> 10 <meta charset="utf-8">
(...skipping 272 matching lines...) Expand 10 before | Expand all | Expand 10 after
283 'GET', 283 'GET',
284 /\/_ah\/api\/config\/v1\/config-sets\?config_set=([a-z\-]+)\/([a-z\- ]+)&include_files=true&include_last_import_attempt=true/, 284 /\/_ah\/api\/config\/v1\/config-sets\?config_set=([a-z\-]+)\/([a-z\- ]+)&include_files=true&include_last_import_attempt=true/,
285 [ 285 [
286 errorNumber, 286 errorNumber,
287 responseHeaders.json, 287 responseHeaders.json,
288 JSON.stringify(param) 288 JSON.stringify(param)
289 ] 289 ]
290 ); 290 );
291 } 291 }
292 292
293 test('displays error if fetch failed due to error 403 when not signed in ', function (done) { 293 test('displays error on 403 when not signed in', function (done) {
294 setserver(403); 294 setserver(403);
295 request = ajax.generateRequest(); 295 request = ajax.generateRequest();
296 server.respond(); 296 server.respond();
297 assert.equal(request.status, 403); 297 assert.equal(request.status, 403);
298 config_set.addEventListener('fetchError', function() { 298 config_set.addEventListener('fetchError', function() {
299 assert.equal(config_set.isLoading, false); 299 assert.equal(config_set.isLoading, false);
300 assert.equal(config_set.errorMessage, "Access denied, please sign in ."); 300 assert.equal(config_set.errorMessage,
301 "Access denied, please sign in.");
301 done(); 302 done();
302 }.bind(config_set)); 303 }.bind(config_set));
303 }); 304 });
304 305
305 test('displays error if fetch failed due to error 403 when signed in', f unction (done) { 306 test('displays error on 403 when signed in', function (done) {
306 setserver(403); 307 setserver(403);
307 config_set.auth_headers = {}; 308 config_set.auth_headers = {};
308 config_set.profile = { 309 config_set.profile = {
309 email: "some_email@google.com" 310 email: "some_email@google.com"
310 }; 311 };
311 request = ajax.generateRequest(); 312 request = ajax.generateRequest();
312 server.respond(); 313 server.respond();
313 assert.equal(request.status, 403); 314 assert.equal(request.status, 403);
314 config_set.addEventListener('fetchError', function() { 315 config_set.addEventListener('fetchError', function() {
315 assert.equal(config_set.isLoading, false); 316 assert.equal(config_set.isLoading, false);
316 assert.equal(config_set.errorMessage, "Access denied, some_email@goo gle.com" + 317 assert.equal(config_set.errorMessage,
318 "Access denied, some_email@google.com" +
317 " is not authorized to access this config set." + 319 " is not authorized to access this config set." +
318 " Request access or sign in as a different user."); 320 " Request access or sign in as a different user.");
319 done(); 321 done();
320 }.bind(config_set)); 322 }.bind(config_set));
321 }); 323 });
322 324
323 test('displays error if fetch failed due to error 500', function (done) { 325 test('displays error if fetch failed due to error 500', function (done) {
324 setserver(500); 326 setserver(500);
325 request = ajax.generateRequest(); 327 request = ajax.generateRequest();
326 server.respond(); 328 server.respond();
327 assert.equal(request.status, 500); 329 assert.equal(request.status, 500);
328 config_set.addEventListener('fetchError', function() { 330 config_set.addEventListener('fetchError', function() {
329 assert.equal(config_set.isLoading, false); 331 assert.equal(config_set.isLoading, false);
330 assert.equal(config_set.errorMessage, "Internal server error."); 332 assert.equal(config_set.errorMessage, "Internal server error.");
331 done(); 333 done();
332 }.bind(config_set)); 334 }.bind(config_set));
333 }); 335 });
334 336
335 }); 337 });
336 338
337 </script> 339 </script>
338 </body> 340 </body>
339 </html> 341 </html>
OLDNEW
« no previous file with comments | « appengine/config_service/ui/src/config-ui/config-set-card.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698