Dobavo Taiga preglejte z dokazili
DokončanoPovežite pobudo, načrt, zagon, diff in preverjanja. Pred sprejetjem odločitve o mergeu ali izdaji preverite trenutno spremembo.
Izdajatelj TaigaKako pišemo
Preverite razumevanjeZagon je dokončan, njegov zapis pa pravi, da zahtevani test ni bil izveden. Kaj dokazuje dokončanje?Opravite vajo
Kaj se boste naučili
- Dobavljenemu delovanju sledite nazaj do zahteve in načrta.
- Prepoznajte nepopolna preverjanja in predpostavke, ki potrebujejo pregled.
- Ločite zaključek zagona, merge, namestitev in dostopnost za uporabnike.
Začnite z rezultatom pobude
Izmišljena storitev opreme zdaj zaposlenim omogoča ogled njihovih zahtevkov. Pregled začnite z rezultatom in obsegom pobude. Določite, kaj mora veljati in kaj mora sprememba ohraniti.
Pri tej dobavi zaposleni ne sme brati zahtevka drugega zaposlenega. Vodje morajo ohraniti določen dostop. Test, ki samo odpre stran, ne dokazuje nobenega od teh pogojev.
Povežite zapise
| Zapis | Vprašanje za pregled |
|---|---|
| Pobuda | Kateri rezultat in obseg sta bila odobrena? |
| Različica načrta | Kateri koraki izvedbe in preverjanja so bili predvideni? |
| Zagon | Kaj se je zgodilo in katere predpostavke je sprejel agent? |
| Pull request in diff | Kaj se je spremenilo v trenutnem commitu? |
| Preverjanja in pregled | Katera dokazila podpirajo sprejetje tega commita? |
| Zapis namestitve | Kateri artefakt je dosegel katero okolje? |
Stran Runs beleži poskuse, vključno z neuspešnimi. Vsak zagon določa načrt, ki ga je izvedel. Stran zagona je zapis za pregled; odločitve, ki spreminjajo delo, sodijo na pobudo.
Preberite dokazila korakov za teste in oblikovanje. Taiga pokaže neuspešna ali neizvedena preverjanja. V povzetku pregleda »ni izvedeno« ne pretvorite v »uspešno«.
Preglejte predpostavke in meje
Poiščite predpostavke o modelu dostopa, shemi, okolju in zunanjih storitvah. Primerjajte jih z objavljenim namenom in dejansko kodo.
Pri storitvi opreme preglejte, kje se preverja lastnik zahtevka. Preizkusite zahtevek, do katerega je dostop dovoljen, zahtevek drugega zaposlenega in manjkajoči zahtevek. Preverite, da dnevniki ne razkrivajo zaupne vsebine zahtevkov.
Preglejte tudi spremembe testov. Uspešen rezultat ima omejeno vrednost, če je sprememba odstranila preverbo, ki bi zaznala napako. Spremembe delovnega toka in testne konfiguracije vključite v obseg pregleda.
Dajte povratne informacije za ukrepanje
Določite delovanje, pričakovani rezultat in potrebna dokazila. Na primer: »Končna točka preverja prijavo, ne pa lastništva zahtevka. Dodajte strežniško preverjanje dostopa in test z zahtevkom drugega zaposlenega.«
Taiga se lahko na povratne informacije pregleda pull requesta in neuspešna preverjanja odzove s spremembami na isti veji. Po posodobitvah preglejte novi commit in njegova preverjanja. Prejšnja dokazila morda ne pokrivajo spremenjenega artefakta.
Če se je zagon ustavil zaradi nepopolnega načrta ali oslabljenega preverjanja, preberite navedeni razlog. Statusa osnutka ne odstranite zgolj zato, ker je vidni povzetek preverjanj zelen.
Sprejmite pravilno odločitev o sprejetju
Zabeležite preverjena merila in tista, ki ostajajo nerešena. Zahtevani pregledi in preverjanja repozitorija naj uveljavljajo mejo mergea. Ohranite vsako ločeno odločitev o izdaji.
Taiga spremlja namestitve, ki jih izvaja vaš cevovod. Pred obvestilom uporabnikom, da je sprememba na voljo, preverite okolje in artefakt. Neuspela namestitev lahko pusti prejšnjo uspešno različico v uporabi.
Rezultat zaključite s preverjanjem na ravni storitve: zaposleni lahko uporablja funkcijo, nepooblaščeni dostop je zavrnjen in odgovorni za delovanje lahko opazuje napake. Nadaljujte z obravnavo prekinitve.
Opravite vajo
Izmišljena sprememba dostopa zaposlenih ima uspešno gradnjo in zapis zagona, da integracijskega testa ni bilo mogoče izvesti. Zapišite dokazila, potrebna pred sprejetjem. Vključite en primer zavrnjenega dostopa in točen artefakt ali commit, ki ga pregledujete.
Prenesi delovni list (Markdown)Če počistite to izbiro, izbrišete ves napredek, shranjen v tem brskalniku.
Napredek ostane v tem brskalniku. Brez računa in sledenja.