-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set up local HAPI server for e2e tests #90
Set up local HAPI server for e2e tests #90
Conversation
…connector into marcelle/2758-set-up-local-hapi-server-for-testing
…into marcelle/2758-set-up-local-hapi-server-for-testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for sorting all of this out!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am so happy this is done 😭 😎
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks so much for doing this! 🤞 this solves all our problems
PULL REQUEST
Summary
This PR adds a new
docker-compose-e2e-tests.yaml
that spins up a local HAPI FHIR server and loads the Hyper Unlucky patient into the server. I chose to add a new dock compose file to prevent the primary,docker compose
file from requiring HAPI, which is only used for e2e tests, but open to hearing others' ideas.I also updated the e2e tests to manually select the local instance of the HAPI server, which is now available in the drop down.
Some lessons learned:
fhir_servers
.Related Issue
Fixes # phdi 2758
Acceptance Criteria
Please copy the acceptance criteria from your ticket and paste it here for your reviewer(s)
For frontend PR’s - include a description (including anything that’s out of scope) for what you want the designers to review
Additional Information
Checklist