Skip to content

Allow triggering different review_status values in mock DDP proofer#6829

Merged
matthinz merged 2 commits intomainfrom
matthinz/lg-7016-mock-ddp-updates
Aug 23, 2022
Merged

Allow triggering different review_status values in mock DDP proofer#6829
matthinz merged 2 commits intomainfrom
matthinz/lg-7016-mock-ddp-updates

Conversation

@matthinz
Copy link
Contributor

This PR adds 3 "magic" SSNs that will trigger review_status values from the mock LexisNexis DDP proofer:

SSN Status
666-77-8888 "reject"
666-77-9999 "review"
666-77-0000 nil

Any other SSN will return pass, as before. This will support testing of the integration of the proofer into the ID verification flow.

@matthinz matthinz requested a review from a team August 23, 2022 21:54
Add "magic" SSNs that will trigger 3 separate `review_status` values from the mock LexisNexis DDP proofer:

| 666-77-8888 | `"reject"` |
| 666-77-9999 | `"review"` |
| 666-77-0000 | `nil` |

This will support testing of the integration of the proofer into the ID verification flow.

changelog: Internal, ThreatMetrix, Allow triggering failures in mock DDP proofer (LG-7016)
@matthinz matthinz force-pushed the matthinz/lg-7016-mock-ddp-updates branch from 3fbcf5e to cbd878d Compare August 23, 2022 22:02
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

TRANSACTION_ID = 'ddp-mock-transaction-id-123'

# Trigger the "REJECT" status
REJECT_STATUS_SSN = '666-77-8888'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add unit tests for these?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matthinz matthinz merged commit 9823a45 into main Aug 23, 2022
@matthinz matthinz deleted the matthinz/lg-7016-mock-ddp-updates branch August 23, 2022 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants