-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug: Fix REC_SSI Cat4 Validator #3129
Conversation
), | ||
T5Factory.create( | ||
RPT_MONTH_YEAR=202010, | ||
CASE_NUMBER='123', | ||
DATE_OF_BIRTH="19970209", | ||
FAMILY_AFFILIATION=2, # validator only applies to fam_affil = 1; won't generate error | ||
REC_AID_TOTALLY_DISABLED=2, | ||
REC_SSI=2 | ||
REC_SSI=0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed this to 0
to prove that the FAMILY_AFFILIATION value was excluding it from the validator and therefore only generating the single error as opposed to two errors.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3129 +/- ##
========================================
Coverage 93.01% 93.01%
========================================
Files 277 277
Lines 7421 7421
Branches 660 660
========================================
Hits 6903 6903
Misses 415 415
Partials 103 103
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! thanks @elipe17 🚀
confirmed the following:
- Cat 4 error raised when fam affil=1 and rec_ssi out of range for a state ✔️
- error not raised when condition not present ✔️
- Cat 4 error raised when fam affil=1 and rec_ssi out of range for a territory ✔️
Summary of Changes
Pull request closes [bug] TDP is raising cat 4 error on TANF/SSP closed case files that is not present #3073
How to Test
Rejected
IN_SSPClosed_anon.txt
MI_TANFClosed_anon.txt
Deliverables
More details on how deliverables herein are assessed included here.
Deliverable 1: Accepted Features
Checklist of ACs:
Deliverable 2: Tested Code
CodeCov Report
comment in PR)CodeCov Report
comment in PR)Deliverable 3: Properly Styled Code
Deliverable 4: Accessible
iamjolly
andttran-hub
using Accessibility Insights reveal any errors introduced in this PR?Deliverable 5: Deployed
Deliverable 6: Documented
Deliverable 7: Secure
Deliverable 8: User Research
Research product(s) clearly articulate(s):