Skip to content

Conversation

@ThomasNap
Copy link
Collaborator

No description provided.

@ThomasNap ThomasNap requested a review from VGiakoumi October 6, 2025 09:50
@ThomasNap ThomasNap changed the base branch from main to Geboortezorg-3-VLK October 13, 2025 12:51
<nts:with-parameter name="count" value="0"/>
<nts:with-parameter name="atLeast"/>
</nts:include>
<nts:contentAsserts href="gz-izv-kwa-casus2\mm-gbz-bc-Woman-FIONA-F-XXX-SWART.xml" nts:in-targets="CheckContent"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

de folder moet hier hetzelfde blijven als ervoor zodat de juiste fixture gevonden wordt,
Verloskunde-Casus2/
en de filename moet wel veranderen.
Hetzelfde voor de andere testen. gz-izv-kwa-casus2\ moet vervangen worden door Verloskunde-Casus2/

Copy link
Contributor

Choose a reason for hiding this comment

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

de integrity check geeft aan dat er ergens een closing tag mist , zou je dat willen nakijken?

</nts:include>
<nts:contentAsserts href="Verloskunde-Casus2/bc-Encounter-4ce993b0-a1ab-48dc-86db-683b7254c264.xml"
selector="iif(Bundle.entry.resource.ofType(Encounter).where(type.where(coding.code='11429006'))[0].period.start &lt; Bundle.entry.resource.ofType(Encounter).where(type.where(coding.code='11429006'))[1].period.start, Bundle.entry.resource.ofType(Encounter).where(type.where(coding.code='11429006'))[0], Bundle.entry.resource.ofType(Encounter).where(type.where(coding.code='11429006'))[1])"
<nts:contentAsserts href="gz-izv-kwa-casus2\mm-gbz-bc-Encounter-a6dcb094-6b39-11ec-1275-020000000000.xml"
Copy link
Contributor

Choose a reason for hiding this comment

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

bij deze en de volgende contentAsserts zijn er geen selectors in huidige edits, maar ik krijg de indruk dat dat nog wel nodig is? Want hoe weet je anders in de test welke opgehaalde Encounter met code 11429006 je moet vergelijken met de inhoud van welke fixture?
De drie fixtures hebben veschillende period.start waardes, op basis van die sortering zou je de selectors kunnen opstellen.
De fixtures zijn niet (meer) compleet identiek dus misschien is er nog een alternatief voor de selectors, maar de period.start gebruiken zou iig moeten lukken.

<nts:discriminator>code.where(coding.where(code='85354-9'))</nts:discriminator>
<nts:discriminator>component.where(value.where(value='108'))</nts:discriminator>
</nts:contentAsserts>
<!-- ADD: bloeddruk T-10 mist als resource dus ook geen path beschikbaar -->
Copy link
Contributor

Choose a reason for hiding this comment

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

als ik kijk naar de ADA omgeving zie ik inderdaad x3 broeddruk onder bouwstenen, maar die bouwstenen komen niet allemaal terug in de invulling daarvoor; de meting van T-10D komt voor zover ik kan zien nergens meer terug.
Ik vermoed dat dit de reden is dat er geen fixture met T-10D is.
Als je vindt dat die er wel zou moeten zijn, weet je waar die onder hoort in het ADA formulier? Dan zou er een vervolgticket moeten komen voor de IA's om het op te pakken.
voor nu zijn er 2 opties

  • of we gaan uit van x2 bloeddruk en passen de count hier aan naar 2,
  • of we moeten het functioneel rechttrekken naar x3, en daarna kan het TestScript ook aangepast worden met de juiste resource naam en de count kan op 3

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