Skip to content

Commit 5508e76

Browse files
authored
Use bogus phone tokens for failure message check to avoid collisions (#21853)
1 parent ec0dbd0 commit 5508e76

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/Identity/Specification.Tests/src/UserManagerSpecificationTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1486,8 +1486,8 @@ public async Task CanVerifyPhoneNumber()
14861486
Assert.NotEqual(token1, token2);
14871487
Assert.True(await manager.VerifyChangePhoneNumberTokenAsync(user, token1, num1));
14881488
Assert.True(await manager.VerifyChangePhoneNumberTokenAsync(user, token2, num2));
1489-
Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, token2, num1));
1490-
Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, token1, num2));
1489+
Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, "bogus", num1));
1490+
Assert.False(await manager.VerifyChangePhoneNumberTokenAsync(user, "bogus", num2));
14911491
IdentityResultAssert.VerifyLogMessage(manager.Logger, $"VerifyUserTokenAsync() failed with purpose: ChangePhoneNumber:{num1} for user {await manager.GetUserIdAsync(user)}.");
14921492
IdentityResultAssert.VerifyLogMessage(manager.Logger, $"VerifyUserTokenAsync() failed with purpose: ChangePhoneNumber:{num2} for user {await manager.GetUserIdAsync(user)}.");
14931493
}

0 commit comments

Comments
 (0)