chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java - Issue 213823004: Calls FireRefreshTokenRevoked if an account is removed.

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java

Issue 213823004: Calls FireRefreshTokenRevoked if an account is removed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: clean up Created 6 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

« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/signin/OAuth2TokenService.java ('k') | chrome/browser/android/signin/signin_manager_android.cc » ('j') | chrome/browser/signin/android_profile_oauth2_token_service.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java

diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java

index 78baa8496d1134b19d6d060afcb4f6d8cd9ba7f2..459bd3f6a480b0080bb4788e2f14a8d7ef895145 100644

--- a/chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java

+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/signin/OAuth2TokenServiceTest.java

@@ -46,9 +46,12 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {

AccountHolder accountHolder1 = AccountHolder.create().account(account1).build();

mAccountManager.addAccountHolderExplicitly(accountHolder1);

+ String[] sysAccounts = OAuth2TokenService.getSystemAccounts(mContext);

+ assertEquals("There should be one registered account", 1, sysAccounts.length);

+ assertEquals("The account should be " + account1, account1.name, sysAccounts[0]);

String[] accounts = OAuth2TokenService.getAccounts(mContext);

- assertEquals("There should be one registered account", 1, accounts.length);

- assertEquals("The account should be " + account1, account1.name, accounts[0]);

+ assertEquals("There should be one registered account", 0, accounts.length);

Roger Tawa OOO till Jul 10th 2014/04/23 15:03:59 nit: assert string should zero, not one.

acleung1 2014/04/24 06:59:15 Done.

}

@SmallTest

@@ -61,12 +64,15 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {

AccountHolder accountHolder2 = AccountHolder.create().account(account2).build();

mAccountManager.addAccountHolderExplicitly(accountHolder2);

- String[] accounts = OAuth2TokenService.getAccounts(mContext);

- assertEquals("There should be one registered account", 2, accounts.length);

+ String[] sysAccounts = OAuth2TokenService.getSystemAccounts(mContext);

+ assertEquals("There should be one registered account", 2, sysAccounts.length);

assertTrue("The list should contain " + account1,

- Arrays.asList(accounts).contains(account1.name));

+ Arrays.asList(sysAccounts).contains(account1.name));

assertTrue("The list should contain " + account2,

- Arrays.asList(accounts).contains(account2.name));

+ Arrays.asList(sysAccounts).contains(account2.name));

+ String[] accounts = OAuth2TokenService.getAccounts(mContext);

+ assertEquals("There should be one registered account", 0, accounts.length);

Roger Tawa OOO till Jul 10th 2014/04/23 15:03:59 nit: assert string should zero, not one.

acleung1 2014/04/24 06:59:15 Done.

}

@SmallTest