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

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

Issue 1457543004: Add Mocha based tests for Settings pages (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 /** @fileoverview Prototype for Settings page tests. */
6
7 /** @const {string} Path to root from chrome/test/data/webui/settings/. */
8 var ROOT_PATH = '../../../../../';
9
10 // Polymer BrowserTest fixture.
11 GEN_INCLUDE(
12 [ROOT_PATH + 'chrome/test/data/webui/polymer_browser_test_base.js']);
13
14 /**
15 * @constructor
16 * @extends {PolymerTest}
17 */
18 function SettingsPageBrowserTest() {}
19
20 SettingsPageBrowserTest.prototype = {
21 __proto__: PolymerTest.prototype,
22
23 /** @override */
24 browsePreload: 'chrome://md-settings/',
25
26 /** @override */
27 extraLibraries: PolymerTest.getLibraries(ROOT_PATH),
28
29 runAccessibilityChecks: false,
30
31 getPage: function(type) {
dpapad 2015/11/18 01:44:54 Can we add type annotations for all parameters and
stevenjb 2015/11/18 20:02:33 Done.
32 var settings = document.querySelector('cr-settings');
33 assertTrue(!!settings);
dpapad 2015/11/18 01:44:54 How about assertNotNull(settings);
stevenjb 2015/11/18 20:02:33 ReferenceError: assertNotNull is not defined I do
34 var settingsUi = settings.shadowRoot.querySelector('settings-ui');
35 assertTrue(!!settingsUi);
36 var settingsMain = settingsUi.shadowRoot.querySelector('settings-main');
37 assertTrue(!!settingsMain);
38 var pages = settingsMain.$.pageContainer;
39 assertTrue(!!pages);
40 var pageType = 'settings-' + type + '-page';
dpapad 2015/11/18 01:44:54 @dbeam Can we start using relatively new v8 featur
41 var page = pages.querySelector(pageType);
42 assertTrue(!!page);
43 return page;
44 },
45
46 getSection: function(page, section) {
47 var sections = page.shadowRoot.querySelectorAll('settings-section');
48 assertTrue(!!sections);
49 var len = sections.length;
50 assertGT(len, 0);
51 for (var i = 0; i < len; ++i) {
52 var s = sections[i];
53 if (s.section == section)
54 return s;
55 }
56 assertNotReached('Section not found: ' + section + ' in ' + page);
57 return undefined;
58 },
59 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698