[Lldb-commits] [PATCH] D43048: [lldb-test/WIP] Allow a way to test autocompletion

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Feb 7 16:20:12 PST 2018


zturner added a comment.

On the issue of keeping the other test, I think when an SB API method is basically a pass-through to a private method, then having a test of the SB API method that verifies "did the correct native method get called" is useful if for no other reason than to verify the correctness of the SWIG binding generation.  If you've tested that the API method invokes the correct native method, and you've tested the native method directly with a large amount of inputs, then that's sufficient overall coverage IMO


https://reviews.llvm.org/D43048





More information about the lldb-commits mailing list