VNG-Realisatie / api-test-platform

6 stars 3 forks source link

Als provider wil ik testruns kunnen bewerken #314

Closed sergei-maertens closed 4 years ago

sergei-maertens commented 4 years ago

...zodat ik de configuratie kan aanpassen als ik fouten maak.

Toelichting: Ik heb via ZGW APIs > Provider > Start testrun een provider test aangemaakt en een aantal URLs vergeten aan te passen. Ook zorgen de geconfigureerde secret voor server errors, wat ik door een ander secret wil aanpassen, maar dat is niet mogelijk.

Oplossingsrichting:

Edit knop om het formulier weer boven te halen :-)

Definition of ready

Definition of done algemeen

Definition of done specifiek voor deze user story

Acceptatiecriteria

Taken Taken worden apart uitgewerkt in issues.

alextreme commented 4 years ago

Lijkt mij inderdaad een vrij essentiele onderdeel van het test-platform (en maakt het gebruik een stuk eenvoudiger).

Aan @HenriKorver om aan te geven of dit onderdeel dient te worden van de volgende sprint.

HenriKorver commented 4 years ago

Terechte wens. Ik heb m in de sprint gezet

alextreme commented 4 years ago

@stevenbal wees vrij om op te pakken en voor de zekerheid te dubbelchecken met Sergei

stevenbal commented 4 years ago

@alextreme deze feature staat nu op test, de update knop is rechtsboven zichtbaar als de user de benodigde permissie heeft: https://test.api-test.nl/server/1/2fdcb0ab-c624-4468-8aa9-787fae0d6ba9/06404121-75c7-459d-a01f-aecce724a495/

alextreme commented 4 years ago

@stevenbal

Verder prima.

stevenbal commented 4 years ago

@alextreme de reset knop zet alle variabelen die aangepast zijn in de form terug naar de originele waarde, of heb je liever een knop die je terug brengt naar de vorige pagina?

alextreme commented 4 years ago

@stevenbal het terug gaan naar de vorige pagina (afbreken van de edit) zou ik in dit geval eerder verwachten dan het terugzetten naar de originele waarden.

stevenbal commented 4 years ago

@alextreme ik heb het aangepast, staat nu op test

alextreme commented 4 years ago

@stevenbal Prima. Wees vrij om door te zetten naar staging.

alextreme commented 4 years ago

@HenriKorver de mogelijkheid om de environments te wijzigen zit inmiddels op staging en je hebt aangegeven om op productie te willen checken.

@stevenbal zie PR 254, we zitten nog wel met een merge conflict tussen staging en master.

stevenbal commented 4 years ago

@HenriKorver deze feature zit nu op productie

HenriKorver commented 4 years ago

Het lijkt nog niet helemaal goed te gaan, zie https://api-test.nl/server/2/66572b52-2e64-434b-8024-0d2cd3455543/

De variabele demo-url (https://demo.api-test.nl/api/v2 ) op de detailpagina is nog niet aangepast aan de waarde van de aangepaste environment (https://demo.api-test.nl/api/v3).

De badge op de detailpagina is groen, hoewel de test in deze aangepaste environment nog nooit is geslaagd. Het lijkt niet handig te zijn om zomaar een environment te kunnen aanpassen. Dus als een bestaande environment niet goed is ingevuld is het beter om de hele combinatie van testscenario en environment te verwijderen en een nieuwe aan te maken. Anders krijg je dit soort rare randeffecten die lijken op geschiedenisvervalsing. Het is wel handig om een dupliceer-button te hebben (of iets dergelijks) zodat je niet alle omgevingsvariabelen opnieuw hoeft in te tikken.

alextreme commented 4 years ago

Goed punt.

Het bij het aanpassen van de environment-variabelen automatisch verwijderen van de eerdere runs met de oude instellingen lijkt mij in dit geval het meest wenselijk: door het aanpassen van de variabelen 'invalidate' je de eerdere runs.

stevenbal commented 4 years ago

@alextreme Ik liep nog tegen een bug aan op productie, waardoor soms de volgorde van de endpoints overhoop gegooid wordt na het updaten van waardes Screenshot from 2019-11-29 12-44-48 Na een update Screenshot from 2019-11-29 12-46-50

Ik zal het fixen

stevenbal commented 4 years ago

Fix voor de ordening van de placeholders is doorgevoerd op staging @alextreme

alextreme commented 4 years ago

Wijzigen van bestaande environments kan getest worden op staging. Bij mij werkt dit nu zoals verwacht, wel is nog een authorisatie-probleem met mijn ZGW tokens maar dat is onderdeel van #238 en is gemeld aan Sergei