Skip to content
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

FI-2232 Convert FHIR Model #18

Merged
merged 15 commits into from
Nov 10, 2023
Merged

FI-2232 Convert FHIR Model #18

merged 15 commits into from
Nov 10, 2023

Conversation

emichaud998
Copy link
Contributor

Summary

Updated the way FHIR resources are passed around in tests from hashes to FHIR models. The only files that seemed to do this were: carin_search_test.rb, date_search_validator.rb, fhir_resource_navigator.rb, and must_support_test.rb. Added simple tests for each module that is used in the test kit just to make sure everything still ran correctly with these changes.

Testing Guidance

Can run the spec test files using bundle exec rspec or you can run Inferno locally and run each of the different tests in the Inferno UI to make sure that they run without error due to the source hash to FHIR model changes.

vanessuniq and others added 14 commits November 7, 2023 11:06
Signed-off-by: Vanessa Fotso <[email protected]>
…bundle also includes the resources that are requested in the _include parameter
…d check to ensure reference id is in the correct format before running the rest of the test. Also added test to include spec test that ensures test fails when reference id is of the wrong format
…e fhir models, and update all files that call and rely on those functions. Also fix c4bb patient stub json example incorrect language param format
…getting rid of using hashes for fhir resources
@emichaud998 emichaud998 force-pushed the FI-2232-convert-fhir-model branch from 7afdaa1 to f335ff2 Compare November 7, 2023 16:09
@emichaud998 emichaud998 merged commit 7e92134 into main Nov 10, 2023
2 checks passed
@emichaud998 emichaud998 deleted the FI-2232-convert-fhir-model branch November 10, 2023 16:08
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