Use selected in-person location details from session

[aduth]

Context: #6635 (comment)

Why: Because the user's selection should be respected.

Screenshot:

[aduth]

Maybe a later pull request if we're not yet targeting GPO support, but we need to figure out how to pass this from GpoVerifyForm:

identity-idp/app/forms/gpo_verify_form.rb

Lines 22 to 26 in 1c84c38

	UspsInPersonProofing::EnrollmentHelper.new.save_in_person_enrollment(
	user,
	pending_profile,
	pii,
	)

This is difficult because (a) we can't rely on the session and (b) the enrollment record hasn't been created yet.

My initial instinct is that we should do something like what I described as the second option of #6629 (comment) : always create the enrollment in the initial flow, but keep it establishing and wait to send it to USPS. That way, we have the selected location details saved in the record. The challenge is that we need to update how we're thinking about deadline dates, since we cannot use creation date of the record (it may be different from the USPS enrollment creation).

[tomas-nava]

I think creating an :establishing enrollment and adding a started_pending_at (or something) column to the enrollments table that we could us to calculate the expiration date would work.

[aduth]

@tomas-nava Would you want to see that here, or should we follow-up with that as part of the GPO supports implementation?

[tomas-nava]

let's follow up with that

[aduth]

This might have been working incorrectly before as well. Note below in create_usps_enrollment we pick out pii attributes from the hash as string keys, not symbols. I confirmed in the FSMv1 flow that all these keys are strings. I haven't yet tested FSMv2. I believe GPO passes as an instance of Pii::Attributes which supports indifferent access.

[2] pry(main)> Pii::Attributes.new_from_hash(Idp::Constants::MOCK_IDV_APPLICANT_WITH_PHONE)['first_name']
=> "FAKEY"

[tomas-nava]

approve! I appreciate the new specs