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

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

Issue 2957153003: MD Settings: remove unsupported routes from guest-mode. (Closed)
Patch Set: merge Created 3 years, 5 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/people_page_test.js
diff --git a/chrome/test/data/webui/settings/people_page_test.js b/chrome/test/data/webui/settings/people_page_test.js
index ca99ca1632070adb48ff22151add5274c58ada86..8a6e31076d5c68f16fa9f5a043d78ba6871db4be 100644
--- a/chrome/test/data/webui/settings/people_page_test.js
+++ b/chrome/test/data/webui/settings/people_page_test.js
@@ -281,7 +281,7 @@ cr.define('settings_people_page', function() {
test('NavigateDirectlyToSignOutURL', function() {
// Navigate to chrome://md-settings/signOut
- settings.navigateTo(settings.Route.SIGN_OUT);
+ settings.navigateTo(settings.routes.SIGN_OUT);
return new Promise(
function(resolve) { peoplePage.async(resolve); }).then(function() {
@@ -303,30 +303,35 @@ cr.define('settings_people_page', function() {
});
test('Signout dialog suppressed when not signed in', function() {
- return browserProxy.whenCalled('getSyncStatus').then(function() {
- settings.navigateTo(settings.Route.SIGN_OUT);
- return new Promise(function(resolve) { peoplePage.async(resolve); });
- }).then(function() {
- assertTrue(peoplePage.$$('#disconnectDialog').open);
+ return browserProxy.whenCalled('getSyncStatus')
+ .then(function() {
+ settings.navigateTo(settings.routes.SIGN_OUT);
+ return new Promise(function(resolve) {
+ peoplePage.async(resolve);
+ });
+ })
+ .then(function() {
+ assertTrue(peoplePage.$$('#disconnectDialog').open);
- var popstatePromise = new Promise(function(resolve) {
- listenOnce(window, 'popstate', resolve);
- });
+ var popstatePromise = new Promise(function(resolve) {
+ listenOnce(window, 'popstate', resolve);
+ });
- cr.webUIListenerCallback('sync-status-changed', {
- signedIn: false,
- });
+ cr.webUIListenerCallback('sync-status-changed', {
+ signedIn: false,
+ });
- return popstatePromise;
- }).then(function() {
- var popstatePromise = new Promise(function(resolve) {
- listenOnce(window, 'popstate', resolve);
- });
+ return popstatePromise;
+ })
+ .then(function() {
+ var popstatePromise = new Promise(function(resolve) {
+ listenOnce(window, 'popstate', resolve);
+ });
- settings.navigateTo(settings.Route.SIGN_OUT);
+ settings.navigateTo(settings.routes.SIGN_OUT);
- return popstatePromise;
- });
+ return popstatePromise;
+ });
});
test('syncStatusNotActionableForManagedAccounts', function() {
« no previous file with comments | « chrome/test/data/webui/settings/people_page_sync_page_test.js ('k') | chrome/test/data/webui/settings/reset_page_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698