ahmednoursphinx wrote:

> Hey @ahmednoursphinx! Thanks for contributing to lldb! I was wondering why 
> you decided to group the extension tests that way ? Why not make a different 
> test function for each of the extension. That way we would get a fine-grain 
> test results and it'd be easier to catch regressions.

Hey @medismailben thanks for the feedback , the grouping was to reduce test 
execution time since each test calls self.build(), which is expensive. grouping 
reduces rebuilds, but this trades off granularity so i have  separated tests to 
 provide better visibility and maintainability

https://github.com/llvm/llvm-project/pull/169236
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to