[EHR] add pdf-parsing feature for tests for SSN feature#5946
[EHR] add pdf-parsing feature for tests for SSN feature#5946ValeriyDyachenko wants to merge 5 commits intorelease/1.26from
Conversation
| test.describe.configure({ mode: 'serial' }); | ||
|
|
||
| test('should display visit details page correctly', async () => {}); | ||
| test('should display SSN from pre-paperwork', async () => { |
There was a problem hiding this comment.
let's make sure this test doesn't run if paperwork in downstream repo doesn't have ssn field
There was a problem hiding this comment.
I added the ability to determine the repository where the test is running and to perform the check only if it's upstream https://github.com/masslight/ottehr/pull/5946/files#diff-dba89a5f00386a7c76cb45dcd9dbc8ec6c9027b20c0b371435fc867349bbf840R64-R74. I think this is a better option than optional checks within tests, because optional checks waste a lot of time on tests. Later, we can add separate checks for downstreams if needed. What do you think about this?
There was a problem hiding this comment.
some downstream repos have ssn field, so this won't work for some repos, this should be checked based on config
ValeriyDyachenko
force-pushed
the
vdyachenko/otr-1649-ehr-ssn-is-not-displayed-in-visit-details-patient-record
branch
from
d5c96af to
c36071a
Compare
ValeriyDyachenko
force-pushed
the
vdyachenko/otr-1649-ehr-ssn-is-not-displayed-in-visit-details-patient-record
branch
from
69c64e7 to
fd2da06
Compare
ValeriyDyachenko
changed the title
…r-1649-ehr-ssn-is-not-displayed-in-visit-details-patient-record
2 participants
https://linear.app/zapehr/issue/OTR-1649/ehr-ssn-is-not-displayed-in-visit-details-patient-record
/run-ehr-e2e