Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Prevent multiple calls to handleOnDialogAnimatedIn." into tm-dev am: b46b98ee48 am: a7f9256be7

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/17509982

Change-Id: Ie757b2f47f6e5d213053ae49691304838b8633ae
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Joe Bolinger
2022-04-04 18:27:14 +00:00
committed by Automerger Merge Worker
parent 4cd454ebe4 a7f9256be7
commit 9a859c8b14
12 changed files with 112 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/java/com/android/internal/statusbar/IStatusBar.aidl
View File

@@ -169,7 +169,7 @@ oneway interface IStatusBar
/** /**
* Used to hide the authentication dialog, e.g. when the application cancels authentication. * Used to hide the authentication dialog, e.g. when the application cancels authentication.
*/ */
void hideAuthenticationDialog(); void hideAuthenticationDialog(long requestId);
/* Used to notify the biometric service of events that occur outside of an operation. */ /* Used to notify the biometric service of events that occur outside of an operation. */
void setBiometicContextListener(in IBiometricContextListener listener); void setBiometicContextListener(in IBiometricContextListener listener);

2
core/java/com/android/internal/statusbar/IStatusBarService.aidl
View File

@@ -131,7 +131,7 @@ interface IStatusBarService
// Used to show an error - the dialog will dismiss after a certain amount of time // Used to show an error - the dialog will dismiss after a certain amount of time
void onBiometricError(int modality, int error, int vendorCode); void onBiometricError(int modality, int error, int vendorCode);
// Used to hide the authentication dialog, e.g. when the application cancels authentication // Used to hide the authentication dialog, e.g. when the application cancels authentication
void hideAuthenticationDialog(); void hideAuthenticationDialog(long requestId);
// Used to notify the biometric service of events that occur outside of an operation. // Used to notify the biometric service of events that occur outside of an operation.
void setBiometicContextListener(in IBiometricContextListener listener); void setBiometicContextListener(in IBiometricContextListener listener);

16
packages/SystemUI/src/com/android/systemui/biometrics/AuthContainerView.java
View File

@@ -126,7 +126,7 @@ public class AuthContainerView extends LinearLayout
int[] mSensorIds; int[] mSensorIds;
boolean mSkipIntro; boolean mSkipIntro;
long mOperationId; long mOperationId;
long mRequestId; long mRequestId = -1;
boolean mSkipAnimation = false; boolean mSkipAnimation = false;
@BiometricMultiSensorMode int mMultiSensorConfig = BIOMETRIC_MULTI_SENSOR_DEFAULT; @BiometricMultiSensorMode int mMultiSensorConfig = BIOMETRIC_MULTI_SENSOR_DEFAULT;
} }
@@ -598,6 +598,11 @@ public class AuthContainerView extends LinearLayout
return mConfig.mOpPackageName; return mConfig.mOpPackageName;
} }
@Override
public long getRequestId() {
return mConfig.mRequestId;
}
@Override @Override
public void animateToCredentialUI() { public void animateToCredentialUI() {
mBiometricView.startTransitionToCredentialUI(); mBiometricView.startTransitionToCredentialUI();
@@ -678,7 +683,9 @@ public class AuthContainerView extends LinearLayout
return; return;
} }
mContainerState = STATE_GONE; mContainerState = STATE_GONE;
mWindowManager.removeView(this); if (isAttachedToWindow()) {
mWindowManager.removeView(this);
}
} }
private void onDialogAnimatedIn() { private void onDialogAnimatedIn() {
@@ -687,6 +694,11 @@ public class AuthContainerView extends LinearLayout
animateAway(AuthDialogCallback.DISMISSED_USER_CANCELED); animateAway(AuthDialogCallback.DISMISSED_USER_CANCELED);
return; return;
} }
if (mContainerState == STATE_ANIMATING_OUT || mContainerState == STATE_GONE) {
Log.d(TAG, "onDialogAnimatedIn(): ignore, already animating out or gone - state: "
+ mContainerState);
return;
}
mContainerState = STATE_SHOWING; mContainerState = STATE_SHOWING;
if (mBiometricView != null) { if (mBiometricView != null) {
mConfig.mCallback.onDialogAnimatedIn(); mConfig.mCallback.onDialogAnimatedIn();

7
packages/SystemUI/src/com/android/systemui/biometrics/AuthController.java
View File

@@ -768,7 +768,7 @@ public class AuthController extends CoreStartable implements CommandQueue.Callba
} }
@Override @Override
public void hideAuthenticationDialog() { public void hideAuthenticationDialog(long requestId) {
if (DEBUG) Log.d(TAG, "hideAuthenticationDialog: " + mCurrentDialog); if (DEBUG) Log.d(TAG, "hideAuthenticationDialog: " + mCurrentDialog);
if (mCurrentDialog == null) { if (mCurrentDialog == null) {
@@ -777,6 +777,11 @@ public class AuthController extends CoreStartable implements CommandQueue.Callba
if (DEBUG) Log.d(TAG, "dialog already gone"); if (DEBUG) Log.d(TAG, "dialog already gone");
return; return;
} }
if (requestId != mCurrentDialog.getRequestId()) {
Log.w(TAG, "ignore - ids do not match: " + requestId + " current: "
+ mCurrentDialog.getRequestId());
return;
}
mCurrentDialog.dismissFromSystemServer(); mCurrentDialog.dismissFromSystemServer();

3
packages/SystemUI/src/com/android/systemui/biometrics/AuthDialog.java
View File

@@ -150,6 +150,9 @@ public interface AuthDialog {
*/ */
String getOpPackageName(); String getOpPackageName();
/** The requestId of the underlying operation within the framework. */
long getRequestId();
/** /**
* Animate to credential UI. Typically called after biometric is locked out. * Animate to credential UI. Typically called after biometric is locked out.
*/ */

15
packages/SystemUI/src/com/android/systemui/statusbar/CommandQueue.java
View File

@@ -323,7 +323,7 @@ public class CommandQueue extends IStatusBar.Stub implements
default void onBiometricError(@Modality int modality, int error, int vendorCode) { default void onBiometricError(@Modality int modality, int error, int vendorCode) {
} }
default void hideAuthenticationDialog() { default void hideAuthenticationDialog(long requestId) {
} }
/** /**
@@ -999,9 +999,11 @@ public class CommandQueue extends IStatusBar.Stub implements
} }
@Override @Override
public void hideAuthenticationDialog() { public void hideAuthenticationDialog(long requestId) {
synchronized (mLock) { synchronized (mLock) {
mHandler.obtainMessage(MSG_BIOMETRIC_HIDE).sendToTarget(); final SomeArgs args = SomeArgs.obtain();
args.argl1 = requestId;
mHandler.obtainMessage(MSG_BIOMETRIC_HIDE, args).sendToTarget();
} }
} }
@@ -1508,11 +1510,14 @@ public class CommandQueue extends IStatusBar.Stub implements
someArgs.recycle(); someArgs.recycle();
break; break;
} }
case MSG_BIOMETRIC_HIDE: case MSG_BIOMETRIC_HIDE: {
final SomeArgs someArgs = (SomeArgs) msg.obj;
for (int i = 0; i < mCallbacks.size(); i++) { for (int i = 0; i < mCallbacks.size(); i++) {
mCallbacks.get(i).hideAuthenticationDialog(); mCallbacks.get(i).hideAuthenticationDialog(someArgs.argl1 /* requestId */);
} }
someArgs.recycle();
break; break;
}
case MSG_SET_BIOMETRICS_LISTENER: case MSG_SET_BIOMETRICS_LISTENER:
for (int i = 0; i < mCallbacks.size(); i++) { for (int i = 0; i < mCallbacks.size(); i++) {
mCallbacks.get(i).setBiometicContextListener( mCallbacks.get(i).setBiometicContextListener(

49
packages/SystemUI/tests/src/com/android/systemui/biometrics/AuthContainerViewTest.kt
View File

@@ -48,6 +48,7 @@ import org.junit.runner.RunWith
import org.mockito.Mock import org.mockito.Mock
import org.mockito.Mockito.anyInt import org.mockito.Mockito.anyInt
import org.mockito.Mockito.eq import org.mockito.Mockito.eq
import org.mockito.Mockito.never
import org.mockito.Mockito.verify import org.mockito.Mockito.verify
import org.mockito.junit.MockitoJUnit import org.mockito.junit.MockitoJUnit
import org.mockito.Mockito.`when` as whenever import org.mockito.Mockito.`when` as whenever
@@ -80,9 +81,30 @@ class AuthContainerViewTest : SysuiTestCase() {
} }
} }
@Test
fun testNotifiesAnimatedIn() {
initializeContainer()
verify(callback).onDialogAnimatedIn()
}
@Test
fun testIgnoresAnimatedInWhenDismissed() {
val container = initializeContainer(addToView = false)
container.dismissFromSystemServer()
waitForIdleSync()
verify(callback, never()).onDialogAnimatedIn()
container.addToView()
waitForIdleSync()
// attaching the view resets the state and allows this to happen again
verify(callback).onDialogAnimatedIn()
}
@Test @Test
fun testActionAuthenticated_sendsDismissedAuthenticated() { fun testActionAuthenticated_sendsDismissedAuthenticated() {
val container = initializeContainer(BiometricManager.Authenticators.BIOMETRIC_WEAK) val container = initializeContainer()
container.mBiometricCallback.onAction( container.mBiometricCallback.onAction(
AuthBiometricView.Callback.ACTION_AUTHENTICATED AuthBiometricView.Callback.ACTION_AUTHENTICATED
) )
@@ -97,7 +119,7 @@ class AuthContainerViewTest : SysuiTestCase() {
@Test @Test
fun testActionUserCanceled_sendsDismissedUserCanceled() { fun testActionUserCanceled_sendsDismissedUserCanceled() {
val container = initializeContainer(BiometricManager.Authenticators.BIOMETRIC_WEAK) val container = initializeContainer()
container.mBiometricCallback.onAction( container.mBiometricCallback.onAction(
AuthBiometricView.Callback.ACTION_USER_CANCELED AuthBiometricView.Callback.ACTION_USER_CANCELED
) )
@@ -115,7 +137,7 @@ class AuthContainerViewTest : SysuiTestCase() {
@Test @Test
fun testActionButtonNegative_sendsDismissedButtonNegative() { fun testActionButtonNegative_sendsDismissedButtonNegative() {
val container = initializeContainer(BiometricManager.Authenticators.BIOMETRIC_WEAK) val container = initializeContainer()
container.mBiometricCallback.onAction( container.mBiometricCallback.onAction(
AuthBiometricView.Callback.ACTION_BUTTON_NEGATIVE AuthBiometricView.Callback.ACTION_BUTTON_NEGATIVE
) )
@@ -141,7 +163,7 @@ class AuthContainerViewTest : SysuiTestCase() {
@Test @Test
fun testActionError_sendsDismissedError() { fun testActionError_sendsDismissedError() {
val container = initializeContainer(BiometricManager.Authenticators.BIOMETRIC_WEAK) val container = initializeContainer()
authContainer!!.mBiometricCallback.onAction( authContainer!!.mBiometricCallback.onAction(
AuthBiometricView.Callback.ACTION_ERROR AuthBiometricView.Callback.ACTION_ERROR
) )
@@ -183,7 +205,7 @@ class AuthContainerViewTest : SysuiTestCase() {
@Test @Test
fun testShowBiometricUI() { fun testShowBiometricUI() {
val container = initializeContainer(BiometricManager.Authenticators.BIOMETRIC_WEAK) val container = initializeContainer()
waitForIdleSync() waitForIdleSync()
@@ -252,7 +274,10 @@ class AuthContainerViewTest : SysuiTestCase() {
assertThat((layoutParams.fitInsetsTypes and WindowInsets.Type.ime()) == 0).isTrue() assertThat((layoutParams.fitInsetsTypes and WindowInsets.Type.ime()) == 0).isTrue()
} }
private fun initializeContainer(authenticators: Int): TestAuthContainerView { private fun initializeContainer(
authenticators: Int = BiometricManager.Authenticators.BIOMETRIC_WEAK,
addToView: Boolean = true
): TestAuthContainerView {
val config = AuthContainerView.Config() val config = AuthContainerView.Config()
config.mContext = mContext config.mContext = mContext
config.mCallback = callback config.mCallback = callback
@@ -291,7 +316,11 @@ class AuthContainerViewTest : SysuiTestCase() {
lockPatternUtils, lockPatternUtils,
Handler(TestableLooper.get(this).looper) Handler(TestableLooper.get(this).looper)
) )
ViewUtils.attachView(authContainer)
if (addToView) {
authContainer!!.addToView()
}
return authContainer!! return authContainer!!
} }
@@ -316,6 +345,12 @@ class AuthContainerViewTest : SysuiTestCase() {
TestableLooper.get(this).processAllMessages() TestableLooper.get(this).processAllMessages()
super.waitForIdleSync() super.waitForIdleSync()
} }
private fun AuthContainerView.addToView() {
ViewUtils.attachView(this)
waitForIdleSync()
assertThat(isAttachedToWindow).isTrue()
}
} }
private fun AuthContainerView.hasBiometricPrompt() = private fun AuthContainerView.hasBiometricPrompt() =

20
packages/SystemUI/tests/src/com/android/systemui/biometrics/AuthControllerTest.java
View File

@@ -20,6 +20,8 @@ import static android.hardware.biometrics.BiometricAuthenticator.TYPE_FINGERPRIN
import static android.hardware.biometrics.BiometricManager.Authenticators; import static android.hardware.biometrics.BiometricManager.Authenticators;
import static android.hardware.biometrics.BiometricManager.BIOMETRIC_MULTI_SENSOR_FINGERPRINT_AND_FACE; import static android.hardware.biometrics.BiometricManager.BIOMETRIC_MULTI_SENSOR_FINGERPRINT_AND_FACE;
import static com.google.common.truth.Truth.assertThat;
import static junit.framework.Assert.assertEquals; import static junit.framework.Assert.assertEquals;
import static junit.framework.Assert.assertNull; import static junit.framework.Assert.assertNull;
@@ -104,6 +106,8 @@ import javax.inject.Provider;
@SmallTest @SmallTest
public class AuthControllerTest extends SysuiTestCase { public class AuthControllerTest extends SysuiTestCase {
private static final long REQUEST_ID = 22;
@Rule @Rule
public final MockitoRule mMockitoRule = MockitoJUnit.rule(); public final MockitoRule mMockitoRule = MockitoJUnit.rule();
@@ -173,6 +177,9 @@ public class AuthControllerTest extends SysuiTestCase {
when(mDialog1.isAllowDeviceCredentials()).thenReturn(false); when(mDialog1.isAllowDeviceCredentials()).thenReturn(false);
when(mDialog2.isAllowDeviceCredentials()).thenReturn(false); when(mDialog2.isAllowDeviceCredentials()).thenReturn(false);
when(mDialog1.getRequestId()).thenReturn(REQUEST_ID);
when(mDialog2.getRequestId()).thenReturn(REQUEST_ID);
when(mFingerprintManager.isHardwareDetected()).thenReturn(true); when(mFingerprintManager.isHardwareDetected()).thenReturn(true);
final List<ComponentInfoInternal> componentInfo = new ArrayList<>(); final List<ComponentInfoInternal> componentInfo = new ArrayList<>();
@@ -482,7 +489,12 @@ public class AuthControllerTest extends SysuiTestCase {
@Test @Test
public void testHideAuthenticationDialog_invokesDismissFromSystemServer() { public void testHideAuthenticationDialog_invokesDismissFromSystemServer() {
showDialog(new int[] {1} /* sensorIds */, false /* credentialAllowed */); showDialog(new int[] {1} /* sensorIds */, false /* credentialAllowed */);
mAuthController.hideAuthenticationDialog();
mAuthController.hideAuthenticationDialog(REQUEST_ID + 1);
verify(mDialog1, never()).dismissFromSystemServer();
assertThat(mAuthController.mCurrentDialog).isSameInstanceAs(mDialog1);
mAuthController.hideAuthenticationDialog(REQUEST_ID);
verify(mDialog1).dismissFromSystemServer(); verify(mDialog1).dismissFromSystemServer();
// In this case, BiometricService sends the error to the client immediately, without // In this case, BiometricService sends the error to the client immediately, without
@@ -512,7 +524,7 @@ public class AuthControllerTest extends SysuiTestCase {
eq(BiometricPrompt.DISMISSED_REASON_CREDENTIAL_CONFIRMED), eq(BiometricPrompt.DISMISSED_REASON_CREDENTIAL_CONFIRMED),
AdditionalMatchers.aryEq(credentialAttestation)); AdditionalMatchers.aryEq(credentialAttestation));
mAuthController.hideAuthenticationDialog(); mAuthController.hideAuthenticationDialog(REQUEST_ID);
} }
@Test @Test
@@ -648,7 +660,7 @@ public class AuthControllerTest extends SysuiTestCase {
verify(mDisplayManager).registerDisplayListener(any(), eq(mHandler)); verify(mDisplayManager).registerDisplayListener(any(), eq(mHandler));
mAuthController.hideAuthenticationDialog(); mAuthController.hideAuthenticationDialog(REQUEST_ID);
verify(mDisplayManager).unregisterDisplayListener(any()); verify(mDisplayManager).unregisterDisplayListener(any());
} }
@@ -704,7 +716,7 @@ public class AuthControllerTest extends SysuiTestCase {
0 /* userId */, 0 /* userId */,
0 /* operationId */, 0 /* operationId */,
"testPackage", "testPackage",
1 /* requestId */, REQUEST_ID,
BIOMETRIC_MULTI_SENSOR_FINGERPRINT_AND_FACE); BIOMETRIC_MULTI_SENSOR_FINGERPRINT_AND_FACE);
} }

5
packages/SystemUI/tests/src/com/android/systemui/statusbar/CommandQueueTest.java
View File

@@ -475,9 +475,10 @@ public class CommandQueueTest extends SysuiTestCase {
@Test @Test
public void testHideAuthenticationDialog() { public void testHideAuthenticationDialog() {
mCommandQueue.hideAuthenticationDialog(); final long id = 4;
mCommandQueue.hideAuthenticationDialog(id);
waitForIdleSync(); waitForIdleSync();
verify(mCallbacks).hideAuthenticationDialog(); verify(mCallbacks).hideAuthenticationDialog(eq(id));
} }
@Test @Test

10
services/core/java/com/android/server/biometrics/AuthSession.java
View File

@@ -462,7 +462,7 @@ public final class AuthSession implements IBinder.DeathRecipient {
mState = STATE_SHOWING_DEVICE_CREDENTIAL; mState = STATE_SHOWING_DEVICE_CREDENTIAL;
mStatusBarService.onBiometricError(modality, error, vendorCode); mStatusBarService.onBiometricError(modality, error, vendorCode);
} else if (error == BiometricConstants.BIOMETRIC_ERROR_CANCELED) { } else if (error == BiometricConstants.BIOMETRIC_ERROR_CANCELED) {
mStatusBarService.hideAuthenticationDialog(); mStatusBarService.hideAuthenticationDialog(mRequestId);
// TODO: If multiple authenticators are simultaneously running, this will // TODO: If multiple authenticators are simultaneously running, this will
// need to be modified. Send the error to the client here, instead of doing // need to be modified. Send the error to the client here, instead of doing
// a round trip to SystemUI. // a round trip to SystemUI.
@@ -480,7 +480,7 @@ public final class AuthSession implements IBinder.DeathRecipient {
// the client and clean up. The only error we should get here is // the client and clean up. The only error we should get here is
// ERROR_CANCELED due to another client kicking us out. // ERROR_CANCELED due to another client kicking us out.
mClientReceiver.onError(modality, error, vendorCode); mClientReceiver.onError(modality, error, vendorCode);
mStatusBarService.hideAuthenticationDialog(); mStatusBarService.hideAuthenticationDialog(mRequestId);
return true; return true;
} }
@@ -489,7 +489,7 @@ public final class AuthSession implements IBinder.DeathRecipient {
break; break;
case STATE_CLIENT_DIED_CANCELLING: case STATE_CLIENT_DIED_CANCELLING:
mStatusBarService.hideAuthenticationDialog(); mStatusBarService.hideAuthenticationDialog(mRequestId);
return true; return true;
default: default:
@@ -665,7 +665,7 @@ public final class AuthSession implements IBinder.DeathRecipient {
cancelAllSensors(); cancelAllSensors();
return false; return false;
default: default:
mStatusBarService.hideAuthenticationDialog(); mStatusBarService.hideAuthenticationDialog(mRequestId);
return true; return true;
} }
} catch (RemoteException e) { } catch (RemoteException e) {
@@ -832,7 +832,7 @@ public final class AuthSession implements IBinder.DeathRecipient {
BiometricConstants.BIOMETRIC_ERROR_CANCELED, BiometricConstants.BIOMETRIC_ERROR_CANCELED,
0 /* vendorCode */ 0 /* vendorCode */
); );
mStatusBarService.hideAuthenticationDialog(); mStatusBarService.hideAuthenticationDialog(mRequestId);
return true; return true;
} catch (RemoteException e) { } catch (RemoteException e) {
Slog.e(TAG, "Remote exception", e); Slog.e(TAG, "Remote exception", e);

4
services/core/java/com/android/server/statusbar/StatusBarManagerService.java
View File

@@ -903,11 +903,11 @@ public class StatusBarManagerService extends IStatusBarService.Stub implements D
} }
@Override @Override
public void hideAuthenticationDialog() { public void hideAuthenticationDialog(long requestId) {
enforceBiometricDialog(); enforceBiometricDialog();
if (mBar != null) { if (mBar != null) {
try { try {
mBar.hideAuthenticationDialog(); mBar.hideAuthenticationDialog(requestId);
} catch (RemoteException ex) { } catch (RemoteException ex) {
} }
} }

16
services/tests/servicestests/src/com/android/server/biometrics/BiometricServiceTest.java
View File

@@ -192,7 +192,7 @@ public class BiometricServiceTest {
waitForIdle(); waitForIdle();
assertNull(mBiometricService.mAuthSession); assertNull(mBiometricService.mAuthSession);
verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(eq(TEST_REQUEST_ID));
verify(mReceiver1, never()).onError(anyInt(), anyInt(), anyInt()); verify(mReceiver1, never()).onError(anyInt(), anyInt(), anyInt());
} }
@@ -211,7 +211,8 @@ public class BiometricServiceTest {
waitForIdle(); waitForIdle();
assertNotNull(mBiometricService.mAuthSession); assertNotNull(mBiometricService.mAuthSession);
verify(mBiometricService.mStatusBarService, never()).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService, never())
.hideAuthenticationDialog(eq(TEST_REQUEST_ID));
assertEquals(STATE_CLIENT_DIED_CANCELLING, assertEquals(STATE_CLIENT_DIED_CANCELLING,
mBiometricService.mAuthSession.getState()); mBiometricService.mAuthSession.getState());
@@ -225,7 +226,7 @@ public class BiometricServiceTest {
BiometricConstants.BIOMETRIC_ERROR_CANCELED, BiometricConstants.BIOMETRIC_ERROR_CANCELED,
0 /* vendorCode */); 0 /* vendorCode */);
waitForIdle(); waitForIdle();
verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(eq(TEST_REQUEST_ID));
verify(mReceiver1, never()).onError(anyInt(), anyInt(), anyInt()); verify(mReceiver1, never()).onError(anyInt(), anyInt(), anyInt());
assertNull(mBiometricService.mAuthSession); assertNull(mBiometricService.mAuthSession);
} }
@@ -666,7 +667,7 @@ public class BiometricServiceTest {
eq(BiometricAuthenticator.TYPE_FACE), eq(BiometricAuthenticator.TYPE_FACE),
eq(BiometricPrompt.BIOMETRIC_ERROR_CANCELED), eq(BiometricPrompt.BIOMETRIC_ERROR_CANCELED),
eq(0) /* vendorCode */); eq(0) /* vendorCode */);
verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(eq(TEST_REQUEST_ID));
verify(mReceiver2, never()).onError(anyInt(), anyInt(), anyInt()); verify(mReceiver2, never()).onError(anyInt(), anyInt(), anyInt());
} }
@@ -745,7 +746,7 @@ public class BiometricServiceTest {
eq(BiometricConstants.BIOMETRIC_ERROR_CANCELED), eq(BiometricConstants.BIOMETRIC_ERROR_CANCELED),
eq(0 /* vendorCode */)); eq(0 /* vendorCode */));
// Dialog is hidden immediately // Dialog is hidden immediately
verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(eq(TEST_REQUEST_ID));
// Auth session is over // Auth session is over
assertNull(mBiometricService.mAuthSession); assertNull(mBiometricService.mAuthSession);
} }
@@ -773,7 +774,8 @@ public class BiometricServiceTest {
eq(TYPE_FINGERPRINT), eq(TYPE_FINGERPRINT),
eq(BiometricConstants.BIOMETRIC_ERROR_UNABLE_TO_PROCESS), eq(BiometricConstants.BIOMETRIC_ERROR_UNABLE_TO_PROCESS),
eq(0 /* vendorCode */)); eq(0 /* vendorCode */));
verify(mBiometricService.mStatusBarService, never()).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService, never())
.hideAuthenticationDialog(eq(TEST_REQUEST_ID));
verify(mReceiver1, never()).onError(anyInt(), anyInt(), anyInt()); verify(mReceiver1, never()).onError(anyInt(), anyInt(), anyInt());
// SystemUI animation completed, client is notified, auth session is over // SystemUI animation completed, client is notified, auth session is over
@@ -1152,7 +1154,7 @@ public class BiometricServiceTest {
verify(mReceiver1).onError(eq(TYPE_FINGERPRINT), verify(mReceiver1).onError(eq(TYPE_FINGERPRINT),
eq(BiometricConstants.BIOMETRIC_ERROR_CANCELED), eq(BiometricConstants.BIOMETRIC_ERROR_CANCELED),
eq(0 /* vendorCode */)); eq(0 /* vendorCode */));
verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(); verify(mBiometricService.mStatusBarService).hideAuthenticationDialog(eq(TEST_REQUEST_ID));
} }
@Test @Test