vapor-ware / synse-server-grpc

The internal gRPC API for Synse Server and its plugins
GNU General Public License v3.0
1 stars 3 forks source link

add unit tests for python client #53

Closed edaniszewski closed 5 years ago

edaniszewski commented 5 years ago

fixes #42 fixes #51

This adds some more unit tests for the python client. I've spent a few hours looking into how to test the grpc methods of the client, but I've had no luck. It turns out that the testing support for grpc (at least for python) is pretty bad, or just not well documented. Since the grpc code is autogenerated and done so in a pretty weird way, its also not easy at all to use standard mocking/monkeypatching techniques.

For now, I'm calling the testing on this done, as I don't think pumping more hours into figuring this out is worthwhile at this point. Maybe later additional unit testing can be revisited.