Skip to content

FQM-318: Standard prefetch expressions test#120

Merged
karlnaden merged 4 commits into
mainfrom
fqm-318-standard-prefetch
Jul 6, 2026
Merged

FQM-318: Standard prefetch expressions test#120
karlnaden merged 4 commits into
mainfrom
fqm-318-standard-prefetch

Conversation

@tstrass

@tstrass tstrass commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

This adds a test for found-29. It checks whether the server's advertised prefetch queries match the standard queries, and emits warnings when they don't.

The IG's language around this requirement is a bit vague, so please check my interpretation.

The actual requirement:

Servers SHALL use prefetch expressions in the manner described below if those data elements are relevant to their coverage determination or other decision support.

Since the server is allowed to have more or fewer prefetch expressions than the standard set, I conclude that whether or not a mismatched prefetch expression violates the requirement depends on the intent of the expression, which is impossible for Inferno to know, and therefore up to the tester's judgement. I also took the wording of the requirement to mean that Inferno should not enforce specific expression key names (e.g. 'pat' versus 'patient').

Testing Guidance

  • Run the Discovery group against the client suite (pick any hook group) with "Run Against the CRD Client Suite" preset. The new test should pass with no warnings.
  • Run the Discovery group against the client suite (pick any hook group) with "Run Against the CRD Client Suite's Preset Subset Services". The new test should pass with 16 warnings.

Anticipated Provider-side Test Impact

None

@tstrass tstrass requested review from Jammjammjamm and arscan June 23, 2026 15:02
@tstrass tstrass self-assigned this Jun 23, 2026
@tstrass tstrass force-pushed the fqm-318-standard-prefetch branch from c846b45 to 92d9855 Compare June 23, 2026 17:03
@tstrass tstrass force-pushed the fqm-318-standard-prefetch branch from 92d9855 to bfc5fab Compare June 23, 2026 17:03
@tstrass tstrass requested a review from Jammjammjamm June 25, 2026 17:33
@arscan arscan marked this pull request as ready for review June 30, 2026 20:11
@arscan arscan requested a review from karlnaden June 30, 2026 20:12
@karlnaden karlnaden merged commit 50d515c into main Jul 6, 2026
4 checks passed
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.

4 participants