[DO NOT MERGE] Update BP "Not recognized" message with UX/UI spec
Change the string on the biometric prompt when fingerprint/face failed according to biometric enrollment status Fixes: 286993443 Fixes: 286967857 Test: Manual (see bug) Test: atest AuthControllerTest atest BiometricsServiceTest Change-Id: Ieaecf678d9498b065972340900ddb5afba97ac1e Merged-In: Ieaecf678d9498b065972340900ddb5afba97ac1e
This commit is contained in:
@@ -1049,8 +1049,16 @@ public class AuthController implements CoreStartable, CommandQueue.Callbacks,
|
||||
}
|
||||
|
||||
private String getNotRecognizedString(@Modality int modality) {
|
||||
return mContext.getString(modality == TYPE_FACE
|
||||
? R.string.biometric_face_not_recognized : R.string.biometric_not_recognized);
|
||||
final int messageRes;
|
||||
final int userId = mCurrentDialogArgs.argi1;
|
||||
if (isFaceAuthEnrolled(userId) && isFingerprintEnrolled(userId)) {
|
||||
messageRes = modality == TYPE_FACE
|
||||
? R.string.biometric_face_not_recognized
|
||||
: R.string.fingerprint_error_not_match;
|
||||
} else {
|
||||
messageRes = R.string.biometric_not_recognized;
|
||||
}
|
||||
return mContext.getString(messageRes);
|
||||
}
|
||||
|
||||
private String getErrorString(@Modality int modality, int error, int vendorCode) {
|
||||
|
||||
@@ -182,6 +182,10 @@ public class BiometricServiceTest {
|
||||
.thenReturn(ERROR_HW_UNAVAILABLE);
|
||||
when(mResources.getString(R.string.biometric_not_recognized))
|
||||
.thenReturn(ERROR_NOT_RECOGNIZED);
|
||||
when(mResources.getString(R.string.biometric_face_not_recognized))
|
||||
.thenReturn(ERROR_NOT_RECOGNIZED);
|
||||
when(mResources.getString(R.string.fingerprint_error_not_match))
|
||||
.thenReturn(ERROR_NOT_RECOGNIZED);
|
||||
when(mResources.getString(R.string.biometric_error_user_canceled))
|
||||
.thenReturn(ERROR_USER_CANCELED);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user