Skip to content

Update README.md with Bulk Export instructions#48

Merged
Shaumik-Ashraf merged 12 commits into
mainfrom
update-readme-for-bulk-export-tests
Jul 7, 2025
Merged

Update README.md with Bulk Export instructions#48
Shaumik-Ashraf merged 12 commits into
mainfrom
update-readme-for-bulk-export-tests

Conversation

@Shaumik-Ashraf

Copy link
Copy Markdown
Contributor

Summary

Add instructions on how to run Bulk Export Server tests, which were missing earlier and aren't supported by the HL7 FHIR Foundry PDex server.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
some server test suite requests. As a work-around, run the server test suite a second time while the
client is waiting for clinical data requests during test 1.2.

## Running the Bulk Export Server Tests

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Another demonstration that would be valuable would be to show how to run against the PDex Client tests. I tried this and it didn't get very far because the bulk tests stop the workflow if they detect even small things like bad HTTP statuses (see FI-3949). Probably doesn't make sense to resolve that or fixes to make this test kit return the right statuses as a part of this ticket, but wanted to mention it.

@Shaumik-Ashraf Shaumik-Ashraf requested a review from karlnaden June 30, 2025 22:52
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment on lines +134 to +136
Since this exercise requires running two test kits which causes port conflicts by default, you are advised to do
it on [the ONC website](https://inferno.healthit.gov/test-kits/) or locally on the
[inferno platform](https://github.com/inferno-framework/inferno-platform-template/).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

But it is all the PDex test kit, so you don't have to run more than 1 test kit at a time locally to do this. I think you can remove this paragraph unless I'm missing something.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You need the smart-app-launch-test-kit for the handshake that's why

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

SMART is included in the pdex test kit:
image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That is a good point - removed

Co-authored-by: Karl Naden <86612928+karlnaden@users.noreply.github.com>
@Shaumik-Ashraf Shaumik-Ashraf requested a review from karlnaden July 2, 2025 20:03
{
"name": "url",
"value": "http://localhost:4567/custom/pdex_payer_client/fhir",
"value": "http://inferno.healthit.gov/suites/custom/pdex_payer_client/fhir",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should turn this into an erb file so that the the host will be filled in correctly based on the deployed location

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

bulk_timeout needs to be 600 seconds to work with the reference server, at least locally, so let's set it there in the preset.

@Shaumik-Ashraf Shaumik-Ashraf requested a review from karlnaden July 3, 2025 21:37

@karlnaden karlnaden left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One small tweak, otherwise good - thanks for catching this and following through on it!

Comment thread README.md Outdated
Shaumik-Ashraf and others added 2 commits July 7, 2025 11:57
Co-authored-by: Karl Naden <86612928+karlnaden@users.noreply.github.com>
@Shaumik-Ashraf Shaumik-Ashraf merged commit b30224b into main Jul 7, 2025
3 checks passed
@Shaumik-Ashraf Shaumik-Ashraf deleted the update-readme-for-bulk-export-tests branch July 7, 2025 16:33
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.

2 participants