LG-12160: Add liveness_checking_required To BE Logs#10150
Conversation
charleyf
changed the title
charleyf
changed the title
charleyf
requested review from
a team and
eileen-nava
and removed request for
a team
|
thanks for adding the logging ... from my testing, it appears that IdV: doc auth image upload form submitted |
Co-authored-by: Amir Reavis-Bey <amir.reavis-bey@gsa.gov>
There was a problem hiding this comment.
thanks for adding the logging ... from my testing, it appears that IdV: doc auth image upload form submitted
and IdV: doc auth image upload vendor submitted events already log liveness_checking_required. unfortunately IdV: doc auth image upload form submitted is only tested in 1 flow in analytics_spec☹️
Fixed both of those things thanks!
- Added the missing event/step into the other flows.
- Removed the duplicate liveness_checking_required where the response already had it.
| 'IdV: doc auth image upload form submitted' => { | ||
| success: true, errors: {}, submit_attempts: 1, remaining_submit_attempts: 3, user_id: user.uuid, flow_path: 'standard', front_image_fingerprint: an_instance_of(String), back_image_fingerprint: an_instance_of(String), selfie_image_fingerprint: nil, liveness_checking_required: boolean | ||
| }, | ||
| 'IdV: doc auth image upload vendor submitted' => hash_including(success: true, flow_path: 'standard', attention_with_barcode: false, doc_auth_result: 'Passed', liveness_checking_required: boolean), |
amirbey
left a comment
amirbey
left a comment
There was a problem hiding this comment.
LGTM 👍🏿
thanks for adding the vendor submitted events to the spec 🙏🏿
2 participants
🎫 Ticket
https://cm-jira.usa.gov/browse/LG-12160
🛠 Summary of changes
Add
liveness_checking_requiredfield to some BE logs, specfically:📜 Testing Plan
Provide a checklist of steps to confirm the changes.
liveness_checking_requiredfield that saysfalse.liveness_checking_requiredfield that saystrue.