[MD settings] adding a test for Appearance settings

chrome/test/data/webui/settings/appearance_browsertest.js

Index: chrome/test/data/webui/settings/appearance_browsertest.js
diff --git a/chrome/test/data/webui/settings/appearance_browsertest.js b/chrome/test/data/webui/settings/appearance_browsertest.js
new file mode 100644
index 0000000000000000000000000000000000000000..58720d60d3019b1f823997d9538442a818fce0c1
--- /dev/null
+++ b/chrome/test/data/webui/settings/appearance_browsertest.js
@@ -0,0 +1,70 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/** @fileoverview Runs Polymer Appearance Settings elements. */
+
+/** @const {string} Path to source root. */
+var ROOT_PATH = '../../../../../';
+
+// Polymer BrowserTest fixture.
+GEN_INCLUDE(
+    [ROOT_PATH + 'chrome/test/data/webui/polymer_browser_test_base.js']);
+
+/**
+ * Test Polymer Appearance Settings elements.
+ * @constructor
+ * @extends {PolymerTest}
+*/
+function AppearanceSettingsBrowserTest() {}
+
+AppearanceSettingsBrowserTest.prototype = {
+  __proto__: PolymerTest.prototype,
+
+  /** @override */
+  browsePreload: 'chrome://md-settings/',
+
+  commandLineSwitches: [{switchName: 'enable-md-settings'}],
+
+  /** @override */
+  extraLibraries: PolymerTest.getLibraries(ROOT_PATH),
+
+  /** @override */
+  runAccessibilityChecks: false,
+
+  /** @return {String} */

[michaelpg, 2015/11/19 02:44:05]

string (lowercase)

[dschuyler, 2015/11/19 17:09:13]

Done.

+  appearancePage: function(selector) {
+    var crSettings = document.querySelector('cr-settings');
+    assertTrue(!!crSettings);
+    var settingsUi = crSettings.shadowRoot.querySelector('settings-ui');

[michaelpg, 2015/11/19 02:44:04]

Polymer provides $$ to querySelector a Polymer elment's shadow root:

crSettings.$$('settings-ui')

[dschuyler, 2015/11/19 17:09:12]

Done.

+    assertTrue(!!settingsUi);
+    var settingsMain = settingsUi.shadowRoot.querySelector('settings-main');
+    assertTrue(!!settingsMain);
+    var basic = settingsMain.shadowRoot.querySelector('settings-basic-page');
+    assertTrue(!!basic);
+    var appearance = basic.shadowRoot.querySelector('settings-appearance-page');
+    assertTrue(!!appearance);
+    var result = appearance.shadowRoot.querySelector(selector);
+    assertTrue(!!result);
+    return result;
+  },
+};
+
+// Run all tests.

[michaelpg, 2015/11/19 02:44:04]

comment isn't really relevant here

[dschuyler, 2015/11/19 17:09:13]

Done.

+TEST_F('AppearanceSettingsBrowserTest', 'uiTests', function() {
+  var self = this;
+  suite('AppearanceHandler', function() {
+    test('font settings', function() {
+      var fontSize = self.appearancePage(
+          '#defaultFontSize').shadowRoot.querySelector(
+          '[class=iron-selected]');
+      assertTrue(fontSize.textContent == "Medium");

[michaelpg, 2015/11/19 02:44:05]

assertEquals('Medium', fS.tC);

[dschuyler, 2015/11/19 17:09:13]

Done.

+    });
+
+    test('home button', function() {
+      var homeButton = self.appearancePage('#showHomeButton');
+      assertTrue(homeButton.attributes['aria-pressed'].nodeValue == "false");

[michaelpg, 2015/11/19 02:44:04]

assertEquals('false', ...)

also: why not homeButton.getAttribute('aria-pressed')?

[dschuyler, 2015/11/19 17:09:13]

Done.

+    });
+  });
+  mocha.run();
+});