rpc: fix 0-ady functions raising an exception

epics-base / p4p

Python bindings for the PVAccess network client and server.

BSD 3-Clause "New" or "Revised" License

27 stars 38 forks source link

rpc: fix 0-ady functions raising an exception #134

Closed Synthetica9 closed 2 months ago

Synthetica9 commented 10 months ago

Closes #130

mdavidsaver commented 10 months ago

For this PR and #131, it would be helpful to have added test coverage to TestProxy in src/p4p/test/test_rpc.py. Including cases where eg. the client-side proxy should throw an exception due to malformed call arguments.

mdavidsaver commented 2 months ago

Thank you for this PR. However, I have applied a different fix with 4f6b9e9e67293acd142cf1ce377c87da9e6ab549.