ping: tests: Cleanup IPOption()s
ClosedPublic
Actions

Authored by jlduran on Oct 16 2024, 5:16 PM.

Details

Reviewers

emaste
kp

Group Reviewers

network

Commits

rG797884a5d990: ping: tests: Cleanup IPOption()s
rG2e29bf7a2ae6: ping: tests: Cleanup IPOption()s

Summary

Scapy now only accepts known options with IPOption(), so our unknown
option must be passed raw.

While here, prefer the named options for EOL and NOP.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 60048
Build 56932: arc lint + arc unit

Event Timeline

jlduran created this revision.Oct 16 2024, 5:16 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 16 2024, 5:16 PM

jlduran requested review of this revision.Oct 16 2024, 5:16 PM

Harbormaster completed remote builds in B60048: Diff 145003.Oct 16 2024, 5:16 PM

jlduran added a reviewer: network.Oct 16 2024, 5:29 PM

kp accepted this revision as: kp.Oct 22 2024, 4:55 PM

This revision is now accepted and ready to land.Oct 22 2024, 4:55 PM

emaste accepted this revision.Oct 22 2024, 5:09 PM

Closed by commit rG2e29bf7a2ae6: ping: tests: Cleanup IPOption()s (authored by jlduran). · Explain WhyNov 5 2024, 2:50 AM

This revision was automatically updated to reflect the committed changes.

jlduran added a commit: rG2e29bf7a2ae6: ping: tests: Cleanup IPOption()s.

jlduran added a commit: rG797884a5d990: ping: tests: Cleanup IPOption()s.Nov 14 2024, 3:26 AM

Revision Contents
Changeset List

Path

Size

sbin/

ping/

tests/

test_ping.py

8 lines

Diff 145003

View Options

sbin/ping/tests/test_ping.py

ping: tests: Cleanup IPOption()sClosedPublicActions