BRP-API / Haal-Centraal-Reisdocumenten-bevragen

https://brp-api.github.io/Haal-Centraal-Reisdocumenten-bevragen/
Other
3 stars 3 forks source link

Lijst van teleurstellingen #90

Open KayodeBakker opened 1 year ago

KayodeBakker commented 1 year ago

Toegevoegd 17-04-2023

Proxy verwachtingen voor datum velden in onderzoek.
Verwacht verkeerde BSN
Regel 117 | >veld 2> | <waarde 2> | . <veld 2> had het moeten zijn.
Undefined bij inserts
De volgende feature mist achtergrond gegevens voor inhoudingOfVermissing waarschijnlijk door verzoek van Kayo initieel

Toegevoegd 21-04-2023

Issues bij geheimhouding inserts
Onduidelijke rule

Toegevoegd 02-05-2023

Dit is een bevraging op bsn niet reisdocumentnummer
'010120,' is geen rubriek. Een komma bij de rubrieknummer.
Achtergrond en scenario matchen niet.
fsamwel commented 1 year ago

'010120,' is geen rubriek. Een komma bij de rubrieknummer.

Scenario: Zoek reisdocumenten op burgerservicenummer # features\zoek-met-burgerservicenummer\gba\protocollering-gba.feature:26 AssertionError: 080910 !== 010120, 080910: expected [ '080910' ] to have the same members as [ '010120,', '080910' ]

@KayodeBakker deze begrijp ik niet. AssertionError: 080910, 123520 !== 010120, 080910: expected [ '080910,', '123520' ] to have the same members as [ '010120,', '080910' ]

Blijkbaar lever je hier 123520 als een van de zoekrubrieken. Dat is het rubrieknummer van een reisdocumentnummer. Maar hier is ZoekMetBurgerservicenummer gebruikt, dus wordt er gezocht op burgerservicenummer (010120) en gemeenteVanInschrijving (080910)

Hoe bedoel je komma in rubrieknummer? Zoekrubrieken is een komma-spatie gescheiden lijst, dus volgens mij staat dat er goed. Zie ook hoe de zoekrubrieken in de personen bevragen protocolleringsfeatures beschreven zijn

KayodeBakker commented 1 year ago

@fsamwel Ik krijg jouw resultaat voorbeeld niet terug in mijn resultaten overzicht. Ik leverde '010120,', '080910'. Ik snapte überhaupt al niet waarom 123520 terug kwam als verwachting (met andere woorden, ik leverde het niet maar het werd verwacht door de scenario gek genoeg.)

Wat betreft de komma, is het vrij makkelijk te zien in jouw eigen voorbeeld. De 3de item bevat een komma tussen de quotes, namelijk "010120,". Dit impliceert dat de volgende resultaat wordt verwacht: [ '010120,', '080910' ]. Zie je hoe er 1 komma tussen twee strings staan en 1 binnen 1 string? Dat is het verschil en dat is een foutieve verwachting. Nu verwacht de scenario een rubriek met een komma geplakt aan de nummers.

Aanvullend hierop blijkt dat de protocollering scenario nu werkt dus ik weet niet wat er is veranderd eerlijk gezegd. @MelvLee Heb jij een aanpassing gemaakt in de step_defs rond 2 mei of daarna?

fsamwel commented 1 year ago

@KayodeBakker in een vorige versie leverde de API 123520, en de verwachting was 010120. In de laatste versie van de API is dat opgelost en slaagt de feature nu