From 1c0c213ff474085bf64bcad22d1fbd70bf599546 Mon Sep 17 00:00:00 2001 From: Emphor <022dakshy@gmail.com> Date: Thu, 29 Jan 2026 12:34:39 +0530 Subject: [PATCH] TRUNK-4860: Assert validation error code for blank gender in PatientValidatorTest TRUNK-4860: Assert validation error codes in PatientValidatorTest --- .../java/org/openmrs/validator/PatientValidatorTest.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/api/src/test/java/org/openmrs/validator/PatientValidatorTest.java b/api/src/test/java/org/openmrs/validator/PatientValidatorTest.java index 0bf2a5783f1d..ed367022b08a 100644 --- a/api/src/test/java/org/openmrs/validator/PatientValidatorTest.java +++ b/api/src/test/java/org/openmrs/validator/PatientValidatorTest.java @@ -29,6 +29,8 @@ import org.springframework.validation.BindException; import org.springframework.validation.Errors; import org.springframework.validation.Validator; +import static org.junit.Assert.assertEquals; + /** * Tests methods on the {@link PatientValidator} class. @@ -56,6 +58,8 @@ public void validate_shouldFailValidationIfAPreferredPatientIdentifierIsNotChose Errors errors = new BindException(pa, "patient"); validator.validate(pa, errors); assertTrue(errors.hasErrors()); + assertTrue(errors.hasGlobalErrors()); + assertEquals("error.preferredIdentifier", errors.getGlobalError().getCode()); } /** @@ -117,6 +121,10 @@ public void validate_shouldFailValidationIfGenderIsBlank() { validator.validate(pa, errors); assertTrue(errors.hasFieldErrors("gender")); + assertTrue( + errors.getFieldErrors("gender").stream() + .anyMatch(e -> "Person.gender.required".equals(e.getCode())) + ); } /** @@ -194,5 +202,6 @@ public void validate_shouldFailValidationIfFieldLengthsAreNotCorrect() { validator.validate(patient, errors); assertTrue(errors.hasFieldErrors("voidReason")); + assertEquals("error.exceededMaxLengthOfField",errors.getFieldError("voidReason").getCode()); } }