Appointment search using VPG

[devin-mccurdy]

Summary

• Changes are behind va_online_scheduling_enable_OH_reads feature flag
• Appointment search requests are now sent to the vpg endpoint instead of vaos

Related issue(s)

https://app.zenhub.com/workspaces/appointments-oracle-health-integration-65a6e99ea522640e4d09393b/issues/gh/department-of-veterans-affairs/va.gov-team/83349

Testing done

• New code is covered by unit tests

What areas of the site does it impact?

These changes impact health appointment creation.

Acceptance criteria

• I fixed|updated|added unit tests and integration tests for each feature (if applicable).
• No error nor warning in the console.
• Events are being sent to the appropriate logging solution

[github-actions]

1 Error

🚫

This PR changes 524 LoC (not counting whitespace/newlines). In order to ensure each PR receives the proper attention it deserves, those exceeding 500 will not be reviewed, nor will they be allowed to merge. Please break this PR up into smaller ones. If you have reason to believe that this PR should be granted an exception, please see the Submitting pull requests for approval - FAQ. File Summary Files modules/mobile/spec/request/appointments_vaos_v2_list_request_spec.rb (+420/-2) modules/vaos/app/services/vaos/v2/appointments_service.rb (+20/-4) modules/vaos/spec/services/v2/appointment_service_spec.rb (+78/-0) Note: We exclude files matching the following when considering PR size: *.csv, *.json, *.tsv, *.txt, Gemfile.lock, app/swagger, modules/mobile/docs, spec/fixtures/, spec/support/vcr_cassettes/, modules/mobile/spec/support/vcr_cassettes/, db/seeds, modules/vaos/app/docs, modules/meb_api/app/docs, modules/appeals_api/app/swagger/, *.bru Big PRs are difficult to review, often become stale, and cause delays.

Generated by 🚫 Danger

[drewconnelly]

The changes look good, but it appears that the size of the PR has tripped the LOC limit. I wonder if some of the tests could be added to a separate ticket.

[aherzberg]

Mobile changes look good to me. Are there any functional differences between VAOS and VPG?

[devin-mccurdy]

Mobile changes look good to me. Are there any functional differences between VAOS and VPG?

No, for now VPG is using VAOS as the source

[aherzberg]

LGTM

[cferris32]

LGTM for VAOS