Skip to content

Patch how to test#151

Merged
mrtn78 merged 7 commits intodevelopfrom
patch-how_to_test
Dec 13, 2024
Merged

Patch how to test#151
mrtn78 merged 7 commits intodevelopfrom
patch-how_to_test

Conversation

@mrtn78
Copy link
Collaborator

@mrtn78 mrtn78 commented Nov 28, 2024

Als voorbereiding op de behandeling van de standaard door het Forum Standaardisatie en de doorontwikkeling van developer.overheid zijn de deeplinks naar de testcode verwijdert en zijn de tests algemener beschreven. Dit geeft developer.overheid meer vrijheid om verschillende manieren van testen op te zetten en uit te werken.

Conform de geest van de design rules zijn de tests meer gericht op het testen van het design aan de hand van de Open API Specification, dan aan het testen van de server configuratie waarop de API draait. ook het onderscheid tussen het design en de implementatie is niet aan ons om te testen. vandaar de focus op het design en de OAS.

Deze wijzigingen zijn ingegeven door de issues van @dvh zoals gebundeld in https://github.com/Geonovum/KP-APIs/milestone/4

De wijzigingen betreffen GEEN inhoudelijke wijzigingen van de standaard, het is slechts een aanpassing in verwijzingen naar externe bronnen en informatieve manieren om de regels van de standaard te kunnen testen.

@mrtn78 mrtn78 added Scope: Klein Kleine wijzigingen met beperkte scope Status: Klaar voor release Het voorstel is verwerkt en klaar voor de volgende release. Type: Documentatie Tekstueele wijziging op de documentatie. Overleg: TO-API Te agenderen voor het Technisch Overleg API labels Nov 28, 2024
@mrtn78 mrtn78 self-assigned this Nov 28, 2024
@dvh
Copy link
Collaborator

dvh commented Dec 2, 2024

Voor de besluitvorming wel goed om ook te vermelden dat dit niet enkel voor developer.overheid.nl is gedaan:

  • "How to test" zou moeten beschrijven hoe je kunt testen of iets aan een regel voldoet. Dit staat los van de manier waarop je dit doet; dit kan handmatig, via eigen tools, onafhankelijke validators, etc. Omdat deze allemaal niet in beheer zijn van de organisatie die de standaard zelf beheert is een harde verwijzing in de standaard een slecht idee.
  • Omdat "OpenAPI Specification" op zichzelf een verplichte standaard is, is het logisch dat bij het valideren van API Design Rules uitgegaan mag worden van de aanwezigheid van een OpenAPI Specification. Sterker nog, één van de argumenten om een OpenAPI Specification verplicht te stellen was júist dat je op een machine leesbaar formaat de scope en het gedrag van de API uitdrukt, voor bijvoorbeeld linters of validators.
  • Een aantal testcases klopten sowieso niet, of schoten het doel van de regel zelf voorbij waardoor het onnodig complex was voor zowel de validerende partij als de API provider.

Copy link
Member

@TheBonheurs TheBonheurs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Buiten de opmerkingen van Joost en Dimitri heb ik verder geen op of aanmerkingen en kan deze gemerged worden.

@mrtn78
Copy link
Collaborator Author

mrtn78 commented Dec 13, 2024

  • "How to test" zou moeten beschrijven hoe je kunt testen of iets aan een regel voldoet. Dit staat los van de manier waarop je dit doet; dit kan handmatig, via eigen tools, onafhankelijke validators, etc. Omdat deze allemaal niet in beheer zijn van de organisatie die de standaard zelf beheert is een harde verwijzing in de standaard een slecht idee.

Ik heb de tekst in de implication aangepast zodatduidelijk is dat het gaat om "an example of the test is included in the automatic tests..."

aangescherpt dat het om een example gaat "an example of the test is included"
@mrtn78 mrtn78 merged commit 213894c into develop Dec 13, 2024
1 check passed
@mrtn78 mrtn78 deleted the patch-how_to_test branch December 13, 2024 15:18
@sanderke sanderke mentioned this pull request Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Overleg: TO-API Te agenderen voor het Technisch Overleg API Scope: Klein Kleine wijzigingen met beperkte scope Status: Klaar voor release Het voorstel is verwerkt en klaar voor de volgende release. Type: Documentatie Tekstueele wijziging op de documentatie.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants