TRUNK-4860: Assert validation error code for blank gender in PatientV…

[emphor11]

TRUNK-4860: Update PatientValidator test validation messages

Description of what I changed

Updated validation behavior and ensured correct validation messages in PatientValidator.
Verified and updated PatientValidatorTest so that all validation scenarios behave as expected.
All tests pass successfully after the changes.

Issue I worked on

see https://issues.openmrs.org/browse/TRUNK-4860

Checklist: I completed these to help reviewers :)

• My IDE is configured to follow the code style of this project.

• I have added tests to cover my changes (or verified existing tests where applicable).

• I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

• All new and existing tests passed.

• My pull request is based on the latest changes of the master branch.

[emphor11]

@dkayiwa
I’ve submitted this PR to address TRUNK-4860.
Could you please take a look when you get a chance?
Thanks!

[mherman22]

the following test methods also need your attention in this very test class:

• validate_shouldFailValidationIfAPreferredPatientIdentifierIsNotChose
• validate_shouldFailValidationIfFieldLengthsAreNotCorrect

[emphor11]

Thanks for the review @mherman22
Good catch — I’ll update those two test methods as well to assert the correct validation messages and push an update shortly.

[emphor11]

@mherman22 @dkayiwa
I’ve pushed the updated changes addressing the feedback.
All tests are passing locally now. Thanks!

[emphor11]

@dkayiwa @mherman22
If there anything else on which you want my attention regarding this PR you can let me know.
Thanks

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[emphor11]

Hi @dkayiwa @mherman22 ,

It looks like some CI checks were cancelled after the recent branch update.
All tests were passing locally and previously on CI.

Could you please re-run the cancelled checks or let me know if you’d like me to update anything?

Thanks!