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

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

Issue 2617533003: MD Settings: change most checkboxes to toggles (leave dialogs alone) (Closed)
Patch Set: fix tests Created 3 years, 11 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/android_apps_page_test.js
diff --git a/chrome/test/data/webui/settings/android_apps_page_test.js b/chrome/test/data/webui/settings/android_apps_page_test.js
index 52d8bc41441296b80ffdf6d5a4c85252b8009f98..95a80c2afb185f8cf16eeabf951dc6cd0098b879 100644
--- a/chrome/test/data/webui/settings/android_apps_page_test.js
+++ b/chrome/test/data/webui/settings/android_apps_page_test.js
@@ -61,17 +61,17 @@ suite('AndroidAppsPageTests', function() {
cr.webUIListenerCallback(
'android-apps-info-update', {appReady: false});
Polymer.dom.flush();
- var checkbox = androidAppsPage.$$('#enabledCheckbox');
- assertTrue(!!checkbox);
- assertFalse(checkbox.disabled);
- assertFalse(checkbox.checked);
+ var control = androidAppsPage.$$('#enabled');
+ assertTrue(!!control);
+ assertFalse(control.disabled);
+ assertFalse(control.checked);
var managed = androidAppsPage.$$('#manageApps');
assertTrue(!!managed);
assertTrue(managed.hidden);
- MockInteractions.tap(checkbox.$.checkbox);
+ MockInteractions.tap(control.$.control);
Polymer.dom.flush();
- assertTrue(checkbox.checked);
+ assertTrue(control.checked);
});
});
@@ -88,15 +88,15 @@ suite('AndroidAppsPageTests', function() {
cr.webUIListenerCallback(
'android-apps-info-update', {appReady: true});
Polymer.dom.flush();
- var checkbox = androidAppsPage.$$('#enabledCheckbox');
- assertTrue(!!checkbox);
- assertFalse(checkbox.disabled);
- assertTrue(checkbox.checked);
+ var control = androidAppsPage.$$('#enabled');
+ assertTrue(!!control);
+ assertFalse(control.disabled);
+ assertTrue(control.checked);
var managed = androidAppsPage.$$('#manageApps');
assertTrue(!!managed);
assertFalse(managed.hidden);
- MockInteractions.tap(checkbox.$.checkbox);
+ MockInteractions.tap(control.$.control);
cr.webUIListenerCallback(
'android-apps-info-update', {appReady: false});
Polymer.dom.flush();
@@ -108,7 +108,7 @@ suite('AndroidAppsPageTests', function() {
assertTrue(!!actionButton);
MockInteractions.tap(actionButton);
Polymer.dom.flush();
- assertFalse(checkbox.checked);
+ assertFalse(control.checked);
assertTrue(managed.hidden);
});
});

Powered by Google App Engine
This is Rietveld 408576698