VNG-Realisatie / api-test-platform

6 stars 3 forks source link

Consumer tests: Scenario cases aanvullen met een omschrijving #270

Closed alextreme closed 5 years ago

alextreme commented 5 years ago

Genoemd door @HenriKorver

Het zou prettig zijn om per scenario case een omschrijving van de case op te nemen via de admin.

Deze zou dan per case boven de case weergegeven worden op de session type detail-scherm en op de session report-scherm.

alextreme commented 5 years ago

Bij deze beschikbaar via het vraagtekentje per case:

https://staging.api-test.nl/consumer/84b567c3-a4fb-4b7c-bab3-796e37037dde/report/

https://staging.api-test.nl/admin/testsession/scenariocase/54/change/?_changelist_filters=collection__id__exact%3D16

HenriKorver commented 5 years ago

Vraagtekentje vind ik verwarrend. Men heeft de neiging erop te willen klikken in plaats van eroverheen te bewegen met de muis. Kan het click-event uitgeschakeld worden?

alextreme commented 5 years ago

Bij deze alleen zichtbaar on-hover:

https://staging.api-test.nl/consumer/84b567c3-a4fb-4b7c-bab3-796e37037dde/report/

HenriKorver commented 5 years ago

Bij nader inzien vind ik dit niet intuïtief werken. Kunnen jullie toch maar na de eerste kolom (http method) een extra kolom toevoegen (beschrijving) met de omschrijving van de method.

alextreme commented 5 years ago

Gechecked en werkt, echter bij het ontbreken van een omschrijving staat er nu 'None'.

https://test.api-test.nl/consumer/50b4740e-b0a6-4ce5-9d49-c66333d3a5bd/report/

Screenshot_2019-10-17_17-13-17

Ik zou tevens ervoor willen zorgen dat je de description-veld in de 'scenario case collection' admin inline erbij ziet staan:

Screenshot_2019-10-17_17-16-06

stevenbal commented 5 years ago

Description is nu een lege string i.p.v. None: Screenshot from 2019-10-21 09-52-49 https://test.api-test.nl/consumer/sessiontype/5/

En de description is toegevoegd aan de inline in ScenarioCaseCollection admin: Screenshot from 2019-10-21 09-53-41 https://test.api-test.nl/admin/testsession/scenariocasecollection/31/change/

stevenbal commented 5 years ago

Nu ook op staging: https://staging.api-test.nl/consumer/sessiontype/5/

https://staging.api-test.nl/admin/testsession/scenariocasecollection/21/change/

alextreme commented 5 years ago

Lijkt mij zo in orde, ter controle