Skip to content

FQM-280: Verify coverage information system actions only add/update the coverage-information extension#83

Merged
karlnaden merged 3 commits into
support-v2.2.0from
fqm-280-coverage-info-add-update
May 20, 2026
Merged

FQM-280: Verify coverage information system actions only add/update the coverage-information extension#83
karlnaden merged 3 commits into
support-v2.2.0from
fqm-280-coverage-info-add-update

Conversation

@tstrass
Copy link
Copy Markdown
Contributor

@tstrass tstrass commented May 15, 2026

Summary

This updates the CRD 2.2.1 server Coverage Information system action validation test to verify resp-37: when a server returns a Coverage Information update, the returned resource may only add or modify the coverage-information extension and must not change any other resource content. I was on the fence about putting it in this test or adding a new test. Happy to switch it to its own test if reviewers prefer.

To get a source resource to compare against, we pull from the hook request context, prefetch, or mock_ehr_bundle as needed across hook types. If we can't find one, we emit a warning.

Testing Guidance

  • Run the server suite against the client suite using the presets.
  • You should see no new errors or warnings for the coverage info system action validation test. There will be some existing errors and warnings for some hooks, unrelated to these changes.
  • Note this demonstrates passing behavior for every hook except encounter-start, which doesn't return a coverage information systemAction.

Anticipated Provider-side Test Impact

None

@tstrass tstrass requested review from Jammjammjamm and arscan May 15, 2026 15:05
@tstrass tstrass self-assigned this May 15, 2026
@tstrass tstrass force-pushed the fqm-280-coverage-info-add-update branch 2 times, most recently from cd6da72 to cd025ba Compare May 19, 2026 20:58
@tstrass tstrass force-pushed the fqm-280-coverage-info-add-update branch from cd025ba to 2f2543e Compare May 20, 2026 12:52
@arscan arscan marked this pull request as ready for review May 20, 2026 14:47
@arscan arscan requested a review from karlnaden May 20, 2026 14:47
@arscan
Copy link
Copy Markdown
Contributor

arscan commented May 20, 2026

FYI @karlnaden, thanks!

@karlnaden karlnaden merged commit cafc337 into support-v2.2.0 May 20, 2026
@karlnaden karlnaden mentioned this pull request May 20, 2026
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