google/gvisor

Combine similar syscall tests

Open

#1.640 aberto em 22 de jan. de 2020

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Go (12.713 stars) (1.061 forks)batch import
area: testsexportedgood first issuepriority: p2type: bug

Description

Some gVisor syscall tests are almost identical and can easily be combined. This requires changes to the parameterized test parameter.

Example tests are UDPBroadcastSendRecvOnSocketBoundToAny and UDPBroadcastSendRecvOnSocketBoundToBroadcast in socket_ipv4_udp_unbound_external_networking.cc.

Guia do colaborador

Pilha de tecnologia
gocpp
Domain
testingbackend
Tipo Issue
refactor
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.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
C++gVisor test framework
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
Look at the file socket ipv4 udp unbound external networking.cc in the gVisor repository. Identify the two similar test functions UDPBroadcastSendRecvOnSocketBoundToAny and UDPBroadcastSendRecvOnSocketBoundToBroadcast. Determine how to combine them by parameterizing the test parameter. Ensure the combined test still covers both cases. No pull request is linked, so this is open for contribution.