| 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 fbd0d71a923b936d267691f3229d1173c728ab2e..911cfa438d15772d9bf01cc2ac590e8fb8c73353 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
|
| @@ -5,12 +5,18 @@
|
| package org.chromium.chrome.browser.signin;
|
|
|
| import android.accounts.Account;
|
| +import android.support.test.InstrumentationRegistry;
|
| import android.support.test.filters.SmallTest;
|
| -import android.test.InstrumentationTestCase;
|
| +
|
| +import org.junit.Assert;
|
| +import org.junit.Before;
|
| +import org.junit.Test;
|
| +import org.junit.runner.RunWith;
|
|
|
| import org.chromium.base.test.util.AdvancedMockContext;
|
| import org.chromium.base.test.util.DisabledTest;
|
| import org.chromium.base.test.util.Feature;
|
| +import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
|
| import org.chromium.components.signin.AccountManagerHelper;
|
| import org.chromium.components.signin.test.util.AccountHolder;
|
| import org.chromium.components.signin.test.util.MockAccountManager;
|
| @@ -19,18 +25,18 @@ import java.util.Arrays;
|
| import java.util.concurrent.TimeUnit;
|
|
|
| /** Tests for OAuth2TokenService. */
|
| -public class OAuth2TokenServiceTest extends InstrumentationTestCase {
|
| -
|
| +@RunWith(ChromeJUnit4ClassRunner.class)
|
| +public class OAuth2TokenServiceTest {
|
| private AdvancedMockContext mContext;
|
| private MockAccountManager mAccountManager;
|
|
|
| - @Override
|
| - protected void setUp() throws Exception {
|
| - super.setUp();
|
| -
|
| + @Before
|
| + public void setUp() throws Exception {
|
| // Mock out the account manager on the device.
|
| - mContext = new AdvancedMockContext(getInstrumentation().getTargetContext());
|
| - mAccountManager = new MockAccountManager(mContext, getInstrumentation().getContext());
|
| + mContext = new AdvancedMockContext(
|
| + InstrumentationRegistry.getInstrumentation().getTargetContext());
|
| + mAccountManager = new MockAccountManager(
|
| + mContext, InstrumentationRegistry.getInstrumentation().getContext());
|
| AccountManagerHelper.overrideAccountManagerHelperForTests(mContext, mAccountManager);
|
| }
|
|
|
| @@ -38,14 +44,16 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {
|
| * @SmallTest
|
| * @Feature({"Sync"})
|
| */
|
| + @Test
|
| @DisabledTest(message = "crbug.com/533417")
|
| public void testGetAccountsNoAccountsRegistered() {
|
| String[] accounts = OAuth2TokenService.getAccounts(mContext);
|
| - assertEquals("There should be no accounts registered", 0, accounts.length);
|
| + Assert.assertEquals("There should be no accounts registered", 0, accounts.length);
|
| }
|
|
|
| /*@SmallTest
|
| @Feature({"Sync"})*/
|
| + @Test
|
| @DisabledTest(message = "crbug.com/527852")
|
| public void testGetAccountsOneAccountRegistered() {
|
| Account account1 = AccountManagerHelper.createAccountFromName("foo@gmail.com");
|
| @@ -53,15 +61,16 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {
|
| mAccountManager.addAccountHolderExplicitly(accountHolder1);
|
|
|
| String[] sysAccounts = OAuth2TokenService.getSystemAccountNames(mContext);
|
| - assertEquals("There should be one registered account", 1, sysAccounts.length);
|
| - assertEquals("The account should be " + account1, account1.name, sysAccounts[0]);
|
| + Assert.assertEquals("There should be one registered account", 1, sysAccounts.length);
|
| + Assert.assertEquals("The account should be " + account1, account1.name, sysAccounts[0]);
|
|
|
| String[] accounts = OAuth2TokenService.getAccounts(mContext);
|
| - assertEquals("There should be zero registered account", 0, accounts.length);
|
| + Assert.assertEquals("There should be zero registered account", 0, accounts.length);
|
| }
|
|
|
| /*@SmallTest
|
| @Feature({"Sync"})*/
|
| + @Test
|
| @DisabledTest(message = "crbug.com/527852")
|
| public void testGetAccountsTwoAccountsRegistered() {
|
| Account account1 = AccountManagerHelper.createAccountFromName("foo@gmail.com");
|
| @@ -72,16 +81,17 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {
|
| mAccountManager.addAccountHolderExplicitly(accountHolder2);
|
|
|
| String[] sysAccounts = OAuth2TokenService.getSystemAccountNames(mContext);
|
| - assertEquals("There should be one registered account", 2, sysAccounts.length);
|
| - assertTrue("The list should contain " + account1,
|
| + Assert.assertEquals("There should be one registered account", 2, sysAccounts.length);
|
| + Assert.assertTrue("The list should contain " + account1,
|
| Arrays.asList(sysAccounts).contains(account1.name));
|
| - assertTrue("The list should contain " + account2,
|
| + Assert.assertTrue("The list should contain " + account2,
|
| Arrays.asList(sysAccounts).contains(account2.name));
|
|
|
| String[] accounts = OAuth2TokenService.getAccounts(mContext);
|
| - assertEquals("There should be zero registered account", 0, accounts.length);
|
| + Assert.assertEquals("There should be zero registered account", 0, accounts.length);
|
| }
|
|
|
| + @Test
|
| @DisabledTest(message = "crbug.com/568620")
|
| @SmallTest
|
| @Feature({"Sync"})
|
| @@ -93,6 +103,7 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {
|
|
|
| /*@SmallTest
|
| @Feature({"Sync"})*/
|
| + @Test
|
| @DisabledTest(message = "crbug.com/527852")
|
| public void testGetOAuth2AccessTokenWithTimeoutOnError() {
|
| String authToken = null;
|
| @@ -114,6 +125,6 @@ public class OAuth2TokenServiceTest extends InstrumentationTestCase {
|
|
|
| String accessToken = OAuth2TokenService.getOAuth2AccessTokenWithTimeout(
|
| mContext, account, scope, 5, TimeUnit.SECONDS);
|
| - assertEquals(expectedToken, accessToken);
|
| + Assert.assertEquals(expectedToken, accessToken);
|
| }
|
| }
|
|
|