Skip to content

Conversation

@ThomasNap
Copy link
Collaborator

No description provided.

@ThomasNap ThomasNap requested a review from VGiakoumi October 9, 2025 08:27
@ThomasNap ThomasNap changed the base branch from main to Geboortezorg-3-VLK October 13, 2025 12:51
<nts:with-parameter name="count" value="1"/>
<nts:with-parameter name="atLeast"/>
</nts:include>
<nts:contentAsserts href="gz-izv-kwa-casus3\mm-gbz-bc-Woman-MICHA-M-XXX-YASSIEN-MOHAMED.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 href moet wijzen naar map Verloskunde-Casus3, dus
href="Verloskunde-Casus3/mm-gbz-bc-Woman-MICHA-M-XXX-YASSIEN-MOHAMED.xml"

zo ook bij de overige tests, gz-izv-kwa-casus3\ moet vervangen worden door Verloskunde-Casus3/

Copy link
Contributor

@VGiakoumi VGiakoumi left a comment

Choose a reason for hiding this comment

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

Ik zal nog navragen bij collega's uit team SGUs of er een reden is om de testen met count=0 te behouden.
Zo niet, dan zou de TestScript voor de Client ook op dezelfde manier moeten worden bewerkt als het TestScript voor de Server, dus equivalente testen moeten bij beide verwijderd worden.
Ook zou daarna de benummering van de testen moeten worden bijgewerkt

<nts:contentAsserts href="gz-izv-kwa-casus3\mm-gbz-zib-BloodPressure85354-9-0899e443-6b74-11ec-2106-020000000000.xml" nts:in-targets="CheckContent">
<nts:discriminator>code.where(coding.where(code=85354-9))</nts:discriminator>
</nts:contentAsserts>
<!-- foetale sterfte -->
Copy link
Contributor

@VGiakoumi VGiakoumi Oct 13, 2025

Choose a reason for hiding this comment

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

dit blok kan er volgens mij helemaal uit, zo te zien aan de casus is de perinatale sterfte onder Kind niet meer aanwezig, en is er dus geen fixture PerinatalDeath meer
(alles onder de comment foetale sterfte, tot en met de contentAssert)

@VGiakoumi
Copy link
Contributor

Ik zal nog navragen bij collega's uit team SGUs of er een reden is om de testen met count=0 te behouden. Zo niet, dan zou de TestScript voor de Client ook op dezelfde manier moeten worden bewerkt als het TestScript voor de Server, dus equivalente testen moeten bij beide verwijderd worden. Ook zou daarna de benummering van de testen moeten worden bijgewerkt

Nagevraagd, en die testen kunnen er uit. Ze hebben idd geen toegevoegde waarde, waarschijnlijk zijn ze overgebleven van een vorige bewerking oid.
@ThomasNap zou je de test nummers willen updaten en willen checken dat het phr en xis TestScript in de pas lopen met elkaar?

@ThomasNap ThomasNap requested a review from VGiakoumi November 13, 2025 11:20
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