Context
Follow-up from PR #353 review (FFI panic boundaries).
Problem
The panic guards added in PR #353 have unit tests verifying each guard catches panics and returns correct sentinel values, but there is no integration test that actually loads a .so plugin which panics and asserts that the pipeline survives.
Suggested approach
- Create a small test plugin crate (e.g.
tests/fixtures/panicking-plugin/) that panics in process_packet().
- Build it as a
.so during cargo test.
- Load it via
NativeNodeWrapper, send a packet, and assert the host receives an error (not an abort).
Labels
testing, plugin-sdk
Context
Follow-up from PR #353 review (FFI panic boundaries).
Problem
The panic guards added in PR #353 have unit tests verifying each guard catches panics and returns correct sentinel values, but there is no integration test that actually loads a
.soplugin which panics and asserts that the pipeline survives.Suggested approach
tests/fixtures/panicking-plugin/) that panics inprocess_packet()..soduringcargo test.NativeNodeWrapper, send a packet, and assert the host receives an error (not an abort).Labels
testing, plugin-sdk