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

Als tester van een API client wil ik volledige dekking in de testdata #35

Open
melsk-r opened this issue Jan 17, 2025 · 15 comments
Open

Comments

@melsk-r
Copy link
Collaborator

melsk-r commented Jan 17, 2025

Originally created by fsamwel (kadaster/WOZ-bevragen#60):

zodat ik een garagelijst kan opstellen... zodat ik alle gegevens in de API kan gebruiken en de verwerking daarvan in mijn applicatie kan testen

Zie de tabel in /test/cases.

Velden waar nog geen waarde voor is:

  • aanduiding.locatieomschrijving
  • adresseerbaarObjectIdentificaties: >1 items
  • belanghebbendeEigenaar.geheimhoudingPersoonsgegevens: true
  • belanghebbendeEigenaar.kvkNummer
  • belanghebbendeEigenaar.vestigingsnummer
  • belanghebbendeEigenaar.type: vestiging
  • belanghebbendeGebruiker.geheimhoudingPersoonsgegevens: true
  • belanghebbendeGebruiker.burgerservicenummer
  • belanghebbendeGebruiker.kvkNummer
  • belanghebbendeGebruiker.vestigingsnummer
  • belanghebbendeGebruiker.type: natuurlijk_persoon en vestiging
@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

Testgevallen:
NPS: 000500055046
NNP: 000500030828
VES: 000500030843

Ligplaats: 800000200021
Standplaats: 800000200022
WOZ met meerdere KOZ en PND relaties: 000500000001

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ kad-verstp @ kad-tromps @ kad-reinda in /wozobjecten/800000051111 is belanghebbendeGebruiker dezelfde persoon als belanghebbendeEigenaar. Kan dat voorkomen?
Voor testdoeleinden lijkt me dat geen handige situatie. Je wilt als juist zien dat de juiste gegevens op de juiste plek getoond worden en dat zie je alleen als de gegevens verschillend zijn.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

er is nu een aantal testgevallen, zoals /wozobjecten/800000051111 zonder waarden. Kan dat voorkomen in productie, dat er geen enkele actuele waarde is?

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

  1. In het komt vaak voor dat een Eigenaar en Gebruiker hetzelfde zijn. Voor het testen is dan niet onderscheidend dus daar ga ik wel in mee zal het aanpassen.

  2. Het kan voorkomen in het echt dat er (nog) geen waarde is voor dat woz-object. Los daarvan heeft niet elke woz-object in de testdatabase heeft een waarde gekregen. De testdatabase is ook een door de jaren heen ontwikkelde verzameling aan data waarbij het toevoegen van een waarde niet nodig was. Voor het testen van HaalCentraal hoort er wel een waarde bij en wordt ook meegenomen bij het maken van nieuwe woz-objecten.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ kad-verstp in kadaster/WOZ-bevragen#60 (comment) noem je testgevallen Ligplaats: 800000200021 en Standplaats: 800000200022. Maar deze hebben geen adresseerbaarObjectIdentificaties.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

Standplaats 800000200022 heeft adresseerbaarObjectIdentificatie 8000032000000000
Ligplaats 800000200021 heeft adresseerbaarObjectIdentificatie 8000020000001115

maar wordt niet in de json weergegeven.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ kad-verstp

maar wordt niet in de json weergegeven.

Waarom wordt dit niet in de json weergegeven? Dit zou toch zichtbaar moeten zijn in adresseerbaarObjectIdentificaties? Het gaat hier toch om het adresseerbaar object waar het WOZ-object aan verbonden is?

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

Dan moet dit worden toegevoegd aan de functionaliteit. Maar bij Stand en Ligplaatsen gaat het meestal een beetje anders @ kad-tromps kan dit beter vertellen.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ fsamwel

In een van de posts hierboven zeg je:

... in /wozobjecten/800000051111 is belanghebbendeGebruiker dezelfde persoon als belanghebbendeEigenaar.

maar in de daaropvolgende post zeg je weer:

er is nu een aantal testgevallen, zoals /wozobjecten/800000051111 zonder waarden.

Lijkt me in tegenspraak met elkaar.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ melsk-r @ fsamwel

Wat is opgemerkt klopt wel de eigenaar en gebruiker is dezelfde niet natuurlijke persoon een object die een waarde heeft wordt getoond maar dus niet elk object in onze testdatabase heeft een waarde

Deze objecten zijn speciaal voor HaalCentraal gemaakt.
Met waarden:
800000003118
800000823525
800000014669
800000024029
800000027395
800000024901
800047970000
800027210003

Zonder waarden
800012345678

800000051111 heeft waarden met begindatum voor 2014 en wordt voortaan wel getoond.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

Excuses ik interpreteerde term 'waarden' verkeerd. Frank bedoelde in zijn opmerking:

er is nu een aantal testgevallen, zoals /wozobjecten/800000051111 zonder waarden. Kan dat voorkomen in productie, dat er geen enkele actuele waarde is?

natuurlijk de property 'waarden' en niet 'waarden' in de zin van 'values'.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ kad-tromps @ kad-verstp veel van de toegevoegde testgevallen bestaan nu wel op acceptatie (https://api.acceptatie.kadaster.nl/lvwoz/api/v1), maar niet op de eto (https://api.kadaster.nl/lvwoz-eto-apikey/api/v1"). Kan de testdata in de eto gelijk gemaakt worden aan acceptatie, zodat

  1. we ook daar onze tests kunnen uitvoeren
  2. aansluiters op de API straks ook voldoende testdata hebben om in hun client het gebruik van de API te testen

daarnaast heb ik op de acceptatie-omgeving nu enkele Woz-objecten niet meer, die wel in dit issue genoemd zijn:

  • /wozobjecten/000500030828
  • /wozobjecten/000500030843

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

qua testdata heb ik verder nog niet:

  • belanghebbendeGebruiker.kvkNummer
  • belanghebbendeGebruiker.vestigingsnummer
  • kadastraalOnroerendeZaakIdentificaties (kan geloof ik nog niet geleverd worden?)

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

@ fsamwel
/wozobjecten/000500030843 is geworden /wozobjecten/800000030843
/wozobjecten/000500030828 is geworden /wozobjecten/800000030828

met wozobjecten/800000030843 kan je filteren op belanghebbendeGebruiker.kvkNummer en belanghebbendeGebruiker.vestigingsnummer

kadastraalOnroerendeZaakIdentificaties (kan geloof ik nog niet geleverd worden?): Als deze er is dan wordt het geleverd volgens mij.

@melsk-r
Copy link
Collaborator Author

melsk-r commented Jan 17, 2025

This comment originally might have been created by someone else.

Completeer de lijst onder /test/cases op basis van de lijst testgevallen die @ kad-verstp eerder in dit issue genoemd heeft. Als dit gedaan is controleer dan of de opmerkingen van @ fsamwel in de eerste comment geadresseerd zijn.

Robert pakt deze op.

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

No branches or pull requests

1 participant