Skip to content

[WIP] Add round-trip tests for PCap writer and reader#159

Draft
benaadams with Copilot wants to merge 1 commit into
masterfrom
copilot/test-pcap-round-trip
Draft

[WIP] Add round-trip tests for PCap writer and reader#159
benaadams with Copilot wants to merge 1 commit into
masterfrom
copilot/test-pcap-round-trip

Conversation

Copilot AI commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original prompt

This section details on the original issue you should resolve

<issue_title>test: PCap round-trip</issue_title>
<issue_description>## Problem

PCap needs tests covering both the existing writer and the new reader.

Sub-tasks

  • Create test/Magma.PCap.Facts project, add to solution
  • Write N packets via PcapFileWriter, read back via PcapFileReader, assert equality
  • Test both endianness variants
  • Include sample .pcap fixture files from real captures (Wireshark samples)
  • Verify link-layer type propagation

Dependencies

Depends on: #111
Unlocks: #123 (pcap-ng + live capture), also useful for testing other protocol layers against real captures

Phase

Phase 1 (Core Protocol & Tooling)</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: PCap round-trip

2 participants