search

VNG-Realisatie / api-test-platform

6 stars 3 forks source link

Als PO wil ik een extra autorisatie-niveau voor derden #312

Closed HenriKorver closed 4 years ago

HenriKorver commented 4 years ago

zodat zij zelf testscripts kunnen uploaden en configureren

Toelichting: Nu zijn API beheerders die gebruik willen maken van het API Testplatform afhankelijk van de admins van VNG Realisatie om een testomgeving aan te maken voor hun API('s). Het is de bedoeling dat ze dat zelf kunnen doen.

Oplossingsrichting:

Definition of ready

Definition of done algemeen

Definition of done specifiek voor deze user story

Acceptatiecriteria

Taken Taken worden apart uitgewerkt in issues.

stevenbal commented 4 years ago

De aanpassingen voor delete/update/create van test scenario's op API niveau zijn doorgevoerd naar test.api-test.nl. Ik heb ook een group gemaakt als voorbeeld voor beheerders van de ATP API

stevenbal commented 4 years ago

@alextreme zo te zien ben ik vergeten om een data migratie te doen voor het FileField met de postman test file, dus die ga ik nog toevoegen zodat niet alle postman tests opnieuw geupload hoeven te worden

alextreme commented 4 years ago

Zo te zien is de WYSIWYG-editor nog niet actief voor de description field:

Screenshot_2019-11-14_14-35-39

alextreme commented 4 years ago

When editting a test scenario, it's not clear that the Variables and Postman test script inline forms have an additional empty form by default at the bottom.

It might be better to not show the empty inline by default (extra=0 on the inline formset?)

Screenshot_2019-11-14_14-37-55

alextreme commented 4 years ago

@HenriKorver de extra autorisatie-niveau is toegevoegd op staging, echter alleen voor de provider test scenario's.

Is het ook de bedoeling dat derden voor hun API ook de consumer instellingen (session types, endpoints, scenario cases etc) kan configureren?

alextreme commented 4 years ago

Besproken. Wordt beperkt tot alleen de provider APIs kunnen beheren.

alextreme commented 4 years ago

Besproken. Het reviewen hiervan is mogelijk op staging.

Het geven van permissies is vorige week gedemonstreerd en werkt via de 'object permissions' van de API in de admin:

https://staging.api-test.nl/admin/servervalidation/api/

Daarna krijg je per API de mogelijkheid om beheer via de frontend uit te voeren:

"Configure test scenario's" knop rechtsbovenin op bijvoorbeeld:

https://staging.api-test.nl/server/2/

Graag controle of dit functioneel werkt zoals verwacht. Ik verwacht wel dat voor overdracht naar API Beheer wel een korte handleiding nodig zal zijn zodat zij weten hoe deze permissies werken.

HenriKorver commented 4 years ago

@alextreme Moet ik dit reviewen op staging? Ik zie namelijk dat ook al een groot deel van de functionaliteit voor het extra autorisatie-niveau uitgerold is naar productie

HenriKorver commented 4 years ago

Op productie zit ook zo'n "Configure test scenario's" knop, zie https://api-test.nl/server/2/

alextreme commented 4 years ago

@HenriKorver klopt, echter graag testen op staging voor de zekerheid.

HenriKorver commented 4 years ago

Lijkt allemaal goed te werken, waarvoor dank! Nog een klein puntje:

Gebruikers die geen super-user rechten hebben voor admin kunnen zelf geen API's toevoegen. Echter dit wordt wel gesuggereerd in de object-permissies

afbeelding

https://staging.api-test.nl/admin/servervalidation/api/2/permissions/