search

VNG-Realisatie / Haal-Centraal-BRK-bevragen

Project repository tbv de ontwikkeling van de Haal Centraal BRK Bevragen API
https://vng-realisatie.github.io/Haal-Centraal-BRK-bevragen/
Other
19 stars 15 forks source link

Stuk testgevallen #916

Open KayodeBakker opened 2 years ago

KayodeBakker commented 2 years ago

Kijkende naar de 'Getting Started' zijn er geen testgevallen om Stuk mee te testen. Ook staat er geen informatie bij de andere gevallen of deze een stuk bevatten of niet.

Hierdoor kan een afnemer niet testen of de implementatie naar behoren functioneert. Graag zien we dat de 'Getting Started' worden uitgebreid met testgevallen hiervoor.

CathyDingemanse commented 2 years ago

@fsamwel, kun jij hier nog naar kijken?

fsamwel commented 2 years ago

@KayodeBakker wat voor soort testgevallen zou je hier verwachten?

Voor de genoemde testgevallen kan je zien of en hoeveel stukken er zijn in https://github.com/VNG-Realisatie/Haal-Centraal-BRK-bevragen/tree/master/test/cases

KayodeBakker commented 2 years ago

@fsamwel Duidelijk dat deze ook daar te vinden zijn. Ik had alleen bij de 'Getting Started' gekeken omdat dit voor mij de enige bekende locatie is.

Persoonlijk vind ik het eigenaardig dat we een tabel aan data tonen in 'Getting Started' maar ook een boel csv's hebben staan. Hoe had een afnemer moeten weten van het bestaan van deze csv's als de 'Getting Started' direct al test gevallen laat die ook up to date lijken (muv stuk en stukdeel). Hier staat immers ook geen referentie daar naar. Los van uit eigen initiatief de files op GitHub nalopen, zie ik niet hoe iemand hier achter had moeten komen.

Onder het mom van laagdrempelig maken, zou ik dat dan veranderen. Ervoor zorgen dat het makkelijk vindbaar is, door bijv. hiernaar te refereren.

fsamwel commented 2 years ago

Ervoor zorgen dat het makkelijk vindbaar is, door bijv. hiernaar te refereren.

Al gedaan :)

Zie pull request #918