google/gvisor

Combine similar syscall tests

Open

#1 640 ouverte le 22 janv. 2020

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)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.

Guide contributeur

Stack technique
gocpp
Domaine
testingbackend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
C++gVisor test framework
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
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.