Skip to content

LG-8932: Fix check for state vs residential address; update test#8124

Merged
NavaTim merged 2 commits intomainfrom
tbradley/lg-8932-fix-address-match-check
Apr 4, 2023
Merged

LG-8932: Fix check for state vs residential address; update test#8124
NavaTim merged 2 commits intomainfrom
tbradley/lg-8932-fix-address-match-check

Conversation

@NavaTim
Copy link
Contributor

@NavaTim NavaTim commented Apr 3, 2023

🎫 Ticket

LG-8932

🛠 Summary of changes

  • Fix check for state vs residential address
    • Expected boolean type, was actually string type

Also see: #8098

📜 Testing Plan

  • Enable feature flag in_person_capture_secondary_id_enabled
  • Complete in in-person proofing process up to the state ID form
  • Complete state ID form, indicating that the residential address differs from the ID
  • Complete the following residential address form with a different address
  • Complete the remainder of the in-person proofing process
  • Verify with USPS that they received the state ID address

@JackRyan1989
Copy link
Contributor

@NavaTim I assume the testing plan is for local development? Can you help me with the last step: Verify with USPS that they received the state ID address?

@NavaTim
Copy link
Contributor Author

NavaTim commented Apr 4, 2023

@NavaTim I assume the testing plan is for local development? Can you help me with the last step: Verify with USPS that they received the state ID address?

@JackRyan1989 For local development, you'll likely need to print out the values (p/puts) when they're sent in app/services/usps_in_person_proofing/enrollment_helper.rb or a similar location.

Copy link
Contributor

@JackRyan1989 JackRyan1989 left a comment

Choose a reason for hiding this comment

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

LGTM!

@eileen-nava
Copy link
Contributor

I reviewed this and confirmed that the state id address is sent to USPS, which is good. 👍🏻 I'm not sure if you want to make the state_id_jurisdiction change here or in another PR. Also, just a heads-up that I didn't look at the CI's failing tests, so I can't speak to those.

@NavaTim NavaTim merged commit 886f937 into main Apr 4, 2023
@NavaTim NavaTim deleted the tbradley/lg-8932-fix-address-match-check branch April 4, 2023 20:09
jc-gsa pushed a commit that referenced this pull request Apr 19, 2023
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.

3 participants