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

Side by Side Diff: chrome/test/data/webui/settings/languages_page_browsertest.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 /** @fileoverview Suite of tests for settings-languages-page. */ 5 /** @fileoverview Suite of tests for settings-languages-page. */
6 6
7 /** @const {string} Path to root from chrome/test/data/webui/settings/. */ 7 /** @const {string} Path to root from chrome/test/data/webui/settings/. */
8 var ROOT_PATH = '../../../../../'; 8 var ROOT_PATH = '../../../../../';
9 9
10 // Polymer BrowserTest fixture. 10 // Polymer BrowserTest fixture.
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 languagesSection = this.getSection(advanced, 'languages'); 61 languagesSection = this.getSection(advanced, 'languages');
62 assertTrue(!!languagesSection); 62 assertTrue(!!languagesSection);
63 languagesPage = languagesSection.querySelector('settings-languages-page'); 63 languagesPage = languagesSection.querySelector('settings-languages-page');
64 assertTrue(!!languagesPage); 64 assertTrue(!!languagesPage);
65 65
66 return LanguageHelperImpl.getInstance().whenReady(); 66 return LanguageHelperImpl.getInstance().whenReady();
67 }.bind(this)); 67 }.bind(this));
68 68
69 teardown(function(done) { 69 teardown(function(done) {
70 // Close the section if we're in a sub-page. 70 // Close the section if we're in a sub-page.
71 if (settings.getCurrentRoute().subpage.length == 0) { 71 if (settings.getCurrentRoute().isSubpage()) {
72 done();
73 } else {
74 settings.navigateTo(settings.Route.ADVANCED); 72 settings.navigateTo(settings.Route.ADVANCED);
75 setTimeout(done); 73 setTimeout(done);
74 } else {
75 done();
76 } 76 }
77 }); 77 });
78 78
79 test('manage languages', function() { 79 test('manage languages', function() {
80 var manageLanguagesButton = 80 var manageLanguagesButton =
81 languagesPage.$.languagesCollapse.querySelector( 81 languagesPage.$.languagesCollapse.querySelector(
82 '.list-button:last-of-type'); 82 '.list-button:last-of-type');
83 MockInteractions.tap(manageLanguagesButton); 83 MockInteractions.tap(manageLanguagesButton);
84 assertTrue(!!languagesPage.$$('settings-manage-languages-page')); 84 assertTrue(!!languagesPage.$$('settings-manage-languages-page'));
85 }); 85 });
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 assertTrue(!!languagesPage.$$('settings-edit-dictionary-page')); 123 assertTrue(!!languagesPage.$$('settings-edit-dictionary-page'));
124 } 124 }
125 }); 125 });
126 }.bind(this)); 126 }.bind(this));
127 127
128 // TODO(michaelpg): Test more aspects of the languages UI. 128 // TODO(michaelpg): Test more aspects of the languages UI.
129 129
130 // Run all registered tests. 130 // Run all registered tests.
131 mocha.run(); 131 mocha.run();
132 }); 132 });
OLDNEW
« no previous file with comments | « chrome/test/data/webui/settings/device_page_tests.js ('k') | chrome/test/data/webui/settings/route_tests.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698