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

Unified Diff: chrome/test/data/webui/settings/cr_settings_browsertest.js

Issue 2837703002: Change SettingsLanguagesPageBrowserTest to CrSettingsLanguagesPageTest (Closed)
Patch Set: cleanup Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/webui/settings/cr_settings_browsertest.js
diff --git a/chrome/test/data/webui/settings/cr_settings_browsertest.js b/chrome/test/data/webui/settings/cr_settings_browsertest.js
index bd9d622a14a96ccb674b3ada1cc70cf2085d514b..d93b41506c3bff20f85e48d5f3a74c8a583432ab 100644
--- a/chrome/test/data/webui/settings/cr_settings_browsertest.js
+++ b/chrome/test/data/webui/settings/cr_settings_browsertest.js
@@ -1087,10 +1087,12 @@ CrSettingsLanguagesTest.prototype = {
/** @override */
extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
'../fake_chrome_event.js',
+ 'test_util.js',
'test_browser_proxy.js',
'fake_language_settings_private.js',
'fake_settings_private.js',
'fake_input_method_private.js',
+ 'test_languages_browser_proxy.js',
'languages_tests.js',
]),
};
@@ -1104,6 +1106,47 @@ TEST_F('CrSettingsLanguagesTest', 'Languages', function() {
* @constructor
* @extends {CrSettingsBrowserTest}
*/
+function CrSettingsLanguagesPageTest() {}
+
+CrSettingsLanguagesPageTest.prototype = {
+ __proto__: CrSettingsBrowserTest.prototype,
+
+ /** @override */
+ browsePreload: 'chrome://md-settings/languages_page/languages_page.html',
+
+ /** @override */
+ extraLibraries: CrSettingsBrowserTest.prototype.extraLibraries.concat([
+ '../fake_chrome_event.js',
+ 'test_util.js',
+ 'test_browser_proxy.js',
+ 'fake_settings_private.js',
+ 'fake_language_settings_private.js',
+ 'fake_input_method_private.js',
+ 'test_languages_browser_proxy.js',
+ 'languages_page_tests.js',
+ ]),
+};
+
+TEST_F('CrSettingsLanguagesPageTest', 'AddLanguagesDialog', function() {
+ mocha.grep(assert(languages_page_tests.TestNames.AddLanguagesDialog)).run();
+});
+
+TEST_F('CrSettingsLanguagesPageTest', 'LanguageMenu', function() {
+ mocha.grep(assert(languages_page_tests.TestNames.LanguageMenu)).run();
+});
+
+TEST_F('CrSettingsLanguagesPageTest', 'InputMethods', function() {
+ mocha.grep(assert(languages_page_tests.TestNames.InputMethods)).run();
+});
+
+TEST_F('CrSettingsLanguagesPageTest', 'Spellcheck', function() {
+ mocha.grep(assert(languages_page_tests.TestNames.Spellcheck)).run();
+});
+
+/**
+ * @constructor
+ * @extends {CrSettingsBrowserTest}
+ */
function CrSettingsRouteTest() {}
CrSettingsRouteTest.prototype = {

Powered by Google App Engine
This is Rietveld 408576698