bpfman / bpfman/bpfman-operator

integration tests should validate packet count

Open
#385 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Go
Stars
38
Forks
27
PR merge metrics
No merged PRs in 30d

Description

Some of the integration tests (`doTcCheck()`, `doTcxCheck()`, `doXdpCheck()`) aren't testing for a non-zero count. They are just checking for the string "packets received" and "bytes received", whereas their App equivalent (`doAppTcCheck()`, `doAppTcxCheck()`, `doAppXdpCheck()`) make sure a non-zero count is received. Make these tests more like their App equivalent.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the integration-test implementations of doTcCheck(), doTcxCheck(), and doXdpCheck(), then compare them with doAppTcCheck(), doAppTcxCheck(), and doAppXdpCheck(). Update the integration checks so they validate a non-zero packet count, and run the relevant integration tests to confirm they reject zero packets.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
testing-qa
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.