Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MM 2055 #39

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

MM 2055 #39

wants to merge 3 commits into from

Conversation

ArdonToonstra
Copy link
Contributor

Hee Pieter!
Zou je eens mee willen kijken?
Ik krijg het nog niet voor elkaar om een extra interne folder (zoals gespecificeerd in targets.additional) te laten genereren. De #default lijkt wel goed te gaan. Zou het te maken hebben dan de betreffende testscripts één folder dieper zitten dan normaal?

Ik heb verschillende dingen geprobeerd. Het staat nu wellicht iets wolliger dan nodig is. Ik heb nu namelijk ook expliciet aangegeven welke stukken in de Nictiz interne folder terecht moeten komen en heb ik bijvoorbeeld ook de splitsing in NTS op de nts:include gezet ipv de nts:with-parameter om te kijken of het daar aanlag.

@@ -10,14 +10,20 @@
<test id="Scenario-3-1b">
<name value="Scenario 3.1b"/>
<description value="Filter op verstrekkingsperiode, alles vanaf T-90."/>
<nts:include value="phr-operation-search-dispense612conversion.xml">
<nts:include nts:in-targets="PHR-Client-Nictiz-Intern" value="phr-operation-search-dispense612conversion.xml">

Choose a reason for hiding this comment

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

Dit zou alleen "Nictiz-Intern" moeten zijn.
Nb. in andere projecten noemen we dat "Nictiz-intern", met een kleine "i".

@jd-nictiz jd-nictiz changed the base branch from main to kcr-july July 22, 2021 08:04
Base automatically changed from kcr-july to main August 3, 2021 08:59
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.

2 participants