temporalio/cli

[Bug] Dev Server concurrent execution test is incorrect and flaky

Aberta

#581 aberto em 4 de jun. de 2024

Ver no GitHubGuia do colaborador
 (0 comentário) (1 reação) (0 responsável)Go (96 forks)auto 404
buggood first issuetests

Métricas do repositório

Stars
 (370 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Describe the bug

We recently added an integration test that starts a total of 80 dev server, with a maximum of 8 instances running concurrently. This is used to assert, among other things, the reliability of the "get ephemeral port number" logic.

However, that test is incorrect, because it calls FailNow and similar from goroutines, which is not allowed. The test is also flaky. Incorrect usage of test APIs is like not the cause of the flakiness, but may be masking the actual errors, making the issue harder to diagnose.

Given that the CLI's test harness internally use FailNow, fixing this will likely require developing a substitute for the existing test harness.

Guia do colaborador

Direção de pesquisa
Examine o teste que inicia várias instâncias do servidor de desenvolvimento simultaneamente. Encontre onde `FailNow` é chamado de uma goroutine e substitua por uma alternativa segura como `t.Error` ou um canal de erro personalizado. Considere refatorar o test harness para lidar corretamente com concorrência.
Pilha de tecnologia
go
Domain
cli
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Gotesting
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50