Reveja uma entrega da Taiga com evidências
ConcluídoLigue a iniciativa, o plano, a execução, o diff e as verificações. Verifique a alteração atual antes de aceitar uma decisão de merge ou de lançamento.
Publicado por TaigaComo escrevemos
Verifique a sua compreensãoA execução terminou, mas o registo indica que um teste obrigatório não foi executado. O que demonstra a conclusão?Faça o exercício
O que vai aprender
- Rastrear um comportamento entregue até ao respetivo requisito e plano.
- Identificar verificações incompletas e pressupostos que precisam de revisão.
- Distinguir conclusão da execução, merge, deployment e disponibilidade para o utilizador.
Comece pelo resultado da iniciativa
O serviço fictício de equipamento já permite aos trabalhadores ver os seus próprios pedidos. Comece a revisão pelo resultado e pelo âmbito da iniciativa. Identifique o que tem de ser verdadeiro e o que a alteração tem de preservar.
Nesta entrega, um trabalhador não pode ler o pedido de outro. Os gestores têm de manter o acesso definido. Um teste que apenas abre a página não demonstra nenhuma destas condições.
Ligue os registos
| Registo | Pergunta de revisão |
|---|---|
| Iniciativa | Que resultado e âmbito foram autorizados? |
| Versão do plano | Que passos de implementação e verificação estavam previstos? |
| Execução | O que aconteceu e que pressupostos fez o agente? |
| Pull request e diff | O que mudou no commit atual? |
| Verificações e revisão | Que evidências sustentam a aceitação desse commit? |
| Registo de deployment | Que artefacto chegou a que ambiente? |
A página Runs regista tentativas, incluindo falhas. Cada execução identifica o plano que executou. Uma página de execução é um registo para inspeção. As decisões que alteram o trabalho pertencem à iniciativa.
Leia as evidências dos passos de testes e formatação. A Taiga torna visíveis as verificações falhadas ou não executadas. Não transforme «não executado» em «aprovado» no resumo da revisão.
Examine os pressupostos e os limites
Procure pressupostos sobre o modelo de acesso, o esquema, o ambiente e os serviços externos. Compare-os com a intenção publicada e com o código real.
No serviço de equipamento, examine onde se verifica o titular do pedido. Teste um pedido autorizado, o pedido de outro trabalhador e um pedido inexistente. Verifique que os logs não revelam conteúdo confidencial dos pedidos.
Reveja também as alterações aos testes. Um resultado bem-sucedido tem valor limitado se a alteração removeu a asserção que detetaria o defeito. Inclua no âmbito da revisão as alterações aos fluxos e à configuração dos testes.
Dê comentários que permitam agir
Identifique o comportamento, o resultado esperado e as evidências necessárias. Por exemplo: «O endpoint verifica o login, mas não a titularidade do pedido. Acrescente a verificação de acesso no servidor e um teste com o pedido de outro trabalhador.»
A Taiga pode responder aos comentários de revisão de pull requests e a verificações falhadas com alterações na mesma branch. Depois das atualizações, examine o novo commit e as suas verificações. As evidências anteriores podem não abranger um artefacto alterado.
Se a execução parou porque o plano estava incompleto ou uma verificação foi enfraquecida, leia a razão indicada. Não retire o estado de rascunho apenas porque o resumo visível das verificações está verde.
Tome a decisão de aceitação correta
Registe os critérios verificados e os que continuam por resolver. Deixe as revisões e verificações obrigatórias do repositório impor o limite de merge. Preserve qualquer decisão separada de lançamento.
A Taiga observa os deployments executados pela sua pipeline. Verifique o ambiente e o artefacto antes de dizer aos utilizadores que a alteração está disponível. Um deployment falhado pode deixar a versão anterior bem-sucedida a servir tráfego.
Conclua com uma verificação ao nível do serviço: o trabalhador consegue usar a funcionalidade, o acesso não autorizado é recusado e o responsável operacional consegue observar falhas. Prossiga para tratar uma interrupção.
Faça o exercício
Uma alteração fictícia ao acesso dos trabalhadores tem um build bem-sucedido e uma nota de execução que indica que um teste de integração não pôde ser executado. Escreva as evidências necessárias antes de a aceitar. Inclua um caso de acesso recusado e o artefacto ou commit exato em revisão.
Descarregar ficha (Markdown)Desmarcar esta opção elimina todo o progresso guardado neste browser.
O progresso fica neste browser. Sem conta nem rastreamento.