diff --git a/src/com/android/settings/accounts/AvatarViewMixin.java b/src/com/android/settings/accounts/AvatarViewMixin.java index 9e762c72574..7eb8cab8f41 100644 --- a/src/com/android/settings/accounts/AvatarViewMixin.java +++ b/src/com/android/settings/accounts/AvatarViewMixin.java @@ -67,7 +67,8 @@ public class AvatarViewMixin implements LifecycleObserver { private final MutableLiveData mAvatarImage; private final ActivityManager mActivityManager; - private String mAccountName; + @VisibleForTesting + String mAccountName; public AvatarViewMixin(SettingsHomepageActivity activity, ImageView avatarView) { mContext = activity.getApplicationContext(); @@ -127,6 +128,7 @@ public class AvatarViewMixin implements LifecycleObserver { if (hasAccount()) { loadAccount(); } else { + mAccountName = null; mAvatarView.setImageResource(R.drawable.ic_account_circle_24dp); } } diff --git a/tests/robotests/src/com/android/settings/accounts/AvatarViewMixinTest.java b/tests/robotests/src/com/android/settings/accounts/AvatarViewMixinTest.java index 8c75e33b397..8dbf2a83cc5 100644 --- a/tests/robotests/src/com/android/settings/accounts/AvatarViewMixinTest.java +++ b/tests/robotests/src/com/android/settings/accounts/AvatarViewMixinTest.java @@ -130,6 +130,17 @@ public class AvatarViewMixinTest { verify(mockAvatar).hasAccount(); } + @Test + @Config(qualifiers = "mcc999") + public void onStart_noAccount_mAccountNameShouldBeNull() { + final AvatarViewMixin avatarViewMixin = new AvatarViewMixin(mActivity, mImageView); + avatarViewMixin.mAccountName = DUMMY_ACCOUNT; + + avatarViewMixin.onStart(); + + assertThat(avatarViewMixin.mAccountName).isNull(); + } + @Test public void queryProviderAuthority_useShadowPackagteManager_returnNull() { final AvatarViewMixin avatarViewMixin = new AvatarViewMixin(mActivity, mImageView);