Ci/#10 scenarios expressief documenteren #97

nielskorporaal · 2024-09-18T06:54:24Z

No description provided.

…p persoon

… cleanup

…n-stepdefs-ouder.js

…s-expressief.js

…de BRP

…ouder 1 en ouder 2

…id met ouder 1 en 2 gehuwd

…rtner informatie

…oegd

fsamwel · 2024-09-24T13:40:43Z

features/docs/gegeven-stap-definities-expressief.feature

+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,P,000000012,19940923,Voorbeeld |
+      |      | kind-1       | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum) VALUES($1,$2,$3,$4,$5,$6)                    | 9999,0,0,K,000000036,20070923           |
+      | 2    | ouder-1-1    | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,1,000000012,19940923,Voorbeeld |
+      |      | inschrijving | INSERT INTO public.lo3_pl(pl_id,mutatie_dt,geheim_ind) VALUES((SELECT COALESCE(MAX(pl_id), 0)+1 FROM public.lo3_pl),current_timestamp,$1) RETURNING * | 0                                       |


deze moet denk ik naar boven, want de pl_id die hieruit komt heb je nodig voor een aantal inserts

inschrijving is verplaatst naar boven

fsamwel · 2024-09-24T13:41:34Z

features/docs/gegeven-stap-definities-expressief.feature

+      | stap | categorie    | text                                                                                                                                                  | values                                  |
+      | 1    | inschrijving | INSERT INTO public.lo3_pl(pl_id,mutatie_dt,geheim_ind) VALUES((SELECT COALESCE(MAX(pl_id), 0)+1 FROM public.lo3_pl),current_timestamp,$1) RETURNING * | 0                                       |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,voor_naam) VALUES($1,$2,$3,$4,$5,$6)                         | 9999,0,0,P,000000036,P1                 |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,P,000000012,19940923,Voorbeeld |


dit is de tweede persoon die je toevoegt, dus moet een andere pl_id krijgen: 10000

voor 1e persoon wordt nu pl_id 9999 gehanteerd, 2e persoon krijgt pl_id 10000 en evt 3e persoon pl_id 10001

fsamwel · 2024-09-24T13:43:51Z

features/docs/gegeven-stap-definities-expressief.feature

+      | stap | categorie    | text                                                                                                                                                  | values                                  |
+      | 1    | inschrijving | INSERT INTO public.lo3_pl(pl_id,mutatie_dt,geheim_ind) VALUES((SELECT COALESCE(MAX(pl_id), 0)+1 FROM public.lo3_pl),current_timestamp,$1) RETURNING * | 0                                       |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,voor_naam) VALUES($1,$2,$3,$4,$5,$6)                         | 9999,0,0,P,000000036,P1                 |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,P,000000012,19940923,Voorbeeld |


ik denk dat het handig is dat de ouder als geslachtsnaam de naam krijgt die je opgeeft, in dit geval 'P2'

gewijzigd in stap-definities, aanduiding wordt bij ouder ingevuld als geslachtsnaam

fsamwel · 2024-09-24T13:56:38Z

features/docs/gegeven-stap-definities-expressief.feature

+      | 1    | inschrijving | INSERT INTO public.lo3_pl(pl_id,mutatie_dt,geheim_ind) VALUES((SELECT COALESCE(MAX(pl_id), 0)+1 FROM public.lo3_pl),current_timestamp,$1) RETURNING * | 0                                       |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,voor_naam) VALUES($1,$2,$3,$4,$5,$6)                         | 9999,0,0,P,000000036,P1                 |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,P,000000012,19940923,Voorbeeld |
+      |      | kind-1       | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum) VALUES($1,$2,$3,$4,$5,$6)                    | 9999,0,0,K,000000036,20070923           |


ik denk dat je hier de gegevens van de persoon moet overnemen. Dus in dit voorbeeld voornaam=P1, geboortedatum=null.
Wanneer er eerder een stap "is minderjarig" of "is meerderjarig" is opgegeven zou wel die geboortedatum gevuld worden

step_definition hiervan is aangepast zodat geboortedatum leeg blijft indien niet vermeld

fsamwel · 2024-09-24T13:57:20Z

features/docs/gegeven-stap-definities-expressief.feature

+      | 1    | inschrijving | INSERT INTO public.lo3_pl(pl_id,mutatie_dt,geheim_ind) VALUES((SELECT COALESCE(MAX(pl_id), 0)+1 FROM public.lo3_pl),current_timestamp,$1) RETURNING * | 0                                       |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,voor_naam) VALUES($1,$2,$3,$4,$5,$6)                         | 9999,0,0,P,000000036,P1                 |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,P,000000012,19940923,Voorbeeld |
+      |      | kind-1       | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum) VALUES($1,$2,$3,$4,$5,$6)                    | 9999,0,0,K,000000036,20070923           |


de pl_id moet horen bij de ouder P1. dat is de tweede persoon die is opgevoerd, dus pl_id=10000

pl_id aangepast zodat het overeenkomt met de tweede persoon

fsamwel · 2024-09-24T13:58:34Z

features/docs/gegeven-stap-definities-expressief.feature

+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,voor_naam) VALUES($1,$2,$3,$4,$5,$6)                         | 9999,0,0,P,000000036,P1                 |
+      |      | persoon      | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,P,000000012,19940923,Voorbeeld |
+      |      | kind-1       | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum) VALUES($1,$2,$3,$4,$5,$6)                    | 9999,0,0,K,000000036,20070923           |
+      | 2    | ouder-1-1    | INSERT INTO public.lo3_pl_persoon(pl_id,stapel_nr,volg_nr,persoon_type,burger_service_nr,geboorte_datum,geslachts_naam) VALUES($1,$2,$3,$4,$5,$6,$7)  | 9999,0,0,1,000000012,19940923,Voorbeeld |


zie eerdere opmerking: ik denk dat het handig is dat de ouder als geslachtsnaam de naam krijgt die je opgeeft, in dit geval 'P2' (in plaats van 'Voorbeeld')

stap-definitie aangepast: ipv 'Voorbeeld' word de aanduiding als geslachtsnaam gebruikt

… inschrijving pl_id fix

…s hetzelfde is

…thub.com/BRP-API/brp-api-gezag into ci/#10-scenarios-expressief-documenteren

…lidatie toegevoegd

…thub.com/BRP-API/brp-api-gezag into ci/#10-scenarios-expressief-documenteren

fsamwel · 2024-10-10T16:22:17Z

aangezien adoptie nu wel zo'n beetje goed is heb ik de volgende feature toegevoegd: recente gebeurtenis hertrouwen.

In het eerste scenario (features/gezag-minderjarige/3.1-recente-gebeurtenis-hertrouwen.feature:68) gaat er iets niet helemaal goed.
De minderjarige 'Jaimy' wordt toegevoegd met pl_id=3 en vervolgens nog een keer met pl_id=6. Alleen bij pl_id=6 zijn de ouders toegevoegd.

fsamwel · 2024-10-11T08:05:21Z

aangezien adoptie nu wel zo'n beetje goed is heb ik de volgende feature toegevoegd: recente gebeurtenis hertrouwen.

In het eerste scenario (features/gezag-minderjarige/3.1-recente-gebeurtenis-hertrouwen.feature:68) gaat er iets niet helemaal goed. De minderjarige 'Jaimy' wordt toegevoegd met pl_id=3 en vervolgens nog een keer met pl_id=6. Alleen bij pl_id=6 zijn de ouders toegevoegd.

ik heb hiervoor issue #131 gemaakt, zodat deze PR gewoon gemerged kan worden

fsamwel · 2024-10-11T13:07:19Z

ik heb de features ook weggehaald zodat deze PR zonder features bestanden kan worden gemerged

nielskorporaal added 12 commits September 2, 2024 16:47

ci(createpersoonmetaanduiding): impl Gegeven persoon met aanduiding

1a13252

ci(step_definitions): aanduiding optioneel en BRP inschrijving

adb6285

ci(step_definitions): impl Gegeven stappen Persoon

bf1e4f3

ci(step_definitions): impl optionele aanduiding, dataTable Gegevensta…

c46080f

…p persoon

ci(step_definitions): impl BRP en RNI inschrijving

3982d4d

ci(step_definitions): removed unused createPersoonMetAanduiding

d62287b

ci(gegeven-stepdefs): scenarios expressief documenteren

cd058dd

ci(persoon step_definitions): aanduiding als voornaam hanteren + code…

35afa6f

… cleanup

ci(stepdefs partner): datum huwelijkssluiting gisteren - 2 jaar

761ee58

ci(step_definitions): gegevens ouder gewijzigd verplaatst naar gegeve…

b699aed

…n-stepdefs-ouder.js

ci(gegeven-stepdefs-expressief): gegeven-stappen aangevuld

69dd120

Merge branch 'main' into ci/#10-scenarios-expressief-documenteren

bae6da4

nielskorporaal requested a review from MelvLee September 18, 2024 07:10

nielskorporaal added 9 commits September 18, 2024 09:28

ci(step_definitions): moved verblijfplaats changes to gegeven-stepdef…

1fda4b0

…s-expressief.js

docs(docs/gegeven-stap-definities): toegevoegd: * is ingeschreven in …

c9662d7

…de BRP

docs(docs/stap-definities): toegevoegd: * is in Nederland geboren en …

107a50e

…ouder 1 en ouder 2

docs(stap-definities-expressief): stap-definities-expressief uitgebre…

fd26bc8

…id met ouder 1 en 2 gehuwd

revert unused parameter

d9b9f29

docs(stap-definities-expressief): toegevoegd: kind-1, ouder-1-1 en pa…

349d868

…rtner informatie

refactor(persoon.js): format persoon.js en refactor gegeven-stap ouder

ec06ac7

docs(stap-definities-expressief): kind, ouders en inschrijving toegev…

5dbcdca

…oegd

Merge branch 'main' into ci/#10-scenarios-expressief-documenteren

eb797e8

nielskorporaal requested a review from fsamwel September 24, 2024 13:18

fsamwel reviewed Sep 24, 2024

View reviewed changes

chore(stap-definities-expressief): geboortedatum leeg indien vermeld,…

b593445

… inschrijving pl_id fix

FrozenSync marked this pull request as draft October 10, 2024 07:51

nielskorporaal and others added 13 commits October 10, 2024 11:17

chore(docs): linting en toepassen nieuwe stijl

654a7ec

chore(docs): gegevens gecorrigeerd nieuwe stijl

d4490e4

chore(stap-definities): gegevens gewijzigd nieuwe stijl

2f98cf6

chore(automation): log queries op level info toegevoegd

b7d53c9

correctie voorbeeldparameters zodat het in de verschillende scenario'…

ebe0aa2

…s hetzelfde is

Merge branch 'main' into ci/#10-scenarios-expressief-documenteren

84051bd

Merge branch 'ci/#10-scenarios-expressief-documenteren' of https://gi…

aa4cba5

…thub.com/BRP-API/brp-api-gezag into ci/#10-scenarios-expressief-documenteren

chore(valideer functionele specs): gezag minderjarige automatische va…

800c448

…lidatie toegevoegd

Merge branch 'main' into ci/#10-scenarios-expressief-documenteren

529d5f2

chore(helpers): toggle cleanup

370b9a9

Merge branch 'ci/#10-scenarios-expressief-documenteren' of https://gi…

b00c5aa

…thub.com/BRP-API/brp-api-gezag into ci/#10-scenarios-expressief-documenteren

chore(stepdefs): provide sql context

4601043

chore(stepdefs): verwijder individuele records

ca0fbb9

fsamwel mentioned this pull request Oct 10, 2024

aktenummer wordt gebruikt ondanks indicatie onjuist #129

Closed

fsamwel added 2 commits October 10, 2024 16:28

irrelevant scenario verwijderd

d2c7231

correctie toelichtingen bij gezag niet te bepalen door onderzoek

7326143

fsamwel added a commit that referenced this pull request Oct 10, 2024

aangepast op basis van recente aanpassingen in #97

65e77c7

feature hertrouwen toegevoegd

a27b567

features verwijderen uit de branch

3d5f09a

nielskorporaal marked this pull request as ready for review October 11, 2024 14:07

fsamwel mentioned this pull request Oct 14, 2024

vreemde waarde in toelichting gezag niet te bepalen bij ontbrekend gegeven #135

Closed

nielskorporaal added 3 commits October 16, 2024 15:22

chore(automation): staat onder curatele toegevoegd

e16c3fc

chore(automation): overlijden toegevoegd

a6281e1

Merge branch 'main' into ci/#10-scenarios-expressief-documenteren

3af84de

nielskorporaal merged commit 043a6e7 into main Oct 17, 2024
1 check passed

nielskorporaal deleted the ci/#10-scenarios-expressief-documenteren branch October 17, 2024 08:25

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ci/#10 scenarios expressief documenteren #97

Ci/#10 scenarios expressief documenteren #97

nielskorporaal commented Sep 18, 2024

fsamwel Sep 24, 2024 •

edited

Loading

nielskorporaal Sep 24, 2024

fsamwel Sep 24, 2024

nielskorporaal Sep 24, 2024

fsamwel Sep 24, 2024

nielskorporaal Sep 24, 2024

fsamwel Sep 24, 2024

nielskorporaal Sep 24, 2024

fsamwel Sep 24, 2024

nielskorporaal Sep 24, 2024

fsamwel Sep 24, 2024

nielskorporaal Sep 24, 2024

fsamwel commented Oct 10, 2024

fsamwel commented Oct 11, 2024 •

edited

Loading

fsamwel commented Oct 11, 2024

Ci/#10 scenarios expressief documenteren #97

Ci/#10 scenarios expressief documenteren #97

Conversation

nielskorporaal commented Sep 18, 2024

fsamwel Sep 24, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fsamwel commented Oct 10, 2024

fsamwel commented Oct 11, 2024 • edited Loading

fsamwel commented Oct 11, 2024

fsamwel Sep 24, 2024 •

edited

Loading

fsamwel commented Oct 11, 2024 •

edited

Loading