Nictiz 29502 new #342
Nictiz 29502 new #342
Conversation
…to solve output build errors
| <nts:include value="assert.response.numResources" scope="common"> | ||
| <nts:with-parameter name="resource" value="DiagnosticReport"/> | ||
| <nts:with-parameter name="count" value="0"/> | ||
| <nts:with-parameter name="resource" value="ReferralRequest"/> |
There was a problem hiding this comment.
Ik zie dat dit in de andere testscripts ook fout staat, zou je daar ook DiagnosticReport naar ReferralRequest kunnen aanpassen, zodat het overal goed staat?
| <nts:with-parameter name="system" value="http://loinc.org"></nts:with-parameter> | ||
| </nts:include> | ||
| <nts:contentAsserts href="Kraam-Casus1/medmij-gbz-zib-BodyWeight29463-7-a4506b72-2291-11ec-8669-020000000000.xml" nts:in-targets="CheckContent"> | ||
| <!-- <nts:contentAsserts href="Kraam-Casus1/medmij-gbz-zib-BodyWeight29463-7-a4506b72-2291-11ec-8669-020000000000.xml" nts:in-targets="CheckContent"> |
There was a problem hiding this comment.
Ging deze content assert mis? Moet dit in commentaar blijven?
There was a problem hiding this comment.
ja dit gaf foutmeldingen tijdens de build output, waardoor de build niet goed verliep. Pieter heeft meegekeken naar waar het probleem zit. Dit is een fixture waar we twee codes hebben, en het probleem was dat er twee keer contentAsserts werden gemaakt voor die fixture. Een paar regels verder worden ze ook gemaakt, met de andere code. Dus de oplossing was om een van de twee te verwijderen. Ik heb de regels nu volledig verwijderd, ipv in comments gehouden
There was a problem hiding this comment.
Hm, uiteindelijk moeten er alleen wel content asserts komen... Dus dat moeten we dan later nog wel fixen.
There was a problem hiding this comment.
Volgens Pieter hoef je dit alleen één keer te doen, en zijn de twee keren overbodig
feat(NICTIZ-29502) Add organization and ReferralRequest fixture to Echo casus 1