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

Side by Side Diff: chrome/test/data/webui/settings/route_tests.js

Issue 2224163002: Settings Router Refactor: Replace route.subpage usage with route.path (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge Created 4 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 suite('route', function() { 5 suite('route', function() {
6 test('tree structure', function() { 6 test('tree structure', function() {
7 // Set up root page routes. 7 // Set up root page routes.
8 var BASIC = new settings.Route('/'); 8 var BASIC = new settings.Route('/');
9 var ADVANCED = new settings.Route('/advanced'); 9 var ADVANCED = new settings.Route('/advanced');
10 assertDeepEquals([], ADVANCED.subpage); 10 assertFalse(ADVANCED.isSubpage());
11 11
12 // Test a section route. 12 // Test a section route.
13 var PRIVACY = ADVANCED.createChild('/privacy'); 13 var PRIVACY = ADVANCED.createChild('/privacy');
14 PRIVACY.section = 'privacy'; 14 PRIVACY.section = 'privacy';
15 assertEquals(ADVANCED, PRIVACY.parent); 15 assertEquals(ADVANCED, PRIVACY.parent);
16 assertDeepEquals([], PRIVACY.subpage); 16 assertFalse(PRIVACY.isSubpage());
17 assertFalse(BASIC.contains(PRIVACY)); 17 assertFalse(BASIC.contains(PRIVACY));
18 assertTrue(ADVANCED.contains(PRIVACY)); 18 assertTrue(ADVANCED.contains(PRIVACY));
19 assertTrue(PRIVACY.contains(PRIVACY)); 19 assertTrue(PRIVACY.contains(PRIVACY));
20 assertFalse(PRIVACY.contains(ADVANCED)); 20 assertFalse(PRIVACY.contains(ADVANCED));
21 21
22 // Test a subpage route. 22 // Test a subpage route.
23 var SITE_SETTINGS = PRIVACY.createChild('/siteSettings', 'site-settings'); 23 var SITE_SETTINGS = PRIVACY.createChild('/siteSettings');
24 assertEquals('/siteSettings', SITE_SETTINGS.path); 24 assertEquals('/siteSettings', SITE_SETTINGS.path);
25 assertEquals(PRIVACY, SITE_SETTINGS.parent); 25 assertEquals(PRIVACY, SITE_SETTINGS.parent);
26 assertFalse(!!SITE_SETTINGS.dialog); 26 assertFalse(!!SITE_SETTINGS.dialog);
27 assertDeepEquals(['site-settings'], SITE_SETTINGS.subpage); 27 assertTrue(SITE_SETTINGS.isSubpage());
28 assertEquals('privacy', SITE_SETTINGS.section); 28 assertEquals('privacy', SITE_SETTINGS.section);
29 assertFalse(BASIC.contains(SITE_SETTINGS)); 29 assertFalse(BASIC.contains(SITE_SETTINGS));
30 assertTrue(ADVANCED.contains(SITE_SETTINGS)); 30 assertTrue(ADVANCED.contains(SITE_SETTINGS));
31 assertTrue(PRIVACY.contains(SITE_SETTINGS)); 31 assertTrue(PRIVACY.contains(SITE_SETTINGS));
32 32
33 // Test a sub-subpage route. 33 // Test a sub-subpage route.
34 var SITE_SETTINGS_ALL = 34 var SITE_SETTINGS_ALL = SITE_SETTINGS.createChild('all');
35 SITE_SETTINGS.createChild('all', 'all-sites');
36 assertEquals('/siteSettings/all', SITE_SETTINGS_ALL.path); 35 assertEquals('/siteSettings/all', SITE_SETTINGS_ALL.path);
37 assertEquals(SITE_SETTINGS, SITE_SETTINGS_ALL.parent); 36 assertEquals(SITE_SETTINGS, SITE_SETTINGS_ALL.parent);
38 assertDeepEquals(['site-settings', 'all-sites'], SITE_SETTINGS_ALL.subpage); 37 assertTrue(SITE_SETTINGS_ALL.isSubpage());
39
40 // Test a dialog route.
41 var CLEAR_BROWSING_DATA =
42 PRIVACY.createDialog('/clearBrowsingData', 'clear-browsing-data');
43 assertEquals(PRIVACY, CLEAR_BROWSING_DATA.parent);
44 assertEquals('clear-browsing-data', CLEAR_BROWSING_DATA.dialog);
45 assertEquals('privacy', CLEAR_BROWSING_DATA.section);
46 assertFalse(BASIC.contains(CLEAR_BROWSING_DATA));
47 assertTrue(ADVANCED.contains(CLEAR_BROWSING_DATA));
48 assertTrue(PRIVACY.contains(CLEAR_BROWSING_DATA));
49 }); 38 });
50 39
51 test('no duplicate routes', function() { 40 test('no duplicate routes', function() {
52 var paths = new Set(); 41 var paths = new Set();
53 Object.values(settings.Route).forEach(function(route) { 42 Object.values(settings.Route).forEach(function(route) {
54 assertFalse(paths.has(route.path), route.path); 43 assertFalse(paths.has(route.path), route.path);
55 paths.add(route.path); 44 paths.add(route.path);
56 }); 45 });
57 }); 46 });
58 }); 47 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698