google/gvisor

Combine similar syscall tests

Open

#1640 opened on Jan 22, 2020

View on 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.

Contributor guide

Tech stack
gocpp
Domain
testingbackend
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
C++gVisor test framework
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
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.