Leerpad 07Les 6 / 8

Review een Taiga-levering met bewijs

Verbind initiatief, plan, run, diff en controles. Controleer de huidige wijziging voordat u een merge- of releasebeslissing accepteert.

Praktijk11 minGereviewd

Gepubliceerd door Hoe we schrijven

Wat u leert

  • Herleid geleverd gedrag tot de eis en het plan.
  • Herken onvolledige controles en aannames die review vereisen.
  • Onderscheid afgeronde runs, merges, deployments en beschikbaarheid voor gebruikers.

Begin met het resultaat van het initiatief

De fictieve materiaalservice laat medewerkers nu hun eigen aanvragen bekijken. Begin de review met het resultaat en de scope van het initiatief. Bepaal wat waar moet zijn en wat de wijziging intact moet laten.

Bij deze levering mag een medewerker de aanvraag van een andere medewerker niet lezen. Managers moeten hun vastgelegde toegang behouden. Een test die alleen de pagina opent, toont geen van beide voorwaarden aan.

Verbind de registraties

RegistratieReviewvraag
InitiatiefWelk resultaat en welke scope zijn geautoriseerd?
PlanversieWelke implementatie- en verificatiestappen waren bedoeld?
RunWat gebeurde en welke aannames maakte de agent?
Pull request en diffWat veranderde in de huidige commit?
Controles en reviewWelk bewijs ondersteunt acceptatie van die commit?
DeploymentregistratieWelk artefact bereikte welke omgeving?

De pagina Runs registreert pogingen, inclusief mislukte. Elke run identificeert het uitgevoerde plan. Een runpagina is een inspectieregistratie; beslissingen die het werk veranderen horen bij het initiatief.

Lees het bewijs per stap voor tests en opmaak. Taiga maakt mislukte of niet-uitgevoerde controles zichtbaar. Verander ‘niet uitgevoerd’ in uw reviewsamenvatting niet in ‘geslaagd’.

Bekijk aannames en grenzen

Zoek aannames over het toegangsmodel, schema, de omgeving en externe services. Vergelijk ze met de gepubliceerde intentie en de werkelijke code.

Controleer bij de materiaalservice waar de eigenaar van een aanvraag wordt gecontroleerd. Test een geautoriseerde aanvraag, een aanvraag van een andere medewerker en een ontbrekende aanvraag. Controleer of logs geen vertrouwelijke aanvraaginhoud onthullen.

Review ook testwijzigingen. Een geslaagde test heeft beperkte waarde als de wijziging de assertie verwijdert die het defect zou detecteren. Neem wijzigingen in workflows en testconfiguratie mee in de reviewscope.

Geef bruikbare feedback

Benoem het gedrag, het verwachte resultaat en het benodigde bewijs. Bijvoorbeeld: ‘Het endpoint controleert inloggen, maar niet wie de aanvraag bezit. Voeg de toegangscontrole aan de serverkant toe en een test met de aanvraag van een andere medewerker.’

Taiga kan op reviewfeedback en falende controles reageren met wijzigingen op dezelfde branch. Bekijk na updates de nieuwe commit en de controles. Eerder bewijs dekt mogelijk een gewijzigd artefact niet.

Als de run stopte omdat het plan onvolledig was of een controle werd verzwakt, lees dan de opgegeven reden. Verwijder de conceptstatus niet alleen omdat het zichtbare controleoverzicht groen is.

Neem de juiste acceptatiebeslissing

Leg vast welke criteria geverifieerd zijn en welke onbeantwoord blijven. Laat verplichte repositoryreviews en controles de mergegrens afdwingen. Behoud een eventuele afzonderlijke releasebeslissing.

Taiga observeert deployments die uw pipeline uitvoert. Controleer omgeving en artefact voordat u gebruikers vertelt dat de wijziging beschikbaar is. Een mislukte deployment kan de vorige geslaagde versie verkeer laten blijven bedienen.

Rond het resultaat af met een servicecontrole: de medewerker kan de functie gebruiken, onbevoegde toegang wordt geweigerd en de beheerverantwoordelijke kan fouten waarnemen. Ga verder met een onderbreking afhandelen.

Maak de oefening

Een fictieve wijziging in medewerkerstoegang heeft een geslaagde build en een runnotitie dat een integratietest niet kon worden uitgevoerd. Beschrijf welk bewijs nodig is vóór acceptatie. Neem één geval van geweigerde toegang op en het exacte artefact of de commit onder review.

Werkblad downloaden (Markdown)

Controleer uw begrip

De run is afgerond, maar de registratie zegt dat een verplichte test niet draaide. Wat toont afronding aan?

Bronnen en verder lezen

Gerelateerd leesmateriaal van Taiga