From 5508e7661c09446b472e2453b56228bdc0d4b2a5 Mon Sep 17 00:00:00 2001 From: Hao Kung Date: Fri, 15 May 2020 14:22:38 -0700 Subject: [PATCH] Use bogus phone tokens for failure message check to avoid collisions (#21853) --- .../Specification.Tests/src/UserManagerSpecificationTests.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Identity/Specification.Tests/src/UserManagerSpecificationTests.cs b/src/Identity/Specification.Tests/src/UserManagerSpecificationTests.cs index bb6924ffb8..4bc1a37559 100644 --- a/src/Identity/Specification.Tests/src/UserManagerSpecificationTests.cs +++ b/src/Identity/Specification.Tests/src/UserManagerSpecificationTests.cs @@ -1486,8 +1486,8 @@ namespace Microsoft.AspNetCore.Identity.Test Assert.NotEqual(token1, token2); Assert.True(await manager.VerifyChangePhoneNumberTokenAsync(user, token1, num1)); Assert.True(await manager.VerifyChangePhoneNumberTokenAsync(user, token2, num2)); - Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, token2, num1)); - Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, token1, num2)); + Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, "bogus", num1)); + Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, "bogus", num2)); IdentityResultAssert.VerifyLogMessage(manager.Logger, $"VerifyUserTokenAsync() failed with purpose: ChangePhoneNumber:{num1} for user {await manager.GetUserIdAsync(user)}."); IdentityResultAssert.VerifyLogMessage(manager.Logger, $"VerifyUserTokenAsync() failed with purpose: ChangePhoneNumber:{num2} for user {await manager.GetUserIdAsync(user)}."); }