Skip to content

Commit 162d3ef

Browse files
leoparenteclaude
andcommitted
fix: load port service names CSV in netflow v5 test
The "Parse netflow v5 stream" test was missing the IpPort::set_csv_iana_ports() call, making it depend on global static state populated by sflow tests that happen to run first. When run in isolation (e.g. --randomize-run-order or filtered by [netflow]), the port map is empty so port 23 resolves to "23" instead of "telnet". Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 4d75943 commit 162d3ef

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

src/handlers/flow/test_flows.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -408,6 +408,7 @@ TEST_CASE("Parse netflow v5 stream", "[netflow][flow]")
408408
FlowInputStream stream{"netflow-test"};
409409
stream.config_set("flow_type", "netflow");
410410
stream.config_set("pcap_file", "tests/fixtures/nf5.pcap");
411+
visor::network::IpPort::set_csv_iana_ports("tests/fixtures/pktvisor-port-service-names.csv");
411412

412413
visor::Config c;
413414
auto stream_proxy = stream.add_event_proxy(c);

0 commit comments

Comments
 (0)