Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

(Scheduled) badge werkt niet #284

Closed
HenriKorver opened this issue Sep 21, 2019 · 10 comments
Closed

(Scheduled) badge werkt niet #284

HenriKorver opened this issue Sep 21, 2019 · 10 comments
Assignees
Labels
bug Something isn't working Prio ZH Prioriteit Zeer Hoog

Comments

@HenriKorver
Copy link
Contributor

HenriKorver commented Sep 21, 2019

Bug

Onderstaande scheduled badge afkomstig uit https://api-test.nl/server/719f6829-2987-4dea-a4b7-8e5c07072245/ komt niet uit op een testrapport als je erop klikt.

Status badge

De andere statische badge werkt wel.

@HenriKorver HenriKorver added bug Something isn't working Prio ZH Prioriteit Zeer Hoog labels Sep 21, 2019
@HenriKorver HenriKorver changed the title (Scheduled) badge voor BRP Bevragingen werkt niet (Scheduled) badge werkt niet Sep 21, 2019
@stevenbal
Copy link
Collaborator

dat zal wel door shields.io komen dan, want bij mij doen beide badges het in het linkje wat je stuurde

@alextreme
Copy link
Collaborator

@stevenbal waarschijnlijk heeft Henri de badge verwisseld of aangepast in de markdown.

Zou je de volgende markdown zoals getoond op de link hierboven kunnen dubbelchecken? Ik vermoed dat de spaties in de Test Scenario naam niet gaat werken en dat je hier een ID moet gaan gebruiken in je URLs:

[![Status badge](https://img.shields.io/endpoint.svg?style=for-the-badge&url=https%3A//api-test.nl/api/v1/provider-latest-badge/BRP%2520Basis%2520Ingeschreven%2520Persoon%2520API%2520tests/henri/)](https://api-test.nl/api/v1/provider-latest-badge/BRP%20Basis%20Ingeschreven%20Persoon%20API%20tests/henri/)

@stevenbal
Copy link
Collaborator

@alextreme ik denk dat het handiger is als ik deze aanpassing voor de laatste badge samen neem met de andere aanpassingen voor de provider test structuur, omdat het waarschijnlijk toch op de schop moet daarvoor

@stevenbal
Copy link
Collaborator

het endpoint gebruikt nu de uuid van het environment dat gebruikt is voor de provider run, die vastgezet is op een test scenario en een user (VNG-Realisatie/api-test-platform-code@6eaddd8)

@stevenbal
Copy link
Collaborator

@HenriKorver @alextreme het webinterface maakt nu onderscheid tussen scheduled provider runs en gewone provider runs. Stel ik heb een bepaald test scenario en een environment bij dat test scenario, ik heb een schedule ingesteld daarvoor en ik heb een gewone provider run uitgevoerd voor dat test scenario + environment: moeten de resultaten van de gewone provider runs ook verschijnen in de lijst met provider runs voor het scheduled scenario (en vice versa)? En moeten de scheduled scenarios en gewone provider runs dus dezelfde badge gebruiken of niet?

@alextreme
Copy link
Collaborator

Uitgangspunt was dat de 'one off' provider runs en de 'scheduled' provider runs gescheiden waren qua lijstweergave, dit aangezien er volgens mij andere gegevens per type run worden getoond en om het verschil duidelijk te maken.

Het is aan @HenriKorver om aan te geven mocht dat gecombineerd moeten worden (wellicht als we het onderscheid gaan maken qua lijstweergave per soort test scenario), maar daar moeten we het eerst over hebben.

Qua badges dienen deze visueel gelijk te zijn. De onderliggende werking zal (vermoed ik) anders blijven, maar ik weet ook dat hier behoorlijk wat aan gesleuteld is en dat eerst overleg hierover nodig is om door te kunnen.

@alextreme
Copy link
Collaborator

Zelfde probleem als met de andere scheduled issue ( #267 ).

Besproken en de wens is om de zaken te combineren binnen een environment (scheduled en one-off provider runs). De actie is om een environment voor een bepaalde API in te richten, en dan kan later bepaald worden of deze automatisch wordt getest of niet.

@alextreme
Copy link
Collaborator

@stevenbal nog even kijken naar de 'Custom badge - Resource not found' melding. Als er nog geen run is uitgevoerd dan dient de badge verborgen te zijn.

@stevenbal
Copy link
Collaborator

stevenbal commented Oct 17, 2019

Op test wordt de badge nu verborgen zolang er nog geen resultaten zijn
Screenshot from 2019-10-17 11-37-56

Ook op de provider run detail pagina
Screenshot from 2019-10-17 11-38-34

@stevenbal
Copy link
Collaborator

Het verbergen van de badges als er nog geen resultaten zijn is nu ook live op staging

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working Prio ZH Prioriteit Zeer Hoog
Projects
None yet
Development

No branches or pull requests

3 participants