Index: chrome/android/junit/src/org/chromium/chrome/browser/superviseduser/SupervisedUserContentProviderUnitTest.java |
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/superviseduser/SupervisedUserContentProviderUnitTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/superviseduser/SupervisedUserContentProviderUnitTest.java |
index 3698d0066530c6c5369e4e000f192217af6cb65c..0fd8bddba7d30391c0cee3b682bd2244d893f83a 100644 |
--- a/chrome/android/junit/src/org/chromium/chrome/browser/superviseduser/SupervisedUserContentProviderUnitTest.java |
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/superviseduser/SupervisedUserContentProviderUnitTest.java |
@@ -21,6 +21,7 @@ import android.content.pm.ProviderInfo; |
import org.junit.After; |
import org.junit.Before; |
+import org.junit.BeforeClass; |
import org.junit.Rule; |
import org.junit.Test; |
import org.junit.runner.RunWith; |
@@ -57,6 +58,14 @@ public class SupervisedUserContentProviderUnitTest { |
private static final String DEFAULT_CALLING_PACKAGE = "com.example.some.app"; |
+ private static AccountManagerDelegate sMockDelegate; |
+ |
+ @BeforeClass |
+ public static void overrideAccountManagerHelper() { |
+ sMockDelegate = mock(AccountManagerDelegate.class); |
+ AccountManagerHelper.overrideAccountManagerHelperForTests(sMockDelegate); |
+ } |
+ |
// Override methods that wrap things that can't be mocked (including native calls). |
private static class MySupervisedUserContentProvider extends SupervisedUserContentProvider { |
@Override |
@@ -218,11 +227,8 @@ public class SupervisedUserContentProviderUnitTest { |
// Mock things called during startup |
ChromeBrowserInitializer mockBrowserInitializer = mock(ChromeBrowserInitializer.class); |
ChromeBrowserInitializer.setForTesting(mockBrowserInitializer); |
- AccountManagerDelegate mockDelegate = mock(AccountManagerDelegate.class); |
- AccountManagerHelper.overrideAccountManagerHelperForTests( |
- RuntimeEnvironment.application, mockDelegate); |
Account account = new Account("Google", "Dummy"); |
- when(mockDelegate.getAccountsByType("Google")).thenReturn(new Account[] {account}); |
+ when(sMockDelegate.getAccountsByType("Google")).thenReturn(new Account[] {account}); |
WebRestrictionsResult result = |
mSupervisedUserContentProvider.shouldProceed(DEFAULT_CALLING_PACKAGE, "url"); |
@@ -237,8 +243,6 @@ public class SupervisedUserContentProviderUnitTest { |
.nativeShouldProceed(eq(5678L), |
any(SupervisedUserContentProvider.SupervisedUserQueryReply.class), |
eq("url")); |
- |
- AccountManagerHelper.resetAccountManagerHelperForTests(); |
} |
@Test |
@@ -246,11 +250,8 @@ public class SupervisedUserContentProviderUnitTest { |
// Mock things called during startup |
ChromeBrowserInitializer mockBrowserInitializer = mock(ChromeBrowserInitializer.class); |
ChromeBrowserInitializer.setForTesting(mockBrowserInitializer); |
- AccountManagerDelegate mockDelegate = mock(AccountManagerDelegate.class); |
- AccountManagerHelper.overrideAccountManagerHelperForTests( |
- RuntimeEnvironment.application, mockDelegate); |
Account account = new Account("Google", "Dummy"); |
- when(mockDelegate.getAccountsByType("Google")).thenReturn(new Account[] {account}); |
+ when(sMockDelegate.getAccountsByType("Google")).thenReturn(new Account[] {account}); |
// Change the behavior of the forced sign-in processor to not sign in. |
doAnswer(new Answer<Void>() { |
@@ -268,8 +269,6 @@ public class SupervisedUserContentProviderUnitTest { |
assertThat(result.shouldProceed(), is(false)); |
assertThat(result.getErrorInt(0), is(5)); |
- |
- AccountManagerHelper.resetAccountManagerHelperForTests(); |
} |
@Test |