Merge pull request cfpb#1096 from nickgrippin/v463-fix

V463 - logic corrected

Author: schbetsy

validation/src/main/scala/hmda/validation/rules/lar/validity/V463.scala

@@ -10,9 +10,8 @@ object V463 extends EditCheck[LoanApplicationRegister] {
   override def name: String = "V463"
 
   override def apply(lar: LoanApplicationRegister): Result = {
-    when(lar.applicant.coEthnicity is equalTo(5)) {
-      (lar.applicant.coRace1 is equalTo(8)) and
-        (lar.applicant.coSex is equalTo(5))
+    when((lar.applicant.coRace1 is equalTo(8)) or (lar.applicant.coSex is equalTo(5))) {
+      lar.applicant.coEthnicity is equalTo(5)
     }
   }
 }

validation/src/test/scala/hmda/validation/rules/lar/validity/V463Spec.scala

@@ -5,9 +5,9 @@ import hmda.validation.rules.EditCheck
 import hmda.validation.rules.lar.LarEditCheckSpec
 
 class V463Spec extends LarEditCheckSpec {
-  property("Succeeds when Co-Applicant ethnicity is not 5") {
+  property("Succeeds when Co-Applicant race != 8 and sex != 5") {
     forAll(larGen) { lar =>
-      whenever(lar.applicant.coEthnicity != 5) {
+      whenever(lar.applicant.coRace1 != 8 && lar.applicant.coSex != 5) {
         lar.mustPass
       }
     }
@@ -21,20 +21,20 @@ class V463Spec extends LarEditCheckSpec {
     }
   }
 
-  property("Fails when Co-Applicant ethnicity=5, race1=8, and sex NOT 5") {
+  property("Fails when Co-Applicant race1=8, and ethnicity!=5") {
     forAll(larGen) { lar =>
-      whenever(lar.applicant.coSex != 5) {
-        val invalidApplicant = lar.applicant.copy(coEthnicity = 5, coRace1 = 8)
+      whenever(lar.applicant.coEthnicity != 5) {
+        val invalidApplicant = lar.applicant.copy(coRace1 = 8)
         val invalidLar = lar.copy(applicant = invalidApplicant)
         invalidLar.mustFail
       }
     }
   }
 
-  property("Fails whenever Applicant ethnicity=5, sex=5, and coRace1 NOT 8") {
+  property("Fails whenever Applicant sex=5, and ethnicity!=8") {
     forAll(larGen) { lar =>
-      whenever(lar.applicant.coRace1 != 8) {
-        val invalidApplicant = lar.applicant.copy(coSex = 5, coEthnicity = 5)
+      whenever(lar.applicant.coEthnicity != 5) {
+        val invalidApplicant = lar.applicant.copy(coSex = 5)
         val invalidLar = lar.copy(applicant = invalidApplicant)
         invalidLar.mustFail
       }