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

Testgevallen in getting started is foutief #50

Open
melsk-r opened this issue Jun 26, 2024 · 1 comment
Open

Testgevallen in getting started is foutief #50

melsk-r opened this issue Jun 26, 2024 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@melsk-r
Copy link
Collaborator

melsk-r commented Jun 26, 2024

Originally created by KayodeBakker (kadaster/BRK-bevragen#815):

Bij de getting started onder de testgevallen voor kadastraal onroerende zaken gevonden op deze pagina staan kadastrale aanduidingen die niet bruikbaar zijn binnen de API.

Enkele voorbeelden van een foutieve kadastrale aanduiding zijn:
Kadastrale identificatie: 22310827210003
Kadastrale aanduiding: 's Gravenhage:881 N 8272 3

In de getting started staat dus "'s Gravenhage:881 N 8272 3" welke niet wordt geaccepteerd in de API. Als er wordt gezocht op de identificatie dan toont het resultaat een kadastrale aanduiding met waarde "'s-Gravenhage N 8272 A3".
Hieruit is dus op te halen dat bij de testgevallen:

  • de streep bij plaatsnamen als 's-Gravenhage wordt vergeten
  • de code voor woonplaats binnen het Kadaster met dubbele punt wordt getoond terwijl deze ongewenst is
  • het index letter wordt weg gelaten in geval van een appartement onroerende zaak

Gebruikers worden via de hoofdpagina van de GitHub geleid naar deze testgevallen, maar bij binnenkomst en eerste pogingen tot het werkende krijgen van de API zullen ze tegen dit soort fouten aanlopen.

Mijn voorstel is om alle testgevallen nogmaals na te lopen en desnoods zelf uit te voeren om zeker te zijn dat wat we tonen inderdaad correct en bruikbaar is.

@melsk-r melsk-r added the help wanted Extra attention is needed label Jun 26, 2024
@melsk-r
Copy link
Collaborator Author

melsk-r commented Jun 26, 2024

This comment originally might have been created by someone else.

@KayodeBakker: dit is aangepast in volgend pull request dat klaar staat: kadaster/BRK-bevragen#821

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant