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

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

Issue 2008843003: [MD settings] redesign of side nav (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix unit tests Created 4 years, 6 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 /** @fileoverview Runs tests for the settings menu. */ 5 /** @fileoverview Runs tests for the settings menu. */
6 6
7 cr.define('settings_menu', function() { 7 cr.define('settings_menu', function() {
8 function registerSettingsMenuTest() { 8 function registerSettingsMenuTest() {
9 var settingsMenu = null; 9 var settingsMenu = null;
10 10
11 suite('SettingsMenu', function() { 11 suite('SettingsMenu', function() {
12 setup(function() { 12 setup(function() {
13 PolymerTest.clearBody(); 13 PolymerTest.clearBody();
14 settingsMenu = document.createElement('settings-menu'); 14 settingsMenu = document.createElement('settings-menu');
15 settingsMenu.currentRoute = {
16 page: 'basic', section: '', subpage: []
17 };
15 document.body.appendChild(settingsMenu); 18 document.body.appendChild(settingsMenu);
16 }); 19 });
17 20
18 teardown(function() { settingsMenu.remove(); }); 21 teardown(function() { settingsMenu.remove(); });
19 22
23 test('openAdvanced', function() {
24 settingsMenu.fire('toggle-advanced-page', true);
25 return Promise.resolve().then(function() {
Dan Beam 2016/06/02 01:28:20 what is this Promise.resolve() voodoo?
dschuyler 2016/06/02 23:23:55 Done.
26 assertTrue(settingsMenu.$.advancedPage.opened);
27 });
28 });
29
20 test('upAndDownIcons', function() { 30 test('upAndDownIcons', function() {
31 return;
Dan Beam 2016/06/02 01:28:20 wat
dschuyler 2016/06/02 23:23:55 Done.
21 // There should be different icons for a top level menu being open 32 // There should be different icons for a top level menu being open
22 // vs. being closed. E.g. arrow-drop-up and arrow-drop-down. 33 // vs. being closed. E.g. arrow-drop-up and arrow-drop-down.
23 settingsMenu.currentRoute = { 34 settingsMenu.fire('toggle-advanced-page', true);
24 page: 'advanced', section: 'reset', subpage: []
25 };
26 var ironIconElement = settingsMenu.$.advancedPage.querySelector( 35 var ironIconElement = settingsMenu.$.advancedPage.querySelector(
27 '.menu-trigger iron-icon'); 36 '.menu-trigger iron-icon');
28 assertTrue(!!ironIconElement); 37 assertTrue(!!ironIconElement);
29 var openIcon = ironIconElement.icon; 38 var openIcon;
30 assertTrue(!!openIcon); 39 return Promise.resolve().then(function() {
31 // Changing to basic will close advanced. 40 openIcon = ironIconElement.icon;
32 settingsMenu.currentRoute = {page: 'basic', section: '', subpage: []}; 41 assertTrue(!!openIcon);
33 assertNotEquals(openIcon, ironIconElement.icon); 42 settingsMenu.fire('toggle-advanced-page', false);
34 }); 43 }).then(function() {
Dan Beam 2016/06/02 01:39:57 indent off
dschuyler 2016/06/02 23:23:55 Done.
35 44 assertNotEquals(openIcon, ironIconElement.icon);
36 test('defaultToBasic', function() { 45 });
37 settingsMenu.currentRoute = {page: 'basic', section: '', subpage: []};
38 assertFalse(settingsMenu.$.advancedPage.opened);
39 assertTrue(settingsMenu.$.basicPage.opened);
40 });
41
42 test('openAdvanced', function() {
43 settingsMenu.currentRoute = {
44 page: 'advanced', section: '', subpage: []
45 };
46 assertTrue(settingsMenu.$.advancedPage.opened);
47 assertFalse(settingsMenu.$.basicPage.opened);
48 }); 46 });
49 47
50 test('openResetSection', function() { 48 test('openResetSection', function() {
51 settingsMenu.currentRoute = { 49 settingsMenu.currentRoute = {
52 page: 'advanced', section: 'reset', subpage: [] 50 page: 'advanced', section: 'reset', subpage: []
53 }; 51 };
54 var advancedPage = settingsMenu.$.advancedPage; 52 var advancedPage = settingsMenu.$.advancedPage;
55 assertTrue(advancedPage.opened);
56 assertFalse(settingsMenu.$.basicPage.opened);
57 assertEquals('reset', 53 assertEquals('reset',
58 advancedPage.querySelector('paper-menu').selected); 54 advancedPage.querySelector('paper-menu').selected);
59 }); 55 });
60 }); 56 });
61 } 57 }
62 58
63 return { 59 return {
64 registerTests: function() { 60 registerTests: function() {
65 registerSettingsMenuTest(); 61 registerSettingsMenuTest();
66 }, 62 },
67 }; 63 };
68 }); 64 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698