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

Unified Diff: components/signin/core/browser/android/javatests/src/org/chromium/components/signin/test/AccountManagerHelperTest.java

Issue 2845113003: Rename MockAccountManager to FakeAccountManagerDelegate (Closed)
Patch Set: Addressed comments Created 3 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
Index: components/signin/core/browser/android/javatests/src/org/chromium/components/signin/test/AccountManagerHelperTest.java
diff --git a/components/signin/core/browser/android/javatests/src/org/chromium/components/signin/test/AccountManagerHelperTest.java b/components/signin/core/browser/android/javatests/src/org/chromium/components/signin/test/AccountManagerHelperTest.java
index 7efc0489f03fd87879e3cf8ab593635b74487f93..f0acd3f0ffc3c1556de8a89be604e3580a0c937c 100644
--- a/components/signin/core/browser/android/javatests/src/org/chromium/components/signin/test/AccountManagerHelperTest.java
+++ b/components/signin/core/browser/android/javatests/src/org/chromium/components/signin/test/AccountManagerHelperTest.java
@@ -11,14 +11,14 @@ import android.test.InstrumentationTestCase;
import org.chromium.components.signin.AccountManagerHelper;
import org.chromium.components.signin.test.util.AccountHolder;
-import org.chromium.components.signin.test.util.MockAccountManager;
+import org.chromium.components.signin.test.util.FakeAccountManagerDelegate;
import org.chromium.components.signin.test.util.SimpleFuture;
/**
* Test class for {@link AccountManagerHelper}.
*/
public class AccountManagerHelperTest extends InstrumentationTestCase {
- private MockAccountManager mAccountManager;
+ private FakeAccountManagerDelegate mDelegate;
private AccountManagerHelper mHelper;
@Override
@@ -26,8 +26,8 @@ public class AccountManagerHelperTest extends InstrumentationTestCase {
super.setUp();
Context context = getInstrumentation().getTargetContext();
- mAccountManager = new MockAccountManager(context, context);
- AccountManagerHelper.overrideAccountManagerHelperForTests(context, mAccountManager);
+ mDelegate = new FakeAccountManagerDelegate(context);
+ AccountManagerHelper.overrideAccountManagerHelperForTests(context, mDelegate);
mHelper = AccountManagerHelper.get();
}
@@ -62,7 +62,7 @@ public class AccountManagerHelperTest extends InstrumentationTestCase {
Account account = AccountManagerHelper.createAccountFromName(accountName);
AccountHolder.Builder accountHolder =
AccountHolder.builder(account).password(password).alwaysAccept(true);
- mAccountManager.addAccountHolderExplicitly(accountHolder.build());
+ mDelegate.addAccountHolderExplicitly(accountHolder.build());
return account;
}

Powered by Google App Engine
This is Rietveld 408576698