diff --git a/src/Identity/IdentityServer/RequestValidators/SendAccess/SendEmailOtpRequestValidator.cs b/src/Identity/IdentityServer/RequestValidators/SendAccess/SendEmailOtpRequestValidator.cs index c7ead714c4..d8d82a62c4 100644 --- a/src/Identity/IdentityServer/RequestValidators/SendAccess/SendEmailOtpRequestValidator.cs +++ b/src/Identity/IdentityServer/RequestValidators/SendAccess/SendEmailOtpRequestValidator.cs @@ -35,7 +35,7 @@ public class SendEmailOtpRequestValidator( // It is an invalid request if the email is missing. if (string.IsNullOrEmpty(email)) { - // Request is the wrong shape and doesn't contain an email field.' + // Request is the wrong shape and doesn't contain an email field. return BuildErrorResult(SendAccessConstants.EmailOtpValidatorResults.EmailRequired); } /* diff --git a/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendEmailOtpReqestValidatorIntegrationTests.cs b/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendEmailOtpReqestValidatorIntegrationTests.cs index 3c4657653b..b8e0370ede 100644 --- a/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendEmailOtpReqestValidatorIntegrationTests.cs +++ b/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendEmailOtpReqestValidatorIntegrationTests.cs @@ -83,7 +83,7 @@ public class SendEmailOtpRequestValidatorIntegrationTests(IdentityApplicationFac // Assert var content = await response.Content.ReadAsStringAsync(); Assert.Contains(OidcConstants.TokenErrors.InvalidRequest, content); - Assert.Contains("email otp sent", content); + Assert.Contains("email and otp are required", content); } [Fact] diff --git a/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendNeverAuthenticateRequestValidatorTest.cs b/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendNeverAuthenticateRequestValidatorTest.cs index a81b01a293..4796d1f913 100644 --- a/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendNeverAuthenticateRequestValidatorTest.cs +++ b/test/Identity.IntegrationTest/RequestValidation/SendAccess/SendNeverAuthenticateRequestValidatorTest.cs @@ -63,7 +63,7 @@ public class SendNeverAuthenticateRequestValidatorIntegrationTests( } [Fact] - public async Task SendAccess_NeverAuthenticateSend_WithEmail_ReturnsEmailInvalid() + public async Task SendAccess_NeverAuthenticateSend_WithEmail_ReturnsEmailAndOtpRequired() { // Arrange var email = "test@example.com"; @@ -77,10 +77,10 @@ public class SendNeverAuthenticateRequestValidatorIntegrationTests( var content = await response.Content.ReadAsStringAsync(); // should be invalid grant - Assert.Contains(OidcConstants.TokenErrors.InvalidGrant, content); + Assert.Contains(OidcConstants.TokenErrors.InvalidRequest, content); // Try to compel the invalid email error - var expectedError = SendAccessConstants.EmailOtpValidatorResults.EmailInvalid; + var expectedError = SendAccessConstants.EmailOtpValidatorResults.EmailAndOtpRequired; Assert.Contains(expectedError, content); }