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

Voeg tweede locatie voor extract_datum metadata toe + sta meerdere ru… #260

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Bouwdie
Copy link

@Bouwdie Bouwdie commented Dec 5, 2018

…ns van mutatie levering xml toe.

Mutatie levering xml zijn anders gevuld dan extract lever bestanden;
hiervoor is een tweede xpath toegevoegd (schema: http://www.kadaster.nl/schemas/bag-verstrekkingen/extract-levering/v20090901/BagvsExtractLeveringsdocument-1.3.xsd).
De mutatie levering xml bestand moet ook vaker gerunned kunnen worden, aangezien deze worden meegeleverd in iedere mutatie zip.
Deze case is wordt nu ge-exclude in de run-once check.

Fixes: #258

…ns van mutatie levering xml toe.

Mutatie levering xml zijn anders gevuld dan extract lever bestanden;
hiervoor is een tweede xpath toegevoegd (schema: http://www.kadaster.nl/schemas/bag-verstrekkingen/extract-levering/v20090901/BagvsExtractLeveringsdocument-1.3.xsd).
De mutatie levering xml bestand moet ook vaker gerunned kunnen worden, aangezien deze worden meegeleverd in iedere mutatie zip.
Deze case is wordt nu ge-exclude in de run-once check.
@Bouwdie
Copy link
Author

Bouwdie commented Dec 5, 2018

Het project heeft geen makkelijk te testen structuur; als ik dit testbaar wil krijgen moet ik behoorlijk wat omgooien. Suggesties zijn welkom!

@justb4
Copy link
Contributor

justb4 commented Dec 6, 2018

tests: ja klopt,is historisch ontstaan...Voor nu: er is een "test" map met daaronder bijv voor mutaties: https://github.com/nlextract/NLExtract/tree/master/bag/test/mutatie . Daar kunnen mutatie-bestanden geplaatst. Deze worden verwerkt via runtests.sh. Je kunt evt aparte map maken en extractie aanroep in runtests.sh voor mutaties volgens Levering v1.3.

Runnen: ./runtests.sh > runtests.log 2>&1 en log en DB inspecteren. Dit is het enige nu voorhanden qua tests. Wanneer we in volgende NLExtract BAGv2 versie Stetl gebruiken hebben we meer mogelijkheden. Stetl heeft zelf ook unittest suite.

@justb4
Copy link
Contributor

justb4 commented Dec 6, 2018

Ok, ik zie dat er wel een leverings mutatie xml is toegevoegd in PR. Is dus de feitelijke mutatie-xml niet gewijzigd qua structuur? Of is er een apart Leverings doc bijgekomen? NB je kunt ook mutatie--zips plaatsen onder de map test/muts.

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