Skip to content

FQM-323: Verify server prefetch support#87

Draft
Jammjammjamm wants to merge 7 commits into
support-v2.2.0from
fqm-323-verify-server-prefetch-support
Draft

FQM-323: Verify server prefetch support#87
Jammjammjamm wants to merge 7 commits into
support-v2.2.0from
fqm-323-verify-server-prefetch-support

Conversation

@Jammjammjamm
Copy link
Copy Markdown
Contributor

Summary

This branch adds a check that at least one service advertises prefetch.

Testing Guidance

Test passes against the client suite. Failure is demonstrated in the unit tests.

Anticipated Provider-side Test Impact

None

@Jammjammjamm Jammjammjamm requested review from arscan and tstrass May 19, 2026 13:28
@Jammjammjamm Jammjammjamm self-assigned this May 19, 2026
@arscan
Copy link
Copy Markdown
Contributor

arscan commented May 19, 2026

Is it valid for an instance of a server to not use prefetch in any service because for some reason it just doesn't need them in that case? In which case, I wouldn't call these invalid as implied in the group title "Discovery response contains valid services". I would probably move this out to another simple test that is just "Server demonstrates ability to request the CDS Client perform prefetch queries' (language taken from https://cds-hooks.hl7.org/2026Jan/en/index.html#response). And then in the description note that the tests required demonstration of at least one prefetch query as evidence that it is capable of using prefetch.

Comment thread spec/davinci_crd_test_kit/v2.2.1/discovery_prefetch_support_test_spec.rb Outdated
@Jammjammjamm Jammjammjamm force-pushed the fqm-323-verify-server-prefetch-support branch from 3f0d277 to cd10232 Compare May 21, 2026 12:21
@Jammjammjamm Jammjammjamm requested review from arscan and tstrass May 21, 2026 12:26
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