Re: [Wireshark-dev] Unit testing dissector code

2021-06-18 Thread João Valverde via Wireshark-dev
On 15/06/21 05:02, João Valverde via Wireshark-dev wrote: On 14/06/21 22:01, Martin Nyhus wrote: On 05/06/2021 02:33, João Valverde wrote: But regarding your PoC having to give extern linkage to the internal dissector code is a big drawback IMO, even if it isn't visible in a DLL (because we

Re: [Wireshark-dev] Unit testing dissector code

2021-06-14 Thread João Valverde via Wireshark-dev
On 14/06/21 22:01, Martin Nyhus wrote: On 05/06/2021 02:33, João Valverde wrote: But regarding your PoC having to give extern linkage to the internal dissector code is a big drawback IMO, even if it isn't visible in a DLL (because we use default hidden visibility when the compiler supports it)

Re: [Wireshark-dev] Unit testing dissector code

2021-06-14 Thread Martin Nyhus
On 05/06/2021 02:33, João Valverde wrote: > But regarding your PoC having to give extern linkage to the internal > dissector code is a big drawback IMO, even if it isn't visible in a DLL > (because we use default hidden visibility when the compiler supports it). > > Maybe that could be solved by in

Re: [Wireshark-dev] Unit testing dissector code

2021-06-04 Thread João Valverde via Wireshark-dev
Hi Martin, This is promising. I think dissecting a TVB and walking the proto_tree to assert the result is probably the way to go about implementing a dissector test suite (instead of reading a pcap with tshark and grepping the output). But regarding your PoC having to give extern linkage to

[Wireshark-dev] Unit testing dissector code

2021-03-01 Thread Martin Nyhus
Hi, since the happy-shark discussion died out I've been looking at other ways of adding test coverage for some bug fixes I've been preparing to push, and right now the idea is to add a new unit test suite based on wmem_test for dissector code. I think this should solve the concern that was mentione