MarriageCertificateBaseNullFieldsValidatorTest (2)

src/main/java/scot/carricksoftware/grants/validators/certificates/marriagecertificate/MarriageCertificateNullFieldsValidatorImpl.java

@@ -91,7 +91,7 @@ public class MarriageCertificateNullFieldsValidatorImpl implements MarriageCerti
 
     private void validateGroomFatherRank(MarriageCertificateCommand marriageCertificateCommand, BindingResult bindingResult) {
         logger.debug("MarriageCertificateNullFieldsValidator::validateGroomFatherRank");
-        validateTypes.validateNullOrEmptyString(marriageCertificateCommand.getGroomFatherRank(), "groomFatherRank", ValidationConstants.BRIDE_FATHER_RANK_IS_NULL, bindingResult);
+        validateTypes.validateNullOrEmptyString(marriageCertificateCommand.getGroomFatherRank(), "groomFatherRank", ValidationConstants.GROOM_FATHER_RANK_IS_NULL, bindingResult);
     }
 
 

src/test/java/scot/carricksoftware/grants/validators/certificates/marriagecertificate/MarriageCertificateNullFieldsValidatorTest.java

@@ -51,6 +51,10 @@ class MarriageCertificateNullFieldsValidatorTest {
                 "brideRank", "The bride rank cannot be null.", bindingResultMock);
         verify(validateTypesMock).validateNullOrEmptyString(marriageCertificateCommand.getBrideCondition(),
                 "brideCondition", "The bride condition cannot be null.", bindingResultMock);
+        verify(validateTypesMock).validateNullOrEmptyString(marriageCertificateCommand.getBrideFatherRank(),
+                "brideFatherRank", "The bride's father's rank cannot be null.", bindingResultMock);
+        verify(validateTypesMock).validateNullOrEmptyString(marriageCertificateCommand.getGroomFatherRank(),
+                "groomFatherRank", "The groom's father's rank cannot be null.", bindingResultMock);
     }
 
     @Test