MD Settings: split browser tests per-file

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

 // 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 the Polymer Settings tests. */
 
 /** @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 fixture for Polymer Settings elements.
  * @constructor
  * @extends {PolymerTest}
 */
 function CrSettingsBrowserTest() {}
 
 CrSettingsBrowserTest.prototype = {
   __proto__: PolymerTest.prototype,
 
   /** @override */
   browsePreload: 'chrome://md-settings/prefs/prefs.html',
 
-  /**
-   * TODO(dbeam): these should not be required monolithically.
-   * @override
-   */
+  /** @override */
   extraLibraries: PolymerTest.getLibraries(ROOT_PATH).concat([
     '../fake_chrome_event.js',
     'fake_settings_private.js',

[dpapad, 2016/06/13 18:19:46]

Are those two files needed by the majority of CrSettingsBrowserTest subclasses?
If not, can we simply move those extra libraries in their respective subclasses
instead?

-    'checkbox_tests.js',
-    'dropdown_menu_tests.js',
-    'pref_util_tests.js',
-    'prefs_test_cases.js',
-    'prefs_tests.js',
   ]),
 
   /** @override */
   setUp: function() {
     PolymerTest.prototype.setUp.call(this);
     // We aren't loading the main document.
     this.accessibilityAuditConfig.ignoreSelectors('humanLangMissing', 'html');
 
     // TODO(michaelpg): Re-enable after bringing in fix for
     // https://github.com/PolymerElements/paper-slider/issues/131.
     this.accessibilityAuditConfig.ignoreSelectors(
         'badAriaAttributeValue', 'paper-slider');
   },
 };
 
 // Have to include command_line.h manually due to GEN calls below.
 GEN('#include "base/command_line.h"');
 
-TEST_F('CrSettingsBrowserTest', 'Checkbox', function() {
+function CrSettingsCheckboxTest() {}
+
+CrSettingsCheckboxTest.prototype = {
+  __proto__: CrSettingsBrowserTest.prototype,
+
+  /** @override */
+  extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
+    'checkbox_tests.js',
+  ]),
+};
+
+TEST_F('CrSettingsCheckboxTest', 'All', function() {
   settings_checkbox.registerTests();
   mocha.run();
 });
 
-TEST_F('CrSettingsBrowserTest', 'DropdownMenu', function() {
+function CrSettingsDropdownMenuTest() {}
+
+CrSettingsDropdownMenuTest.prototype = {
+  __proto__: CrSettingsBrowserTest.prototype,
+

[dpapad, 2016/06/13 18:19:46]

Since we split this to a separate C++ test fixture, how about specifying
browserPreload here and remove it from
https://cs.chromium.org/chromium/src/chrome/test/data/webui/settings/dropdown....

[michaelpg, 2016/06/14 00:30:21]

can't remember why i didn't just import i18n_setup.html in
settings_dropdown_menu.html. If that works, +1

+  /** @override */
+  extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
+    'dropdown_menu_tests.js',
+  ]),
+};
+
+TEST_F('CrSettingsDropdownMenuTest', 'All', function() {
   settings_dropdown_menu.registerTests();
   mocha.run();
 });
 
-TEST_F('CrSettingsBrowserTest', 'PrefUtil', function() {
+function CrSettingsPrefUtilTest() {}
+
+CrSettingsPrefUtilTest.prototype = {
+  __proto__: CrSettingsBrowserTest.prototype,
+

[dpapad, 2016/06/13 18:19:46]

Since we split this to a separate C++ test fixture, how about specifying
browserPreload here and remove it from
https://cs.chromium.org/chromium/src/chrome/test/data/webui/settings/pref_uti....

+  /** @override */
+  extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
+    'pref_util_tests.js',
+  ]),
+};
+
+TEST_F('CrSettingsPrefUtilTest', 'All', function() {
   settings_prefUtil.registerTests();
   mocha.run();
 });
 
-TEST_F('CrSettingsBrowserTest', 'Prefs', function() {
+function CrSettingsPrefsTest() {}
+
+CrSettingsPrefsTest.prototype = {
+  __proto__: CrSettingsBrowserTest.prototype,
+
+  /** @override */
+  extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
+    'prefs_test_cases.js',
+    'prefs_tests.js',
+  ]),
+};
+
+TEST_F('CrSettingsPrefsTest', 'All', function() {
   settings_prefs.registerTests();
   mocha.run();
 });
 
 /**
  * @constructor
  * @extends {CrSettingsBrowserTest}
  */
 function CrSettingsAboutPageTest() {}
 
@@ skipping 468 matching lines @@
   extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
     'fake_language_settings_private.js',
     'fake_settings_private.js',
     'languages_page_tests.js',
   ]),
 };
 
 TEST_F('CrSettingsLanguagesPageTest', 'LanguagesPage', function() {
   mocha.run();
 });