Skip to content

Nictiz - 33769] - Rewrite testscript for verloskunde uitbreiding casus 2#402

Open
ThomasNap wants to merge 371 commits into
Geboortezorg-3-VLKfrom
NICTIZ-33769]
Open

Nictiz - 33769] - Rewrite testscript for verloskunde uitbreiding casus 2#402
ThomasNap wants to merge 371 commits into
Geboortezorg-3-VLKfrom
NICTIZ-33769]

Conversation

@ThomasNap

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

lisaDekker-nictiz and others added 30 commits December 18, 2025 13:32
Fix tellingen and added new patients
* Fresh start

* Add required build.properties output

* Add changes to generation src-properties

* Temp edits in build.properties

* Temp edits in buildscripts and cleanup

* Revert temp build edits

* ada2nts

* Regenerated output MP9-3 - Test/Cert

---------

Co-authored-by: Jorn Duwel <[email protected]>
* Vital signs id's updated

* MP907 id's updated

* regenerate output for test

* additional changes to id's vital signs
* removal of scripts en fixtures regarding negotiation Cert

* removal of scripts and fixtures regarding negotiation Test plus refactoring test flow logic

* rebuild output eOverdracht

* adding leading zero to testscript.id for UI sequence

* CL-109: Add origins and destinations configuration for receiving XIS scenario and update task ID references Test environment

* CL-109: rebuild output

* EOVDR-833: replacing hint to description and fixing references to Touchstone

* remove practitioner and practitioner role in receiving organization following profile

* test: regenerate output

* fix: remove practitioner and practitioner role in receiving organization following profile

* adding description of variable task and generalizing task id value

* changing filename consistent with id

* CLI-13: change to correct code following TestScript r5

* make instruction of description of variable

* change filename of cert for consistency

* regenerate output for test
PDFA - Remove xml/json accept headers from operations that explicitly…
* CL-316 - Edit MP907 Cert Dispense612Conversion PHR scripts. Edit groovy script, remove asserts that check server response

* Fix 1 script

* Add descriptions to Groovy asserts

* Test query-param assert

* Replace RequestURLContains groovy script with query-param asserts

* Fix tests with multiple query-param asserts

* Fix periodofuse

* Fix le/lt

* Rebuild output
Resolved conflicts by adding src-properties.json files in src/Geboortezorg-3-0/Cert/PHR-Client/ and ../XIS-Server
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.