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

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

Issue 2469393004: MD Settings: Preserve search URL param in subpages. (Closed)
Patch Set: Function params nit. Created 4 years, 1 month 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
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 Polymer.dom.flush(); 60 Polymer.dom.flush();
61 assertNotEquals(openIcon, ironIconElement.icon); 61 assertNotEquals(openIcon, ironIconElement.icon);
62 }); 62 });
63 63
64 test('openResetSection', function() { 64 test('openResetSection', function() {
65 settingsMenu.currentRoute = settings.Route.RESET; 65 settingsMenu.currentRoute = settings.Route.RESET;
66 var advancedPage = settingsMenu.$.advancedPage; 66 var advancedPage = settingsMenu.$.advancedPage;
67 assertEquals('/reset', 67 assertEquals('/reset',
68 advancedPage.querySelector('paper-menu').selected); 68 advancedPage.querySelector('paper-menu').selected);
69 }); 69 });
70
71 // Test that navigating via the paper menu always clears the current
72 // search URL parameter.
73 test('clearsUrlSearchParam', function() {
74 var urlParams = new URLSearchParams('search=foo');
75 settings.navigateTo(settings.Route.BASIC, urlParams);
76 assertEquals(
77 urlParams.toString(),
78 settings.getQueryParameters().toString());
79 MockInteractions.tap(settingsMenu.$.people);
80 assertEquals('', settings.getQueryParameters().toString());
81 });
70 }); 82 });
71 } 83 }
72 84
73 return { 85 return {
74 registerTests: function() { 86 registerTests: function() {
75 registerSettingsMenuTest(); 87 registerSettingsMenuTest();
76 }, 88 },
77 }; 89 };
78 }); 90 });
OLDNEW
« no previous file with comments | « chrome/test/data/webui/settings/route_tests.js ('k') | chrome/test/data/webui/settings/settings_ui_browsertest.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698